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

extensions/renderer/script_injection.cc

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "extensions/renderer/script_injection.h"
 
 #include <map>
 
 #include "base/lazy_instance.h"
 #include "base/metrics/histogram.h"
@@ skipping 66 matching lines @@
   }
 
   // We need to set the isolated world origin and CSP even if it's not a new
   // world since these are stored per frame, and we might not have used this
   // isolated world in this frame before.
   frame->setIsolatedWorldSecurityOrigin(
       id, blink::WebSecurityOrigin::create(extension->url()));
   frame->setIsolatedWorldContentSecurityPolicy(
       id,
       blink::WebString::fromUTF8(CSPInfo::GetContentSecurityPolicy(extension)));
+  frame->setIsolatedWorldHumanReadableName(

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

Should it be added to translate_helper.cc as well?

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

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. 

+      id,
+      blink::WebString::fromUTF8(extension->name()));
 
   return id;
 }
 
 }  // namespace
 
 // static
 std::string ScriptInjection::GetExtensionIdForIsolatedWorld(
     int isolated_world_id) {
   IsolatedWorldMap& isolated_worlds = g_isolated_worlds.Get();
@@ skipping 211 matching lines @@
   std::vector<std::string> css_sources =
       injector_->GetCssSources(run_location_);
   for (std::vector<std::string>::const_iterator iter = css_sources.begin();
        iter != css_sources.end();
        ++iter) {
     frame->document().insertStyleSheet(blink::WebString::fromUTF8(*iter));
   }
 }
 
 }  // namespace extensions