DevTools: add tests to fixate sourcemap bindings behavior

DevTools: add tests to fixate sourcemap bindings behavior

These tests fixate how source map bindings, such as
CompilerScriptMapping and SASSSourceMapping, produce sources
correctly for frames.

As these tests reveal, CompilerScriptMapping currently doesn't
attach sources for all frames.

BUG=670180, 685552
R=dgozman

Review-Url: https://codereview.chromium.org/2731403009
Cr-Commit-Position: refs/heads/master@{#455547}
Committed: https://chromium.googlesource.com/chromium/src/+/95ff5bc751ba619a7c37155eec85c1adcadb01bc

[lushnikov, 2017-03-08 18:17:44]

please, take a look

[lushnikov, 2017-03-08 18:17:50]

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

[commit-bot: I haz the power, 2017-03-08 18:18:09]

Dry run: 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-08 18:22:01]

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

[commit-bot: I haz the power, 2017-03-08 18:22:02]

Dry run: Try jobs failed on following builders:
  ios-device-xcode-clang on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-device-xcode-...)
  ios-simulator on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-simulator/bui...)
  ios-simulator-xcode-clang on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-simulator-xco...)

[dgozman, 2017-03-08 19:07:09]

lgtm

https://codereview.chromium.org/2731403009/diff/1/third_party/WebKit/LayoutTe...
File
third_party/WebKit/LayoutTests/http/tests/inspector/bindings/bindings-test.js
(right):

https://codereview.chromium.org/2731403009/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/http/tests/inspector/bindings/bindings-test.js:73:
sourceMapCallbacks.delete(urlSuffix);
nit: delete callback before calling it for re-entrance.

[lushnikov, 2017-03-08 19:42:18]

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

[commit-bot: I haz the power, 2017-03-08 19:42:56]

Dry run: 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-08 21:18:32]

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

[commit-bot: I haz the power, 2017-03-08 21:18:33]

Dry run: This issue passed the CQ dry run.

[lushnikov, 2017-03-08 21:33:08]

https://codereview.chromium.org/2731403009/diff/1/third_party/WebKit/LayoutTe...
File
third_party/WebKit/LayoutTests/http/tests/inspector/bindings/bindings-test.js
(right):

https://codereview.chromium.org/2731403009/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/http/tests/inspector/bindings/bindings-test.js:73:
sourceMapCallbacks.delete(urlSuffix);
On 2017/03/08 19:07:09, dgozman wrote:
> nit: delete callback before calling it for re-entrance.

That's not needed since this is promise resolution

[lushnikov, 2017-03-08 21:33:14]

The CQ bit was checked by lushnikov@chromium.org

[lushnikov, 2017-03-08 21:33:14]

The patchset sent to the CQ was uploaded after l-g-t-m from dgozman@chromium.org
Link to the patchset: https://codereview.chromium.org/2731403009/#ps20001
(title: "rebaseline")

[commit-bot: I haz the power, 2017-03-08 21:33:47]

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-08 21:40:07]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1489008794235200,
"parent_rev": "2734438cde42ce98227c4292b35a092fb75ee371", "commit_rev":
"95ff5bc751ba619a7c37155eec85c1adcadb01bc"}

[commit-bot: I haz the power, 2017-03-08 21:41:11]

Description was changed from

==========
DevTools: add tests to fixate sourcemap bindings behavior

These tests fixate how source map bindings, such as
CompilerScriptMapping and SASSSourceMapping, produce sources
correctly for frames.

As these tests reveal, CompilerScriptMapping currently doesn't
attach sources for all frames.

BUG=670180,685552
R=dgozman
==========

to

==========
DevTools: add tests to fixate sourcemap bindings behavior

These tests fixate how source map bindings, such as
CompilerScriptMapping and SASSSourceMapping, produce sources
correctly for frames.

As these tests reveal, CompilerScriptMapping currently doesn't
attach sources for all frames.

BUG=670180,685552
R=dgozman

Review-Url: https://codereview.chromium.org/2731403009
Cr-Commit-Position: refs/heads/master@{#455547}
Committed:
https://chromium.googlesource.com/chromium/src/+/95ff5bc751ba619a7c37155eec85...
==========

[commit-bot: I haz the power, 2017-03-08 21:41:12]

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