Closed Bug 1267311 Opened 8 years ago Closed 8 years ago

Intermittent test_pointerevent_pointerleave_touch-manual.html | pointerleave event - got 1, expected +0

Categories

(Core :: DOM: Events, defect, P5)

Product:
Core
Component:
DOM: Events
Platform:
Unspecified
Windows
Type:
defect

Tracking

()

Status:
RESOLVED WORKSFORME
Tracking Flags:
Tracking Status
e10s + ---
firefox48 --- affected

People

(Reporter: RyanVM, Assigned: stone)

References

(Blocks 1 open bug)

Details

(Keywords: intermittent-failure)

Attachments

(1 file)

test screenshot
1.06 MB, image/png
Details 
+++ This bug was initially created as a clone of Bug #1267310 +++

All Windows debug e10s so far, from what I'm seeing.

https://treeherder.mozilla.org/logviewer.html#?job_id=170880&repo=ash

07:01:46     INFO -  272 INFO TEST-START | dom/events/test/pointerevents/test_pointerevent_pointerleave_touch-manual.html
07:01:46     INFO -  [Parent 3704] WARNING: no triggering principal available via loadInfo, assuming load is cross-origin: file c:/builds/moz2_slave/ash-w32-d-00000000000000000000/build/src/netwerk/protocol/http/HttpBaseChannel.cpp, line 1405
07:01:46     INFO -  [Parent 3704] WARNING: Found channel with no loadinfo, assuming third-party request: file c:/builds/moz2_slave/ash-w32-d-00000000000000000000/build/src/dom/base/ThirdPartyUtil.cpp, line 235
07:01:46     INFO -  ++DOMWINDOW == 87 (12B38800) [pid = 2656] [serial = 87] [outer = 10617C00]
07:01:46     INFO -  ++DOCSHELL 12B41000 == 13 [pid = 2656] [id = 19]
07:01:46     INFO -  ++DOMWINDOW == 88 (12B41400) [pid = 2656] [serial = 88] [outer = 00000000]
07:01:46     INFO -  ++DOMWINDOW == 89 (12B41C00) [pid = 2656] [serial = 89] [outer = 12B41400]
07:01:46     INFO -  [Parent 3704] WARNING: no triggering principal available via loadInfo, assuming load is cross-origin: file c:/builds/moz2_slave/ash-w32-d-00000000000000000000/build/src/netwerk/protocol/http/HttpBaseChannel.cpp, line 1405
07:01:46     INFO -  [Parent 3704] WARNING: Found channel with no loadinfo, assuming third-party request: file c:/builds/moz2_slave/ash-w32-d-00000000000000000000/build/src/dom/base/ThirdPartyUtil.cpp, line 235
07:01:46     INFO -  --DOMWINDOW == 17 (15D0B800) [pid = 3704] [serial = 4] [outer = 00000000] [url = about:blank]
07:01:46     INFO -  --DOMWINDOW == 16 (1DAF3800) [pid = 3704] [serial = 16] [outer = 00000000] [url = about:blank]
07:01:46     INFO -  --DOMWINDOW == 15 (19C5F400) [pid = 3704] [serial = 14] [outer = 00000000] [url = about:blank]
07:01:46     INFO -  --DOMWINDOW == 14 (19FAC800) [pid = 3704] [serial = 11] [outer = 00000000] [url = about:blank]
07:01:46     INFO -  ++DOMWINDOW == 90 (12202400) [pid = 2656] [serial = 90] [outer = 12B41400]
07:01:46     INFO -  273 INFO TEST-PASS | dom/events/test/pointerevents/test_pointerevent_pointerleave_touch-manual.html | pointerleave event is a PointerEvent event
07:01:46     INFO -  274 INFO TEST-PASS | dom/events/test/pointerevents/test_pointerevent_pointerleave_touch-manual.html | pointerleave.pointerId attribute exists
07:01:46     INFO -  275 INFO TEST-PASS | dom/events/test/pointerevents/test_pointerevent_pointerleave_touch-manual.html | pointerleave.pointerId is readonly
07:01:46     INFO -  276 INFO TEST-PASS | dom/events/test/pointerevents/test_pointerevent_pointerleave_touch-manual.html | pointerleave.pointerId IDL type long (JS type was number)
07:01:46     INFO -  277 INFO TEST-PASS | dom/events/test/pointerevents/test_pointerevent_pointerleave_touch-manual.html | pointerleave.width attribute exists
07:01:46     INFO -  278 INFO TEST-PASS | dom/events/test/pointerevents/test_pointerevent_pointerleave_touch-manual.html | pointerleave.width is readonly
07:01:46     INFO -  279 INFO TEST-PASS | dom/events/test/pointerevents/test_pointerevent_pointerleave_touch-manual.html | pointerleave.width IDL type float (JS type was number)
07:01:46     INFO -  280 INFO TEST-PASS | dom/events/test/pointerevents/test_pointerevent_pointerleave_touch-manual.html | pointerleave.height attribute exists
07:01:46     INFO -  281 INFO TEST-PASS | dom/events/test/pointerevents/test_pointerevent_pointerleave_touch-manual.html | pointerleave.height is readonly
07:01:46     INFO -  282 INFO TEST-PASS | dom/events/test/pointerevents/test_pointerevent_pointerleave_touch-manual.html | pointerleave.height IDL type float (JS type was number)
07:01:46     INFO -  283 INFO TEST-PASS | dom/events/test/pointerevents/test_pointerevent_pointerleave_touch-manual.html | pointerleave.pressure attribute exists
07:01:46     INFO -  284 INFO TEST-PASS | dom/events/test/pointerevents/test_pointerevent_pointerleave_touch-manual.html | pointerleave.pressure is readonly
07:01:46     INFO -  285 INFO TEST-PASS | dom/events/test/pointerevents/test_pointerevent_pointerleave_touch-manual.html | pointerleave.pressure IDL type float (JS type was number)
07:01:46     INFO -  286 INFO TEST-PASS | dom/events/test/pointerevents/test_pointerevent_pointerleave_touch-manual.html | pointerleave.tiltX attribute exists
07:01:46     INFO -  287 INFO TEST-PASS | dom/events/test/pointerevents/test_pointerevent_pointerleave_touch-manual.html | pointerleave.tiltX is readonly
07:01:46     INFO -  288 INFO TEST-PASS | dom/events/test/pointerevents/test_pointerevent_pointerleave_touch-manual.html | pointerleave.tiltX IDL type long (JS type was number)
07:01:46     INFO -  289 INFO TEST-PASS | dom/events/test/pointerevents/test_pointerevent_pointerleave_touch-manual.html | pointerleave.tiltY attribute exists
07:01:46     INFO -  290 INFO TEST-PASS | dom/events/test/pointerevents/test_pointerevent_pointerleave_touch-manual.html | pointerleave.tiltY is readonly
07:01:46     INFO -  291 INFO TEST-PASS | dom/events/test/pointerevents/test_pointerevent_pointerleave_touch-manual.html | pointerleave.tiltY IDL type long (JS type was number)
07:01:46     INFO -  292 INFO TEST-PASS | dom/events/test/pointerevents/test_pointerevent_pointerleave_touch-manual.html | pointerleave.pointerType attribute exists
07:01:46     INFO -  293 INFO TEST-PASS | dom/events/test/pointerevents/test_pointerevent_pointerleave_touch-manual.html | pointerleave.pointerType is readonly
07:01:46     INFO -  294 INFO TEST-PASS | dom/events/test/pointerevents/test_pointerevent_pointerleave_touch-manual.html | pointerleave.pointerType IDL type string (JS type was string)
07:01:46     INFO -  295 INFO TEST-PASS | dom/events/test/pointerevents/test_pointerevent_pointerleave_touch-manual.html | pointerleave.isPrimary attribute exists
07:01:46     INFO -  296 INFO TEST-PASS | dom/events/test/pointerevents/test_pointerevent_pointerleave_touch-manual.html | pointerleave.isPrimary is readonly
07:01:46     INFO -  297 INFO TEST-PASS | dom/events/test/pointerevents/test_pointerevent_pointerleave_touch-manual.html | pointerleave.isPrimary IDL type boolean (JS type was boolean)
07:01:46     INFO -  298 INFO TEST-PASS | dom/events/test/pointerevents/test_pointerevent_pointerleave_touch-manual.html | pointerleave.pressure value is valid
07:01:46     INFO -  299 INFO TEST-PASS | dom/events/test/pointerevents/test_pointerevent_pointerleave_touch-manual.html | pointerleave properties for pointerType = mouse
07:01:46     INFO -  TEST-INFO | started process screenshot
07:01:47     INFO -  TEST-INFO | screenshot: exit 0
07:01:47     INFO -  300 INFO TEST-UNEXPECTED-FAIL | dom/events/test/pointerevents/test_pointerevent_pointerleave_touch-manual.html | pointerleave event - got 1, expected +0
07:01:47     INFO -      SimpleTest.is@SimpleTest/SimpleTest.js:268:5
07:01:47     INFO -      result_function@dom/events/test/pointerevents/mochitest_support_external.js:33:3
07:01:47     INFO -  Tests.prototype.notify_result/<@http://mochi.test:8888/resources/testharness.js:1865:21
07:01:47     INFO -  forEach@http://mochi.test:8888/resources/testharness.js:2519:17
07:01:47     INFO -  Tests.prototype.notify_result@http://mochi.test:8888/resources/testharness.js:1862:9
07:01:47     INFO -  Tests.prototype.result@http://mochi.test:8888/resources/testharness.js:1856:9
07:01:47     INFO -  Test.prototype.done@http://mochi.test:8888/resources/testharness.js:1490:9
07:01:47     INFO -  Test.prototype.step@http://mochi.test:8888/resources/testharness.js:1390:13
07:01:47     INFO -      run/<@dom/events/test/pointerevents/pointerevent_pointerleave_touch-manual.html:29:21
07:01:47     INFO -  EventListener.handleEvent*on_event@http://mochi.test:8888/resources/testharness.js:658:9
07:01:47     INFO -      run@dom/events/test/pointerevents/pointerevent_pointerleave_touch-manual.html:26:17
07:01:47     INFO -      onload@dom/events/test/pointerevents/pointerevent_pointerleave_touch-manual.html:1:1
07:01:47     INFO -  MEMORY STAT | vsize 328MB | vsizeMaxContiguous 459MB | residentFast 165MB | heapAllocated 32MB
07:01:47     INFO -  301 INFO TEST-OK | dom/events/test/pointerevents/test_pointerevent_pointerleave_touch-manual.html | took 1089ms
Flags: needinfo?(alessarik)
Summary: Intermittent test_pointerevent_pointerleave_descendant_over-manual.html | pointerleave shouldn't be received on descendant's pointerover - got 1, expected +0 → Intermittent test_pointerevent_pointerleave_touch-manual.html | pointerleave event - got 1, expected +0
Attached image test screenshot — 

          tracking-e10s:
          --- → +
