[DevTools] Remove Browser domain, switch clients to Target.

[DevTools] Remove Browser domain, switch clients to Target.

BUG=648775
Committed: https://crrev.com/a91888d84288248b3a2b9c301cdaa4e46c5ee22e
Cr-Commit-Position: refs/heads/master@{#427483}

[dgozman, 2016-10-20 01:07:59]

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

[commit-bot: I haz the power, 2016-10-20 01:08:34]

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, 2016-10-20 01:12:27]

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

[commit-bot: I haz the power, 2016-10-20 01:12:30]

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

[dgozman, 2016-10-20 20:08:10]

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

[commit-bot: I haz the power, 2016-10-20 20:08:47]

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, 2016-10-20 21:07:16]

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

[commit-bot: I haz the power, 2016-10-20 21:07:17]

Dry run: Try jobs failed on following builders:
  cast_shell_linux on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/cast_shell_linu...)

[dgozman, 2016-10-20 23:24:11]

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

[commit-bot: I haz the power, 2016-10-20 23:24:43]

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

[dgozman, 2016-10-21 00:52:40]

dgozman@chromium.org changed reviewers:
+ pfeldman@chromium.org, skyostil@chromium.org

[dgozman, 2016-10-21 00:52:41]

Could you please take a look?

[pfeldman, 2016-10-21 01:02:44]

lgtm

[commit-bot: I haz the power, 2016-10-21 02:49:10]

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

[commit-bot: I haz the power, 2016-10-21 02:49:11]

Dry run: This issue passed the CQ dry run.

[dgozman, 2016-10-21 17:54:05]

Description was changed from

==========
[DevTools] Remove Browser domain, switch clients to Target.

BUG=648775
==========

to

==========
[DevTools] Remove Browser domain, switch clients to Target.

BUG=648775
==========

[dgozman, 2016-10-21 17:54:05]

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

[dgozman, 2016-10-21 17:55:31]

dgozman@chromium.org changed reviewers:
+ alexclarke@chromium.org
- alexclarke@chomium.org

[dgozman, 2016-10-21 17:56:05]

Alex, could you please take a look?

[alex clarke (OOO till 29th), 2016-10-24 10:07:33]

Looks like the headless bot is currently transitioning to a release version on
the main waterfall.  I'll patch this in locally and see if things still run.

[Sami, 2016-10-25 17:48:34]

FYI looks like headless/public/util/dom_tree_extractor_browsertest.cc needs some
updates for this.

[dgozman, 2016-10-25 18:46:47]

On 2016/10/25 17:48:34, Sami wrote:
> FYI looks like headless/public/util/dom_tree_extractor_browsertest.cc needs
some
> updates for this.

Thanks, unnecessary include sneaked in! Please take another look.

[dgozman, 2016-10-25 18:47:09]

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

[commit-bot: I haz the power, 2016-10-25 18:47:43]

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

[alex clarke (OOO till 29th), 2016-10-25 19:54:10]

On 2016/10/25 18:46:47, dgozman wrote:
> On 2016/10/25 17:48:34, Sami wrote:
> > FYI looks like headless/public/util/dom_tree_extractor_browsertest.cc needs
> some
> > updates for this.
> 
> Thanks, unnecessary include sneaked in! Please take another look.

I patched this in locally and the headless_browsertests pass. LGTM

[commit-bot: I haz the power, 2016-10-25 20:36:20]

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

[commit-bot: I haz the power, 2016-10-25 20:36:21]

Dry run: This issue passed the CQ dry run.

[dgozman, 2016-10-25 21:25:48]

The CQ bit was checked by dgozman@chromium.org

[dgozman, 2016-10-25 21:25:49]

The patchset sent to the CQ was uploaded after l-g-t-m from
pfeldman@chromium.org
Link to the patchset: https://codereview.chromium.org/2436763003/#ps40001
(title: "include, rebase")

[commit-bot: I haz the power, 2016-10-25 21:26:36]

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-10-25 21:34:23]

Description was changed from

==========
[DevTools] Remove Browser domain, switch clients to Target.

BUG=648775
==========

to

==========
[DevTools] Remove Browser domain, switch clients to Target.

BUG=648775
==========

[commit-bot: I haz the power, 2016-10-25 21:34:25]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-10-25 21:36:01]

Description was changed from

==========
[DevTools] Remove Browser domain, switch clients to Target.

BUG=648775
==========

to

==========
[DevTools] Remove Browser domain, switch clients to Target.

BUG=648775

Committed: https://crrev.com/a91888d84288248b3a2b9c301cdaa4e46c5ee22e
Cr-Commit-Position: refs/heads/master@{#427483}
==========

[commit-bot: I haz the power, 2016-10-25 21:36:02]

Patchset 3 (id:??) landed as
https://crrev.com/a91888d84288248b3a2b9c301cdaa4e46c5ee22e
Cr-Commit-Position: refs/heads/master@{#427483}

[dgozman, 2016-10-25 21:38:31]

On 2016/10/25 19:54:10, alex clarke wrote:
> On 2016/10/25 18:46:47, dgozman wrote:
> > On 2016/10/25 17:48:34, Sami wrote:
> > > FYI looks like headless/public/util/dom_tree_extractor_browsertest.cc
needs
> > some
> > > updates for this.
> > 
> > Thanks, unnecessary include sneaked in! Please take another look.
> 
> I patched this in locally and the headless_browsertests pass. LGTM

Thank you for review!