Cross-process postMessage

chrome/renderer/chrome_content_renderer_client.cc

 // Copyright (c) 2011 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/renderer/chrome_content_renderer_client.h"
 
 #include <string>
 
 #include "base/command_line.h"
 #include "base/logging.h"
@@ skipping 715 matching lines @@
     // In all other cases, we want to compare against the top frame's URL (as
     // opposed to the opener frame's), since that's what determines the type of
     // process.  This allows iframes outside an app to open a popup in the app.
     old_url = frame->top()->opener()->top()->document().url();
   }
 
   // Determine if the old URL is an extension (excluding bookmark apps).
   const Extension* old_url_extension = GetNonBookmarkAppExtension(extensions,
                                                                   old_url);
 
-  // TODO(creis): Temporary workaround for crbug.com/59285: Only return true if
-  // we would enter an extension app's extent from a non-app, or if we leave an
-  // extension with no web extent.  We avoid swapping processes to exit a hosted
-  // app for now, since we do not yet support postMessage calls from outside the
-  // app back into it (e.g., as in Facebook OAuth 2.0).
-  bool old_url_is_hosted_app = old_url_extension &&
-      !old_url_extension->web_extent().is_empty();
-  if (old_url_is_hosted_app)
-    return false;
+  // TODO(supersat): Make sure this change is correct

[Charlie Reis, 2011/12/01 23:13:02]

Yes, this is what I intended.

[supersat, 2011/12/09 23:08:20]

Done.

+  return old_url_extension != new_url_extension;
 
-  return old_url_extension != new_url_extension;
 }
 
 void ChromeContentRendererClient::OnPurgeMemory() {
   DVLOG(1) << "Resetting spellcheck in renderer client";
   RenderThread* thread = RenderThread::Get();
   if (spellcheck_.get())
     thread->RemoveObserver(spellcheck_.get());
   SpellCheck* new_spellcheck = new SpellCheck();
   if (spellcheck_provider_)
     spellcheck_provider_->SetSpellCheck(new_spellcheck);
@@ skipping 22 matching lines @@
 bool ChromeContentRendererClient::IsOtherExtensionWithWebRequestInstalled() {
   return extension_dispatcher_->IsOtherExtensionWithWebRequestInstalled();
 }
 
 void ChromeContentRendererClient::RegisterPPAPIInterfaceFactories(
     webkit::ppapi::PpapiInterfaceFactoryManager* factory_manager) {
   factory_manager->RegisterFactory(ChromePPAPIInterfaceFactory);
 }
 
 }  // namespace chrome