Rollup: replace config is wrong

[xirzec]

Per https://github.com/rollup/plugins/tree/master/packages/replace

Our replace config should look like

    replace({
        delimiters: ["", ""],
        // replace dynamic checks with if (true) since this is for node only.
        // Allows rollup's dead code elimination to be more aggressive.
        "if (isNode)": "if (true)"
      }),

instead of

   replace({
        delimiters: ["", ""],
        values: {
          // replace dynamic checks with if (true) since this is for node only.
          // Allows rollup's dead code elimination to be more aggressive.
          "if (isNode)": "if (true)"
        }
      }),

Which doesn't replace anything anymore. Not sure if this changed since a previous plugin version, but tested it on core-http.

Not a big deal since we're not shipping browser bundles, but still seems like it should be cleaned up.

[jeremymeng]

Fixed in #8909