Fix the bug that webview guest cannot load script of a SharedWorker.

chrome/browser/extensions/extension_renderer_state.cc

 // Copyright (c) 2013 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 "chrome/browser/extensions/extension_renderer_state.h"
 
 #include "base/bind.h"
 #include "base/bind_helpers.h"
 #include "chrome/browser/chrome_notification_types.h"
 #include "chrome/browser/sessions/session_tab_helper.h"
@@ skipping 205 matching lines @@
   if (iter != map_.end()) {
     *tab_id = iter->second.first;
     *window_id = iter->second.second;
     return true;
   }
   return false;
 }
 
 bool ExtensionRendererState::IsWebViewRenderer(int render_process_id) {
   DCHECK_CURRENTLY_ON(BrowserThread::IO);
-  for (WebViewInfoMap::iterator i = webview_info_map_.begin();
-       i != webview_info_map_.end(); ++i) {
-    if (i->first.first == render_process_id)
-      return true;
-  }
-  return false;
+  return webview_partition_id_map.find(render_process_id) !=
+         webview_partition_id_map.end();
 }
 
 void ExtensionRendererState::AddWebView(int guest_process_id,
                                         int guest_routing_id,
                                         const WebViewInfo& webview_info) {
   DCHECK_CURRENTLY_ON(BrowserThread::IO);
   RenderId render_id(guest_process_id, guest_routing_id);
   webview_info_map_[render_id] = webview_info;
 }
 
 void ExtensionRendererState::RemoveWebView(int guest_process_id,
                                            int guest_routing_id) {
   DCHECK_CURRENTLY_ON(BrowserThread::IO);
   RenderId render_id(guest_process_id, guest_routing_id);
   webview_info_map_.erase(render_id);
 }
 
 bool ExtensionRendererState::GetWebViewInfo(int guest_process_id,
                                             int guest_routing_id,
                                             WebViewInfo* webview_info) {
   DCHECK_CURRENTLY_ON(BrowserThread::IO);
   RenderId render_id(guest_process_id, guest_routing_id);
   WebViewInfoMap::iterator iter = webview_info_map_.find(render_id);
   if (iter != webview_info_map_.end()) {
     *webview_info = iter->second;
     return true;
   }
   return false;
 }
+
+void ExtensionRendererState::AddWebViewPartitionID(

[Fady Samuel, 2014/06/16 21:01:56]

Remove this method.

[Xi Han, 2014/06/16 21:53:55]

Done.

+    int guest_process_id,
+    const std::string& webview_partition_id) {
+  DCHECK_CURRENTLY_ON(BrowserThread::IO);
+  webview_partition_id_map[guest_process_id] = webview_partition_id;

[Fady Samuel, 2014/06/16 21:01:56]

Move this to AddWebView.

You probably also need a counter here. The counter needs to keep track of the
number of webviews are in a process. You don't always want to remove an item
from this map.

[Xi Han, 2014/06/16 21:53:55]

Done.

+}
+
+void ExtensionRendererState::RemoveWebViewPartitionID(int guest_process_id) {
+  DCHECK_CURRENTLY_ON(BrowserThread::IO);
+  webview_partition_id_map.erase(guest_process_id);

[Fady Samuel, 2014/06/16 21:01:56]

Move this to RemoveWebView.

[Xi Han, 2014/06/16 21:53:55]

Done.

+}
+
+bool ExtensionRendererState::GetWebViewPartitionID(int guest_process_id,
+                                                   std::string* partition_id) {
+  DCHECK_CURRENTLY_ON(BrowserThread::IO);
+  WebViewPartitionIDMap::iterator iter =
+      webview_partition_id_map.find(guest_process_id);
+  if (iter != webview_partition_id_map.end()) {
+    *partition_id = iter->second;
+    return true;
+  }
+  return false;
+}