Title: [144070] trunk/Source/WebKit2
- Revision
- 144070
- Author
- [email protected]
- Date
- 2013-02-26 10:29:27 -0800 (Tue, 26 Feb 2013)
Log Message
[WK2][GTK] REGRESSION (r143463): unit test failure in WebKit2APITests/TestContextMenu
https://bugs.webkit.org/show_bug.cgi?id=110869
Patch by Manuel Rego Casasnovas <[email protected]> on 2013-02-26
Reviewed by Martin Robinson.
Test was failing because of new media controls needs more space to be
painted than just 10x10 pixels.
* UIProcess/API/gtk/tests/TestContextMenu.cpp:
(testContextMenuDefaultMenu): Modified test in order to give more space
to the media controls.
Modified Paths
Diff
Modified: trunk/Source/WebKit2/ChangeLog (144069 => 144070)
--- trunk/Source/WebKit2/ChangeLog 2013-02-26 18:04:38 UTC (rev 144069)
+++ trunk/Source/WebKit2/ChangeLog 2013-02-26 18:29:27 UTC (rev 144070)
@@ -1,3 +1,17 @@
+2013-02-26 Manuel Rego Casasnovas <[email protected]>
+
+ [WK2][GTK] REGRESSION (r143463): unit test failure in WebKit2APITests/TestContextMenu
+ https://bugs.webkit.org/show_bug.cgi?id=110869
+
+ Reviewed by Martin Robinson.
+
+ Test was failing because of new media controls needs more space to be
+ painted than just 10x10 pixels.
+
+ * UIProcess/API/gtk/tests/TestContextMenu.cpp:
+ (testContextMenuDefaultMenu): Modified test in order to give more space
+ to the media controls.
+
2013-02-26 Dean Jackson <[email protected]>
Draw intermediate snapshots if possible
Modified: trunk/Source/WebKit2/UIProcess/API/gtk/tests/TestContextMenu.cpp (144069 => 144070)
--- trunk/Source/WebKit2/UIProcess/API/gtk/tests/TestContextMenu.cpp 2013-02-26 18:04:38 UTC (rev 144069)
+++ trunk/Source/WebKit2/UIProcess/API/gtk/tests/TestContextMenu.cpp 2013-02-26 18:29:27 UTC (rev 144070)
@@ -354,8 +354,8 @@
" <a style='position:absolute; left:1; top:1' href='' title='WebKitGTK+ Title'>WebKitGTK+ Website</a>"
" <img style='position:absolute; left:1; top:10' src='' width=5 height=5></img>"
" <a style='position:absolute; left:1; top:20' href='' title='WebKitGTK+ Logo'><img src='' width=5 height=5></img></a>"
- " <video style='position:absolute; left:1; top:30' width=10 height=10 controls='controls'><source src='' type='video/ogg' /></video>"
- " <input style='position:absolute; left:1; top:50' size='10'></input>"
+ " <input style='position:absolute; left:1; top:30' size='10'></input>"
+ " <video style='position:absolute; left:1; top:50' width='300' height='300' controls='controls'><source src='' type='video/ogg' /></video>"
"</body></html>";
test->loadHtml(linksHTML, "file:///");
test->waitUntilLoadFinished();
@@ -382,11 +382,11 @@
// Context menu for image video.
test->m_expectedMenuType = ContextMenuDefaultTest::Video;
- test->showContextMenuAtPositionAndWaitUntilFinished(1, 30);
+ test->showContextMenuAtPositionAndWaitUntilFinished(1, 50);
// Context menu for editable.
test->m_expectedMenuType = ContextMenuDefaultTest::Editable;
- test->showContextMenuAtPositionAndWaitUntilFinished(5, 55);
+ test->showContextMenuAtPositionAndWaitUntilFinished(5, 35);
}
class ContextMenuCustomTest: public ContextMenuTest {
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes
