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

chrome/browser/guest_view/web_view/web_view_guest.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 "chrome/browser/guest_view/web_view/web_view_guest.h"
 
 #include "base/message_loop/message_loop.h"
 #include "base/strings/stringprintf.h"
 #include "base/strings/utf_string_conversions.h"
 #include "chrome/browser/chrome_notification_types.h"
@@ skipping 1099 matching lines @@
   }
   DCHECK(embedder_extension_id() == partition_domain);
 
   ExtensionRendererState::WebViewInfo webview_info;
   webview_info.embedder_process_id = embedder_render_process_id();
   webview_info.instance_id = view_instance_id();
   webview_info.partition_id =  partition_id;
   webview_info.embedder_extension_id = embedder_extension_id();
 
   content::BrowserThread::PostTask(
-      content::BrowserThread::IO, FROM_HERE,
-      base::Bind(
-          &ExtensionRendererState::AddWebView,
-          base::Unretained(ExtensionRendererState::GetInstance()),
-          guest_web_contents()->GetRenderProcessHost()->GetID(),
-          guest_web_contents()->GetRoutingID(),
-          webview_info));
+      content::BrowserThread::IO,
+      FROM_HERE,
+      base::Bind(&ExtensionRendererState::AddWebView,
+                 base::Unretained(ExtensionRendererState::GetInstance()),
+                 guest_web_contents()->GetRenderProcessHost()->GetID(),
+                 guest_web_contents()->GetRoutingID(),
+                 partition_id,

[Fady Samuel, 2014/06/16 22:02:32]

This extra parameter is unnecessary. You can grab the partition ID from
webview_info.partition_id.

While you're at it, for consistency, please rename the variable above from
webview_info to web_view_info.

[Xi Han, 2014/06/17 13:42:30]

Done.

+                 webview_info));
 }
 
 // static
 void WebViewGuest::RemoveWebViewFromExtensionRendererState(
     WebContents* web_contents) {
   content::BrowserThread::PostTask(
       content::BrowserThread::IO, FROM_HERE,
       base::Bind(
           &ExtensionRendererState::RemoveWebView,
           base::Unretained(ExtensionRendererState::GetInstance()),
@@ skipping 454 matching lines @@
     bool allow,
     const std::string& user_input) {
   WebViewGuest* guest =
       WebViewGuest::From(embedder_render_process_id(), new_window_instance_id);
   if (!guest)
     return;
 
   if (!allow)
     guest->Destroy();
 }