DevTools: Support wss:// endpoints

DevTools: Support wss:// endpoints

BUG=

Review-Url: https://codereview.chromium.org/2729623006
Cr-Commit-Position: refs/heads/master@{#456258}
Committed: https://chromium.googlesource.com/chromium/src/+/8925c3c45f3923bc78ffc841842183cc592a0143

[paulirish, 2017-03-03 06:41:22]

The CQ bit was checked by paulirish@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-03-03 06:41:30]

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

[paulirish, 2017-03-03 20:59:10]

paulirish@chromium.org changed reviewers:
+ pfeldman@chromium.org

[dgozman, 2017-03-05 06:08:12]

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

[dgozman, 2017-03-05 06:08:13]

https://codereview.chromium.org/2729623006/diff/20001/third_party/WebKit/Sour...
File third_party/WebKit/Source/devtools/front_end/sdk/TargetManager.js (right):

https://codereview.chromium.org/2729623006/diff/20001/third_party/WebKit/Sour...
third_party/WebKit/Source/devtools/front_end/sdk/TargetManager.js:428: var ws =
wsParam ? `ws://${wsParam}` : `wss://${wssParam}`;
Last time we checked wss was only allowed from secure origin, and
chrome-devtools://devtools was not secure. Did that change?

[pfeldman, 2017-03-06 18:08:22]

it does not hurt, lgtm

[paulirish, 2017-03-10 23:25:02]

The CQ bit was checked by paulirish@chromium.org

[commit-bot: I haz the power, 2017-03-10 23:25:26]

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, 2017-03-11 00:49:49]

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

[commit-bot: I haz the power, 2017-03-11 00:49:51]

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_...)

[paulirish, 2017-03-11 00:57:20]

The CQ bit was checked by paulirish@chromium.org

[commit-bot: I haz the power, 2017-03-11 00:57:45]

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, 2017-03-11 02:18:48]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1489193840131830,
"parent_rev": "085571c2c97e2915a1e40262f6d1aaf20e9defb7", "commit_rev":
"8925c3c45f3923bc78ffc841842183cc592a0143"}

[commit-bot: I haz the power, 2017-03-11 02:19:25]

Description was changed from

==========
DevTools: Support wss:// endpoints

BUG=
==========

to

==========
DevTools: Support wss:// endpoints

BUG=

Review-Url: https://codereview.chromium.org/2729623006
Cr-Commit-Position: refs/heads/master@{#456258}
Committed:
https://chromium.googlesource.com/chromium/src/+/8925c3c45f3923bc78ffc8418421...
==========

[commit-bot: I haz the power, 2017-03-11 02:19:26]

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/chromium/src/+/8925c3c45f3923bc78ffc8418421...