Chromium Code Reviews| 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 d238d28761ca81c6d7af2a45a7efb90e52383361..2c55ae6bef1434182088c26b0530d6d9a2667e9e 100644 |
| --- a/chrome/renderer/chrome_content_renderer_client.cc |
| +++ b/chrome/renderer/chrome_content_renderer_client.cc |
| @@ -733,17 +733,9 @@ 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; |
| - |
| + // 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; |
| + |
| } |
| void ChromeContentRendererClient::OnPurgeMemory() { |