Comment #5 on issue 21841 by bugdro...@chromium.org: LayoutTest regression
- npp plugin test failing
http://code.google.com/p/chromium/issues/detail?id=21841
The following revision refers to this bug:
http://src.chromium.org/viewvc/chrome?view=rev&revision=26546
------------------------------------------------------------------------
r26546 | da...@chromium.org | 2009-09-17 20:53:05 -0700 (Thu, 17 Sep 2009)
| 13 lines
Changed paths:
M
http://src.chromium.org/viewvc/chrome/trunk/src/webkit/glue/webframeloaderclient_impl.cc?r1=26546&r2=26545
M
http://src.chromium.org/viewvc/chrome/trunk/src/webkit/tools/layout_tests/test_expectations.txt?r1=26546&r2=26545
M
http://src.chromium.org/viewvc/chrome/trunk/src/webkit/tools/npapi_layout_test_plugin/main.cpp?r1=26546&r2=26545
Fix plugin crash caused by removal of EMBED element during NPP_New.
We get into trouble if FrameLoaderClient::createPlugin returns non-
null if the EMBED element was removed from the DOM during NPP_New.
This CL makes changes to main.cpp to support the layout test, which
corresponds to http://trac.webkit.org/changeset/48361.
R=jam
BUG=21841
TEST=covered by layout test
Review URL: http://codereview.chromium.org/211008
------------------------------------------------------------------------
--
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
-~----------~----~----~----~------~----~------~--~---
