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

Unified Diff: third_party/WebKit/LayoutTests/plugins/overlay-scrollbar-mouse-capture.html

Issue 2784313002: Change hittest node to be scrollbar's parent when hittest include a scrollbar (Closed)
Patch Set: mustaq comments addressed Created 3 years, 8 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 | « no previous file | third_party/WebKit/LayoutTests/plugins/overlay-scrollbar-mouse-capture-expected.txt » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: third_party/WebKit/LayoutTests/plugins/overlay-scrollbar-mouse-capture.html
diff --git a/third_party/WebKit/LayoutTests/plugins/overlay-scrollbar-mouse-capture.html b/third_party/WebKit/LayoutTests/plugins/overlay-scrollbar-mouse-capture.html
index 0c7081142048e5635007293cf099f9d5628ff7c2..6b6abc1460a6d9021c612b56c20c61920e49402a 100644
--- a/third_party/WebKit/LayoutTests/plugins/overlay-scrollbar-mouse-capture.html
+++ b/third_party/WebKit/LayoutTests/plugins/overlay-scrollbar-mouse-capture.html
@@ -38,11 +38,9 @@
// element under the mouse.
eventSender.mouseUp();
- // TODO(bokan): This is wrong, if the scrollbar captures the input the
- // plugin shouldn't be receiving events. crbug.com/636436.
- shouldBe('eventHistory.length', '2');
- shouldBe('eventHistory[0]', '"plugin.mousedown"');
- shouldBe('eventHistory[1]', '"plugin.mouseup"');
+ // The scrollbar captures the input and the plugin should not receive
+ // events.
+ shouldBe('eventHistory.length', '0');
requestAnimationFrame(function() {
shouldBeTrue('window.scrollY > 0');
finishJSTest();
« no previous file with comments | « no previous file | third_party/WebKit/LayoutTests/plugins/overlay-scrollbar-mouse-capture-expected.txt » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698