DevTools: omit remoteBase query param when starting a shared worker

DevTools: omit remoteBase query param when starting a shared worker

Shared workers can't be started with mismatching URLs, so strip
remoteBase when starting a shared worker.

Committed: https://crrev.com/9008feb1e26578cfd503ecc196adaf6ffd080a49
Cr-Commit-Position: refs/heads/master@{#436408}

[caseq, 2016-12-02 19:51:53]

caseq@chromium.org changed reviewers:
+ dgozman@chromium.org

[dgozman, 2016-12-02 19:55:53]

https://codereview.chromium.org/2543343002/diff/1/third_party/WebKit/Source/d...
File third_party/WebKit/Source/devtools/front_end/services/ServiceManager.js
(right):

https://codereview.chromium.org/2543343002/diff/1/third_party/WebKit/Source/d...
third_party/WebKit/Source/devtools/front_end/services/ServiceManager.js:39: var
worker = isSharedWorker ? new SharedWorker(url, appName) : new Worker(url);
Should we instead pass different appName for different urls?

[dgozman, 2016-12-02 20:19:31]

lgtm per offline discussion

[caseq, 2016-12-02 21:27:03]

The CQ bit was checked by caseq@chromium.org

[commit-bot: I haz the power, 2016-12-02 21:28:13]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-12-03 01:19:17]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-12-03 01:19:18]

Try jobs failed on following builders:
  win_chromium_rel_ng on master.tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_rel_...)

[caseq, 2016-12-05 19:00:50]

The CQ bit was checked by caseq@chromium.org

[commit-bot: I haz the power, 2016-12-05 19:01:13]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-12-05 21:21:34]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1480964450420420, "parent_rev":
"f53cab5a8598de5ead724b72dcb928835fdaee23", "commit_rev":
"cecf4dd273a401d162320dbaa8b51adcb4b97457"}

[commit-bot: I haz the power, 2016-12-05 21:22:01]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-12-05 21:24:20]

Description was changed from

==========
DevTools: omit remoteBase query param when starting a shared worker

Shared workers can't be started with mismatching URLs, so strip
remoteBase when starting a shared worker.
==========

to

==========
DevTools: omit remoteBase query param when starting a shared worker

Shared workers can't be started with mismatching URLs, so strip
remoteBase when starting a shared worker.

Committed: https://crrev.com/9008feb1e26578cfd503ecc196adaf6ffd080a49
Cr-Commit-Position: refs/heads/master@{#436408}
==========

[commit-bot: I haz the power, 2016-12-05 21:24:21]

Patchset 1 (id:??) landed as
https://crrev.com/9008feb1e26578cfd503ecc196adaf6ffd080a49
Cr-Commit-Position: refs/heads/master@{#436408}