DevTools: set extension name as human-readable name of isolated world

DevTools: set extension name as human-readable name of isolated world

The patch sets up human-readable name of isolated worlds, generated by
extension, to the name of the parent extension. This will be very useful
for extension debugging in developer tools.

BUG=160977
R=vsevik, yurys

Committed: https://crrev.com/b53144f3c6a8d6e1fcafa4da544aed75e7b94322
Cr-Commit-Position: refs/heads/master@{#293095}

[lushnikov, 2014-08-23 20:49:30]

PTAL

This is a chromium-side of the patch.
The blink side: https://codereview.chromium.org/15764004/

[yurys, 2014-08-25 09:25:52]

lgtm

https://codereview.chromium.org/500823002/diff/1/extensions/renderer/script_i...
File extensions/renderer/script_injection.cc (right):

https://codereview.chromium.org/500823002/diff/1/extensions/renderer/script_i...
extensions/renderer/script_injection.cc:87:
frame->setIsolatedWorldHumanReadableName(
Should it be added to translate_helper.cc as well?

[lushnikov, 2014-08-30 18:19:00]

https://codereview.chromium.org/500823002/diff/1/extensions/renderer/script_i...
File extensions/renderer/script_injection.cc (right):

https://codereview.chromium.org/500823002/diff/1/extensions/renderer/script_i...
extensions/renderer/script_injection.cc:87:
frame->setIsolatedWorldHumanReadableName(
On 2014/08/25 09:25:52, yurys(OOO) wrote:
> Should it be added to translate_helper.cc as well?

The translate extension already has readable origin
(https://translate.googleapis.com/) which is displayed in context switcher by
default.

The point of the patch is to fight chrome-extension:// url's in devtools context
switcher which are very hard to distinguish.

[lushnikov, 2014-08-30 18:21:24]

lushnikov@chromium.org changed reviewers:
+ rockot@chromium.org

[lushnikov, 2014-08-30 18:21:24]

@rockot, could you please take a look on the patch?

[Ken Rockot(use gerrit already), 2014-09-02 22:55:14]

lgtm

[lushnikov, 2014-09-03 05:28:46]

The CQ bit was checked by lushnikov@chromium.org

[commit-bot: I haz the power, 2014-09-03 05:35:17]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/lushnikov@chromium.org/500823002/1

[commit-bot: I haz the power, 2014-09-03 06:34:56]

FYI, CQ is re-trying this CL (attempt #1).
The failing builders are:
  android_dbg_tests_recipe on tryserver.chromium.linux
(http://build.chromium.org/p/tryserver.chromium.linux/builders/android_dbg_tes...)

[commit-bot: I haz the power, 2014-09-03 07:44:49]

Committed patchset #1 (id:1) as a4986fbea875aa0bf7fb5aaa24efaef79b564fb3

[commit-bot: I haz the power, 2014-09-10 03:24:20]

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