DevTools: Use resource name to detect toolbox web contents resource

DevTools: Use resource name to detect toolbox web contents resource

This is the Chromium part of a two-sided patch.
The Blink part is https://codereview.chromium.org/615893004

R=dgozman, vsevik

Committed: https://crrev.com/2ffa5f6b52416fe1684ec64b36b876641f3a9d6b
Cr-Commit-Position: refs/heads/master@{#297806}

[dgozman, 2014-10-02 10:08:10]

lgtm

https://codereview.chromium.org/620953003/diff/1/chrome/browser/devtools/devt...
File chrome/browser/devtools/devtools_window.cc (right):

https://codereview.chromium.org/620953003/diff/1/chrome/browser/devtools/devt...
chrome/browser/devtools/devtools_window.cc:864:
target_url.path().rfind("toolbox.html") != std::string::npos) {
There is no difference between find and rfind here.

[apavlov, 2014-10-02 10:10:23]

https://codereview.chromium.org/620953003/diff/1/chrome/browser/devtools/devt...
File chrome/browser/devtools/devtools_window.cc (right):

https://codereview.chromium.org/620953003/diff/1/chrome/browser/devtools/devt...
chrome/browser/devtools/devtools_window.cc:864:
target_url.path().rfind("toolbox.html") != std::string::npos) {
On 2014/10/02 10:08:10, dgozman wrote:
> There is no difference between find and rfind here.

We know that it will be A TAD faster :)

[apavlov, 2014-10-02 10:15:00]

The CQ bit was checked by apavlov@chromium.org

[apavlov, 2014-10-02 10:15:49]

apavlov@chromium.org changed reviewers:
+ yurys@chromium.org

[commit-bot: I haz the power, 2014-10-02 10:15:55]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/620953003/1

[commit-bot: I haz the power, 2014-10-02 11:36:49]

Committed patchset #1 (id:1) as 704fa2f3912e329ee4b34d0e74fcbae8023348ca

[commit-bot: I haz the power, 2014-10-02 11:37:22]

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