Bug 105266 - [WK2] The new npruntime/embed-property-iframe-equality.html fails
Summary: [WK2] The new npruntime/embed-property-iframe-equality.html fails
Status: NEW
Alias: None
Product: WebKit
Classification: Unclassified
Component: Plug-ins (show other bugs)
Version: 420+
Hardware: All All
: P2 Normal
Assignee: Nobody
URL:
Keywords: Qt, QtTriaged
Depends on:
Blocks: 87008 104921
  Show dependency treegraph
 
Reported: 2012-12-18 01:57 PST by Csaba Osztrogonác
Modified: 2012-12-18 09:57 PST (History)
5 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Csaba Osztrogonác 2012-12-18 01:57:01 PST
npruntime/embed-property-iframe-equality.html introduced in 
https://trac.webkit.org/changeset/137964, but fails on all
WK2 platforms. (Apple Mac, GTK, EFL, Qt):

--- /home/webkitbuildbot/slaves/release64bitWebKit2_EC2/buildslave/qt-linux-64-release-webkit2/build/layout-test-results/plugins/npruntime/embed-property-iframe-equality-expected.txt
+++ /home/webkitbuildbot/slaves/release64bitWebKit2_EC2/buildslave/qt-linux-64-release-webkit2/build/layout-test-results/plugins/npruntime/embed-property-iframe-equality-actual.txt
@@ -2,4 +2,4 @@
 
 
 
-Send two references of a JavaScript object to the plugin for identity comparison in C++ PASS
+Send two references of a JavaScript object to the plugin for identity comparison in C++ FAIL
Comment 1 Csaba Osztrogonác 2012-12-18 02:08:35 PST
I added it to wk2/TestExpectations to paint the bots green. 
Please unskip it with the proper fix.
Comment 2 Matthew Dempsky 2012-12-18 08:42:21 PST
I added that test case for a regression I fixed in the V8/NPAPI bindings that seemed generally applicable.  Assuming the embed-property-equality.html test (i.e., the one embed-property-iframe-equality.html is derived from) is passing on the "WK2 platforms", then I suspect there's a similar bug in their NPAPI bindings too.

But I politely ask that you find someone more familiar with WK2 and NPAPI than me to investigate this for what the proper fix is.  The only fix I'm qualified to make here is to figure out how to change the test to be V8-specific, which doesn't seem helpful if it turns out there's actually a bug elsewhere in WebCore that should be fixed.
Comment 3 Alexey Proskuryakov 2012-12-18 09:57:19 PST
We may already have an old bug tracking this issue.