Comment #6 on issue 10036 by [email protected]: Add WebPlugin to the
WebKit API
http://code.google.com/p/chromium/issues/detail?id=10036
The following revision refers to this bug:
http://src.chromium.org/viewvc/chrome?view=rev&revision=24696
------------------------------------------------------------------------
r24696 | [email protected] | 2009-08-27 16:23:56 -0700 (Thu, 27 Aug 2009)
| 12 lines
Changed paths:
M
http://src.chromium.org/viewvc/chrome/trunk/src/webkit/api/src/WebPluginContainerImpl.cpp?r1=24696&r2=24695
A
http://src.chromium.org/viewvc/chrome/trunk/src/webkit/data/layout_tests/chrome/plugins/get-url-with-iframe-target-no-crash-expected.txt
A
http://src.chromium.org/viewvc/chrome/trunk/src/webkit/data/layout_tests/chrome/plugins/get-url-with-iframe-target-no-crash.html
Fix a crash that the reliability bots caught.
This was a silly oops. I had intended to call clearPluginContainer on each
WebPluginLoadObserver held in WebPluginContainerImpl's m_pluginLoadObservers
array when the WebPluginContainerImpl is destroyed, but I never hooked that
up.
Here it is.
R=jam
BUG=10036
TEST=layout test: chrome/plugins/get-url-with-iframe-target-no-crash.html
Review URL: http://codereview.chromium.org/182003
------------------------------------------------------------------------
--
You received this message because you are listed in the owner
or CC fields of this issue, or because you starred this issue.
You may adjust your issue notification preferences at:
http://code.google.com/hosting/settings
--~--~---------~--~----~------------~-------~--~----~
Automated mail from issue updates at http://crbug.com/
Subscription options: http://groups.google.com/group/chromium-bugs
-~----------~----~----~----~------~----~------~--~---
