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

Side by Side Diff: LayoutTests/fast/forms/select/listbox-disabled-no-autoscroll.html

Issue 14996003: Disabled select element should not fire onchange event. (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: Added test and moved the checks to RenderListBox Created 7 years, 7 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 unified diff | Download patch
« no previous file with comments | « no previous file | LayoutTests/fast/forms/select/listbox-disabled-no-autoscroll-expected.html » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
(Empty)
1 <!DOCTYPE html>
2 <html>
3 <head>
4 <script src="../resources/common.js"></script>
5 <style>
6 select {background-color: red; }
7 </style>
8 </head>
9 <body>
10 <select disabled id="test" size=4">
11 <option selected>foo1</option>
12 <option>foo2</option>
13 <option>foo3</option>
14 <option>foo4</option>
15 <option>foo5</option>
16 </select>
17
18 <div id="result">Success if selection did not change.</div>
19 <div id="selection"></div>
20 <script>
21 if (window.testRunner) {
22 testRunner.waitUntilDone();
23 }
24
25 if (window.eventSender) {
26 eventSender.dragMode = false;
27 mouseMoveToIndexInListbox(0, 'test');
28 eventSender.mouseDown();
29 mouseMoveToIndexInListbox(1, 'test');
30 mouseMoveToIndexInListbox(2, 'test');
31 setTimeout(function() {
32 mouseMoveToIndexInListbox(3, 'test');
33 testRunner.notifyDone();
34 }, 100);
35 }
36 </script>
37 </html>
38
OLDNEW
« no previous file with comments | « no previous file | LayoutTests/fast/forms/select/listbox-disabled-no-autoscroll-expected.html » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698