Chromium Code Reviews Chromium Code Reviews
Issue 2270833002:
Fix the crash of issue 2209773002 (Closed)
Base URL: https://chromium.googlesource.com/chromium/src.git@master| OLD | NEW |
|---|---|
| (Empty) | |
| 1 <!DOCTYPE html> | |
| 2 <script src='../../../resources/testharness.js'></script> | |
| 3 <script src='../../../resources/testharnessreport.js'></script> | |
| 4 | |
| 5 <style type='text/css'> | |
| 6 *:before{-webkit-appearance:slider-vertical;} | |
| 7 </style> | |
| 8 | |
| 9 <select multiple='multiple'> | |
| 10 <optgroup> | |
| 11 <option>c</option> | |
| 12 </optgroup> | |
| 13 </select> | |
| 14 | |
| 15 <input type='range' id='slider' step='any' min = '0' max = '100' oninput='destro yLayout()'></input> | |
| 16 | |
| 17 <script> | |
| 18 var isMac = navigator.platform.indexOf('Mac') == 0; | |
| 19 function destroyLayout() { | |
| 20 var slider = document.getElementById('slider'); | |
| 21 slider.style = "display:none"; | |
| 22 } | |
| 23 | |
| 24 function forceLayoutUpdate() { | |
| 25 document.body.offsetTop; | |
| 26 } | |
| 27 | |
| 28 forceLayoutUpdate(); | |
| 29 | |
| 30 var testSliderNoCrash = async_test('Does not crash when <input type="range"> has no layoutObject and when a non-element node has "slider-vertical" appearance.') ; | |
| 31 testSliderNoCrash.step(function () { | |
| 32 if (isMac) { | |
|
tkent
2016/08/25 00:43:56
Why do you exclude Mac?
| |
| 33 return; | |
| 34 } | |
| 35 var slider = document.getElementById('slider'); | |
| 36 const touchSourceType = 1; | |
| 37 var w = slider.clientWidth; | |
|
tkent
2016/08/25 00:43:56
Wrong indentation.
| |
| 38 var h = slider.clientHeight; | |
| 39 var x = slider.offsetLeft + w / 2; | |
| 40 var y = slider.offsetTop + h / 2; | |
| 41 chrome.gpuBenchmarking.smoothDrag(x, y, x + w, y, function() {testSliderNoCr ash.done()}, touchSourceType); | |
| 42 }); | |
| 43 </script> | |
| OLD | NEW |
