Bug 86535 - [Chromium] Reset ScriptInstance in HTMLPlugInElement on WebPluginContainerImpl::setPlugin
Summary: [Chromium] Reset ScriptInstance in HTMLPlugInElement on WebPluginContainerImp...
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: New Bugs (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Fady Samuel
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-05-15 15:06 PDT by Fady Samuel
Modified: 2012-05-15 18:59 PDT (History)
2 users (show)

See Also:


Attachments
Patch (4.25 KB, patch)
2012-05-15 15:09 PDT, Fady Samuel
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Fady Samuel 2012-05-15 15:06:26 PDT
[Chromium] Reset ScriptInstance in HTMLPlugInElement on WebPluginContainerImpl::setPlugin
Comment 1 Fady Samuel 2012-05-15 15:09:11 PDT
Created attachment 142071 [details]
Patch
Comment 2 Fady Samuel 2012-05-15 15:18:43 PDT
We would like to be able to postMessage to the browser plugin. However, as we do cross-process navigation, we swap out WebPluginImpls and a stale, invalid ScriptInstance object continues being used in HTMLPlugInElement and so the postMessages never arrive after cross-process navigation. This patch fixes this issue.
Comment 3 WebKit Review Bot 2012-05-15 18:59:17 PDT
Comment on attachment 142071 [details]
Patch

Clearing flags on attachment: 142071

Committed r117203: <http://trac.webkit.org/changeset/117203>
Comment 4 WebKit Review Bot 2012-05-15 18:59:21 PDT
All reviewed patches have been landed.  Closing bug.