Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(712)

Unified Diff: LayoutTests/compositing/overflow/resources/update-widget-positions-on-nested-frames-and-scrollers-inner-frame.html

Issue 736883002: Implement <select> Popup Menu using PagePopup (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Fixed tests for mac Created 5 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « LayoutTests/TestExpectations ('k') | LayoutTests/fast/forms/resources/picker-common.js » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: LayoutTests/compositing/overflow/resources/update-widget-positions-on-nested-frames-and-scrollers-inner-frame.html
diff --git a/LayoutTests/compositing/overflow/resources/update-widget-positions-on-nested-frames-and-scrollers-inner-frame.html b/LayoutTests/compositing/overflow/resources/update-widget-positions-on-nested-frames-and-scrollers-inner-frame.html
index 5487b36beb8768373f213f2cf9b4756d81bccc48..297a93ae33871d48383dbcabe9043c30aba30805 100644
--- a/LayoutTests/compositing/overflow/resources/update-widget-positions-on-nested-frames-and-scrollers-inner-frame.html
+++ b/LayoutTests/compositing/overflow/resources/update-widget-positions-on-nested-frames-and-scrollers-inner-frame.html
@@ -1,6 +1,7 @@
<!DOCTYPE html>
<html>
<head>
+ <script src="../../../fast/forms/resources/picker-common.js"></script>
<style>
select {
display: block;
@@ -11,7 +12,7 @@
// We need to 'click' the select element by sending an actual event
// through internals, since the regular javascript click() method
// won't trigger the right events for this test.
- function clickSelect() {
+ function clickSelect(callback) {
if (!window.eventSender)
return;
@@ -19,18 +20,18 @@
// FIXME: it would be really nice to use getBoundingClientRect() and not
// hard-code pixel coordinates here, but since it's in an iframe, the
// coordinates are translated anyway.
- eventSender.mouseMoveTo(50, 120);
- eventSender.mouseDown();
- eventSender.mouseUp();
+ clickToOpenPicker(50, 120, callback, function () {
+ setTimeout(callback, 0);
+ });
}
// We'll get a message from the outer frame when it's done scrolling,
// so we know it's time to click on a select element.
window.onmessage = function() {
- clickSelect();
-
- if (window.testRunner)
- window.setTimeout(function() { window.testRunner.notifyDone(); }, 0);
+ clickSelect(function () {
+ if (window.testRunner)
+ window.testRunner.notifyDone();
+ });
};
// This function just saves us from hard-coding 30 select elements
« no previous file with comments | « LayoutTests/TestExpectations ('k') | LayoutTests/fast/forms/resources/picker-common.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698