diff options
| author | nednguyen <nednguyen@google.com> | 2015-09-10 16:08:43 -0700 |
|---|---|---|
| committer | Commit bot <commit-bot@chromium.org> | 2015-09-10 23:09:19 +0000 |
| commit | 0e66d57071b66863773249452c49384cff83b9ac (patch) | |
| tree | b9b11cf509d8344c1dd38f9f094551eb4520b70e /extensions/common/extension_messages.h | |
| parent | f2aedcc017d906abfd2e9d567cf63c11f4d096e4 (diff) | |
| download | chromium_src-0e66d57071b66863773249452c49384cff83b9ac.zip chromium_src-0e66d57071b66863773249452c49384cff83b9ac.tar.gz chromium_src-0e66d57071b66863773249452c49384cff83b9ac.tar.bz2 | |
Revert of Check the tab ID before delivering an extension message. (patchset #4 id:60001 of https://codereview.chromium.org/1318153002/ )
Reason for revert:
Speculative revert, this may cause compile failure on [precise32, precise64] stable
BUG=530347
Original issue's description:
> Check the tab ID before delivering an extension message.
>
> r335753 changed extension messaging to use RenderFrames, which had the
> side-effect of delivering messages to every RenderFrame in a process,
> which may include other tabs.
>
> This patch fixes that by sending the target tab ID along with the message. A
> more principled fix would have been to track RenderFrames on the browser and
> send to precisely the right ones, but this would need to be part of a more
> comprehensive refactor.
>
> I also fixed up ExtensionApiTest.Connect and re-enabled, which was disabled
> years ago due to flakiness. Hopefully my test JS changes will fix that.
>
> BUG=520303
> R=rdevlin.cronin@chromium.org, dcheng@chromium.org
>
> Committed: https://crrev.com/1344c7dfde448d4b80693f597b02993b7f6bbd5b
> Cr-Commit-Position: refs/heads/master@{#346176}
TBR=dcheng@chromium.org,rdevlin.cronin@chromium.org,kalman@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=520303
Review URL: https://codereview.chromium.org/1339463002
Cr-Commit-Position: refs/heads/master@{#348274}
Diffstat (limited to 'extensions/common/extension_messages.h')
| -rw-r--r-- | extensions/common/extension_messages.h | 7 |
1 files changed, 1 insertions, 6 deletions
diff --git a/extensions/common/extension_messages.h b/extensions/common/extension_messages.h index 5ec63d1..4454934 100644 --- a/extensions/common/extension_messages.h +++ b/extensions/common/extension_messages.h @@ -180,12 +180,7 @@ IPC_STRUCT_BEGIN(ExtensionMsg_ExternalConnectionInfo) // The URL of the frame that initiated the request. IPC_STRUCT_MEMBER(GURL, source_url) - // The ID of the tab that is the target of the request, or -1 if there is no - // target tab. - IPC_STRUCT_MEMBER(int, target_tab_id) - - // The ID of the frame that is the target of the request, or -1 if there is - // no target frame (implying the message is for all frames). + // The ID of the frame that is the target of the request. IPC_STRUCT_MEMBER(int, target_frame_id) // The process ID of the webview that initiated the request. |