Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(35)

Unified Diff: chrome/renderer/chrome_content_renderer_client.cc

Issue 8760024: Cross-process postMessage (Closed) Base URL: http://git.chromium.org/chromium/src.git@master
Patch Set: New patch, still not quite done Created 9 years ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/renderer/chrome_content_renderer_client.cc
diff --git a/chrome/renderer/chrome_content_renderer_client.cc b/chrome/renderer/chrome_content_renderer_client.cc
index a3f8bf224d80967f7dec970f5249304abafe73ac..9455bacef73dea6b12559a68b3241c80c9412706 100644
--- a/chrome/renderer/chrome_content_renderer_client.cc
+++ b/chrome/renderer/chrome_content_renderer_client.cc
@@ -779,17 +779,8 @@ bool ChromeContentRendererClient::CrossesExtensionExtents(
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;
-
return old_url_extension != new_url_extension;
+
}
void ChromeContentRendererClient::OnPurgeMemory() {

Powered by Google App Engine
This is Rietveld 408576698