Priority: -- → P5

    
  
Assignee: nobody → sshih

    
    

    
      
  

    
    

    
      
  

    
    

    
  
I found this issue is caused by an additional mousemove event. Before the test case starts to synthesize touch events, a mouse is dispatched, which generates a pointermove event. Then the pointermove event triggers a pointerleave event and notifies the callback in this test case. The callback expects to receive pointerleave with type=touch. 

The normal sequence is
start test_pointerevent_pointerleave_touch-manual.html
synthesize eTouchStart
synthesize eTouchEnd --> trigger pointerleave, check type == touch then passed

The abnormal sequence is
start test_pointerevent_pointerleave_touch-manual.html
an additional mousemove is dispatched --> trigger pointerleave, check type != touch then failed
synthesize eTouchStart
synthesize eTouchEnd

I may need more time to analyze where the mousemove comes from.

    
    

    
  
Looks like this is related to bug1285128. Test on try server with the patch for bug1285128, there are no intermittent failures happened.

    
    

    
  
(In reply to Ming-Chou Shih [:stone] from comment #5)
> Looks like this is related to bug1285128. Test on try server with the patch
> for bug1285128, there are no intermittent failures happened.

https://treeherder.mozilla.org/#/jobs?repo=try&revision=a7f86b6e0267f188000b9274c9c58451314d436b

    
  
Depends on: 1285713

    
    

    
      
  

    
  
Depends on: 1285128

    
    

    
      
  

    
    

    
  
This bug should be fixed by the patch of bug 1285128. Closed it since [1] shows that it wasn't happened in the past two weeks.

[1] https://brasstacks.mozilla.com/orangefactor/?display=Bug&bugid=1267311&startday=2016-08-01&endday=2016-08-30&tree=all
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → WORKSFORME

    
  
Flags: needinfo?(alessarik)

          You need to log in
          before you can comment on or make changes to this bug.
        






  

    
  







  

    

      
Attachment

      

      
      
      
      
    

    

      

        

          

            
General

            

              Creator: 



            

            
Created: 

            
Updated: 

            
Size: