Comment #1 on issue 27164 by [email protected]: plugins open layout tests are failing. http://code.google.com/p/chromium/issues/detail?id=27164
The following revision refers to this bug:
http://src.chromium.org/viewvc/chrome?view=rev&revision=31991
------------------------------------------------------------------------
r31991 | [email protected] | 2009-11-13 19:23:32 -0800 (Fri, 13 Nov 2009)
| 17 lines
Changed paths:
A
http://src.chromium.org/viewvc/chrome/trunk/src/webkit/data/layout_tests/platform/chromium-linux/LayoutTests/plugins/document-open-expected.txt
A
http://src.chromium.org/viewvc/chrome/trunk/src/webkit/data/layout_tests/platform/chromium-linux/LayoutTests/plugins/window-open-expected.txt
A
http://src.chromium.org/viewvc/chrome/trunk/src/webkit/data/layout_tests/platform/chromium-mac/LayoutTests/plugins/document-open-expected.txt
A
http://src.chromium.org/viewvc/chrome/trunk/src/webkit/data/layout_tests/platform/chromium-mac/LayoutTests/plugins/window-open-expected.txt
A
http://src.chromium.org/viewvc/chrome/trunk/src/webkit/data/layout_tests/platform/chromium-win/LayoutTests/plugins/document-open-expected.txt
A
http://src.chromium.org/viewvc/chrome/trunk/src/webkit/data/layout_tests/platform/chromium-win/LayoutTests/plugins/window-open-expected.txt
M
http://src.chromium.org/viewvc/chrome/trunk/src/webkit/glue/plugins/plugin_stream_url.cc?r1=31991&r2=31990
M
http://src.chromium.org/viewvc/chrome/trunk/src/webkit/glue/plugins/plugin_string_stream.cc?r1=31991&r2=31990
M
http://src.chromium.org/viewvc/chrome/trunk/src/webkit/tools/npapi_layout_test_plugin/PluginObject.cpp?r1=31991&r2=31990
M
http://src.chromium.org/viewvc/chrome/trunk/src/webkit/tools/npapi_layout_test_plugin/PluginObject.h?r1=31991&r2=31990
M
http://src.chromium.org/viewvc/chrome/trunk/src/webkit/tools/npapi_layout_test_plugin/main.cpp?r1=31991&r2=31990
This CL fixes the document-open.html and window-open.html plugin layout
tests. These were landed in webkit
as fixes for bug https://bugs.webkit.org/show_bug.cgi?id=31067
The layout test plugin changes are on similar lines as the version upstream.
We need to rebaseline the expectations as the line numbers differ in test
shell and dumprendertree.
The document-open.html crashes in test_shell as this test causes the
existing document to be taken down
which takes down the plugin instance and the associated plugin stream. This
works in the webkit plugin
implementation as they grab a reference on the plugin stream before calling
into the plugin to protect
from the stream being destroyed.
Our fix is on similar lines.
Bug=27164
Review URL: http://codereview.chromium.org/393017
------------------------------------------------------------------------
--
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
