DevTools: move front-end URL handling to DevToolsUIBindingds

DevTools: move front-end URL handling to DevToolsUIBindingds

BUG=662859
Committed: https://crrev.com/eea3300239f0b53e172a320eb8de59d0bea65f27
Cr-Commit-Position: refs/heads/master@{#440926}

[caseq, 2016-12-28 23:05:44]

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

[caseq, 2016-12-28 23:49:29]

Patchset #1 (id:1) has been deleted

[caseq, 2016-12-28 23:51:24]

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

[commit-bot: I haz the power, 2016-12-28 23:51:40]

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-12-29 00:44:45]

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

[commit-bot: I haz the power, 2016-12-29 00:44:46]

Dry run: This issue passed the CQ dry run.

[dgozman, 2016-12-29 01:16:50]

lgtm

https://codereview.chromium.org/2607833002/diff/20001/chrome/browser/devtools...
File chrome/browser/devtools/devtools_ui_bindings.cc (right):

https://codereview.chromium.org/2607833002/diff/20001/chrome/browser/devtools...
chrome/browser/devtools/devtools_ui_bindings.cc:1015: NOTREACHED();
This could actually happen, because navigation is async wrt our messages.

https://codereview.chromium.org/2607833002/diff/20001/chrome/browser/devtools...
chrome/browser/devtools/devtools_ui_bindings.cc:1204: GURL url =
web_contents_->GetVisibleURL();
Wrong indent.

[caseq, 2016-12-29 01:19:41]

The CQ bit was checked by caseq@chromium.org

[caseq, 2016-12-29 01:19:41]

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/2607833002/#ps40001
(title: "review comments addressed")

[commit-bot: I haz the power, 2016-12-29 01:19:53]

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-29 02:19:22]

CQ is committing da patch.

Bot data: {"patchset_id": 40001, "attempt_start_ts": 1482974381108430,
"parent_rev": "7b8226bd1185796cbcf52473603033f93247bd93", "commit_rev":
"45deb48f70797610e1049ad31bb7182bc97d3727"}

[commit-bot: I haz the power, 2016-12-29 02:19:48]

Description was changed from

==========
DevTools: move front-end URL handling to DevToolsUIBindingds

BUG=662859
==========

to

==========
DevTools: move front-end URL handling to DevToolsUIBindingds

BUG=662859

Review-Url: https://codereview.chromium.org/2607833002
==========

[commit-bot: I haz the power, 2016-12-29 02:19:48]

Committed patchset #2 (id:40001)

[commit-bot: I haz the power, 2017-01-02 15:51:27]

Description was changed from

==========
DevTools: move front-end URL handling to DevToolsUIBindingds

BUG=662859

Review-Url: https://codereview.chromium.org/2607833002
==========

to

==========
DevTools: move front-end URL handling to DevToolsUIBindingds

BUG=662859

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

[commit-bot: I haz the power, 2017-01-02 15:51:27]

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

[caseq, 2017-01-10 23:58:10]

A revert of this CL (patchset #2 id:40001) has been created in
https://codereview.chromium.org/2620193002/ by caseq@chromium.org.

The reason for reverting is: A better fix is coming, reverting this one to make
it a part of the new fix so that one CL may be merged onto a branch..