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

Side by Side Diff: LayoutTests/fast/events/touch/touch-action-double-remove.html

Issue 647503002: Only unregister touch action handler conditionally (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Reduced test case. Created 6 years, 2 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 | Annotate | Revision Log
« no previous file with comments | « no previous file | LayoutTests/fast/events/touch/touch-action-double-remove-expected.txt » ('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 <style>
3 br {
4 touch-action: none;
5 }
6 </style>
7 <!--
8 This is a regression test against a double unregistration of a touch action
9 event handler on a <br> element. When the document finishes loading and is
10 attached to the view, all of its RenderObjects are destroyed and recreated. If
11 a RenderObject had a touch action associated with it, it will need to remove
12 the fake event handler it registered for it in the EventHandlerRegistry.
13 However, when the document gets detached, all of the event handlers within it
14 are automatically unregistered. When the RenderObject tries to unregister its
15 own handler after the fact, it is no longer there and we hit an assertion
16 failure.
17
18 Note that this test doesn't use js-test.js because it also inadvertedly
19 triggers the same bug by appending <br> elements to the document.
20
21 The test passes if it doesn't trigger an assertion failure in a debug build.
22 -->
23 <script>
24 if (window.testRunner)
25 testRunner.dumpAsText();
26 </script>
27 PASS<br>
OLDNEW
« no previous file with comments | « no previous file | LayoutTests/fast/events/touch/touch-action-double-remove-expected.txt » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698