diff options
| author | rdevlin.cronin <rdevlin.cronin@chromium.org> | 2015-07-10 09:03:17 -0700 |
|---|---|---|
| committer | Commit bot <commit-bot@chromium.org> | 2015-07-10 16:03:46 +0000 |
| commit | f01734ae15fc5c2a39181b25999683d336109dbf (patch) | |
| tree | 9304f0d02aed195af8b540d7946ea885dc9a5acd /extensions/common/extension_messages.h | |
| parent | ac899ca82a52961b7c3f162b1014a3c31341748c (diff) | |
| download | chromium_src-f01734ae15fc5c2a39181b25999683d336109dbf.zip chromium_src-f01734ae15fc5c2a39181b25999683d336109dbf.tar.gz chromium_src-f01734ae15fc5c2a39181b25999683d336109dbf.tar.bz2 | |
[Extensions] Fix improper extension tab ownership bug
An extension can "own" a tab if it owns the main frame, and
we block content scripts in this case. But there was a race
that meant the tab wasn't being updated in time for document_start.
Instead of informing the tab of its owner across IPC, use the
security origin of the top frame, since this can now be
accessed from remote frames as well.
BUG=507461
Review URL: https://codereview.chromium.org/1225413002
Cr-Commit-Position: refs/heads/master@{#338300}
Diffstat (limited to 'extensions/common/extension_messages.h')
| -rw-r--r-- | extensions/common/extension_messages.h | 5 |
1 files changed, 0 insertions, 5 deletions
diff --git a/extensions/common/extension_messages.h b/extensions/common/extension_messages.h index f5f1a39..8ae6d19 100644 --- a/extensions/common/extension_messages.h +++ b/extensions/common/extension_messages.h @@ -427,11 +427,6 @@ IPC_MESSAGE_CONTROL2(ExtensionMsg_SetSystemFont, IPC_MESSAGE_CONTROL1(ExtensionMsg_ActivateExtension, std::string /* extension_id */) -// Notifies the renderer that the current tab is an extension page (we limit -// what other extensions can do on these pages). -IPC_MESSAGE_ROUTED1(ExtensionMsg_SetMainFrameExtensionOwner, - std::string /* extension_id */) - // Notifies the renderer that extensions were loaded in the browser. IPC_MESSAGE_CONTROL1(ExtensionMsg_Loaded, std::vector<ExtensionMsg_Loaded_Params>) |