DevTools: add tests to fixate bindings behavior

DevTools: add tests to fixate bindings behavior
    
This patch adds a bunch of tests to fixate bindings behavior:
- bindings-* tests ensure that workspace is populated correctly
- navigator-* tests ensure that navigator is populated correctly

The binding-* and navigator-* tests are very similar, but since
navigator might rely less on workspace in future, they're kept separate.

These tests will help to evaluate changes which will happen to bindings
and navigator during bindings refactoring.

BUG=670180
R=dgozman

Review-Url: https://codereview.chromium.org/2735933002
Cr-Commit-Position: refs/heads/master@{#455531}
Committed: https://chromium.googlesource.com/chromium/src/+/f4fbdf6154d37fcb940bfd37b15b2c2bb0e5ada0

[lushnikov, 2017-03-07 02:15:27]

please, take a look

[lushnikov, 2017-03-07 02:15:51]

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

[commit-bot: I haz the power, 2017-03-07 02:16:58]

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-07 03:58:21]

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

[commit-bot: I haz the power, 2017-03-07 03:58:22]

Dry run: Try jobs failed on following builders:
  mac_chromium_rel_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[lushnikov, 2017-03-07 06:00:01]

Description was changed from

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

This patch adds a bunch of tests to fixate bindings behavior under
different conditions, such as frame attachment/detachment.

BUG=670180
R=dgozman
==========

to

==========
DevTools: add tests to fixate bindings behavior
    
This patch adds a bunch of tests to fixate bindings behavior:
- bindings-* tests ensure that workspace is populated correctly
- navigator-* tests ensure that navigator is populated correctly

The binding-* and navigator-* tests are very similar, but since
navigator might rely less on workspace in future, they're kept separate.

BUG=670180
R=dgozman
==========

[lushnikov, 2017-03-07 06:03:55]

Description was changed from

==========
DevTools: add tests to fixate bindings behavior
    
This patch adds a bunch of tests to fixate bindings behavior:
- bindings-* tests ensure that workspace is populated correctly
- navigator-* tests ensure that navigator is populated correctly

The binding-* and navigator-* tests are very similar, but since
navigator might rely less on workspace in future, they're kept separate.

BUG=670180
R=dgozman
==========

to

==========
DevTools: add tests to fixate bindings behavior
    
This patch adds a bunch of tests to fixate bindings behavior:
- bindings-* tests ensure that workspace is populated correctly
- navigator-* tests ensure that navigator is populated correctly

The binding-* and navigator-* tests are very similar, but since
navigator might rely less on workspace in future, they're kept separate.

These tests will help to evaluate changes which will happen to bindings
and navigator during bindings refactoring.

BUG=670180
R=dgozman
==========

[lushnikov, 2017-03-07 06:04:02]

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

[commit-bot: I haz the power, 2017-03-07 06:04:53]

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-07 07:16:38]

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

[commit-bot: I haz the power, 2017-03-07 07:16:39]

Dry run: This issue passed the CQ dry run.

[dgozman, 2017-03-07 20:05:25]

lgtm

[lushnikov, 2017-03-08 00:24:42]

The CQ bit was checked by lushnikov@chromium.org

[lushnikov, 2017-03-08 00:24:43]

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/2735933002/#ps40001
(title: "markstep")

[commit-bot: I haz the power, 2017-03-08 00:25:43]

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 01:57:56]

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

[commit-bot: I haz the power, 2017-03-08 01:57:57]

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

[lushnikov, 2017-03-08 18:03:28]

The CQ bit was checked by lushnikov@chromium.org

[lushnikov, 2017-03-08 18:03:28]

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/2735933002/#ps60001
(title: "nit")

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

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

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

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

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

[lushnikov, 2017-03-08 19:30:13]

The CQ bit was checked by lushnikov@chromium.org

[lushnikov, 2017-03-08 19:30:13]

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/2735933002/#ps80001
(title: "DevTools: add tests to fixate bindings behavior")

[commit-bot: I haz the power, 2017-03-08 19:31:11]

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 20:38:18]

CQ is committing da patch.

Bot data: {"patchset_id": 80001, "attempt_start_ts": 1489001413237790,
"parent_rev": "199953cd647ec50f5f07d9025e5a8ba6948f8079", "commit_rev":
"f4fbdf6154d37fcb940bfd37b15b2c2bb0e5ada0"}

[commit-bot: I haz the power, 2017-03-08 20:39:04]

Description was changed from

==========
DevTools: add tests to fixate bindings behavior
    
This patch adds a bunch of tests to fixate bindings behavior:
- bindings-* tests ensure that workspace is populated correctly
- navigator-* tests ensure that navigator is populated correctly

The binding-* and navigator-* tests are very similar, but since
navigator might rely less on workspace in future, they're kept separate.

These tests will help to evaluate changes which will happen to bindings
and navigator during bindings refactoring.

BUG=670180
R=dgozman
==========

to

==========
DevTools: add tests to fixate bindings behavior
    
This patch adds a bunch of tests to fixate bindings behavior:
- bindings-* tests ensure that workspace is populated correctly
- navigator-* tests ensure that navigator is populated correctly

The binding-* and navigator-* tests are very similar, but since
navigator might rely less on workspace in future, they're kept separate.

These tests will help to evaluate changes which will happen to bindings
and navigator during bindings refactoring.

BUG=670180
R=dgozman

Review-Url: https://codereview.chromium.org/2735933002
Cr-Commit-Position: refs/heads/master@{#455531}
Committed:
https://chromium.googlesource.com/chromium/src/+/f4fbdf6154d37fcb940bfd37b15b...
==========

[commit-bot: I haz the power, 2017-03-08 20:39:05]

Committed patchset #5 (id:80001) as
https://chromium.googlesource.com/chromium/src/+/f4fbdf6154d37fcb940bfd37b15b...