Closed Bug 1411467 Opened 6 years ago Closed 6 years ago

Keeping PointerEvent enabled in release build

Categories

(Core :: DOM: Events, enhancement, P2)

Product:
Core
Component:
DOM: Events
Type:
enhancement

Tracking

()

Status:
RESOLVED FIXED
Milestone:
mozilla59
Tracking Flags:
Tracking Status
firefox57 --- unaffected
firefox58 --- wontfix
firefox59 --- fixed

People

(Reporter: stone, Assigned: stone)

Details

(Keywords: dev-doc-complete)

Attachments

(1 file, 1 obsolete file)

Keeping PointerEvent enabled in release build
2.18 KB, patch
smaug
: review+
Details | Diff | Splinter Review 
      No description provided.
Assignee: nobody → sshih
status-firefox57: --- → unaffected
Priority: -- → P2
Summary: Keeping PointerEvent enabled → Keeping PointerEvent enabled in early beta
Stone, the merge day is Nov. 13, let's have this in by then for smooth train riding and get to more users.
Attachment #8926765 - Flags: review?(bugs)
Attachment #8926765 - Flags: review?(bugs) → review+
Pushed by sshih@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/2ed0ff57330d
Keeping PointerEvent enabled in early beta. r=smaug.
https://hg.mozilla.org/mozilla-central/rev/2ed0ff57330d
Status: NEW → RESOLVED
Closed: 6 years ago
status-firefox58: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla58
This patch didn't take test_interfaces into account at all and unsurprisingly, it's burning Beta badly. Backing out because we can't have burning Beta builds with b3 gtb scheduled for Monday.
https://treeherder.mozilla.org/logviewer.html#?job_id=143859638&repo=mozilla-beta

https://hg.mozilla.org/mozilla-central/rev/933f9cd9b3b9030399a11c19cb4e5117b29e2772
Status: RESOLVED → REOPENED
status-firefox58: fixed → affected
Flags: needinfo?(sshih)
Resolution: FIXED → ---
Target Milestone: mozilla58 → ---
Can't only enable some interfaces in test_interfaces.js for early beta. So we plan to enable it in release build and ride the next train (FF59).
Flags: needinfo?(sshih)
Summary: Keeping PointerEvent enabled in early beta → Keeping PointerEvent enabled in release build

        Attachment #8926765 -
        Attachment is obsolete: true

    
  

        Attachment #8932040 -
        Flags: review?(bugs)

    
    

    
  
Comment on attachment 8932040 [details] [diff] [review]
Keeping PointerEvent enabled in release build

Hmm, why we can't enable on Android too? At least file a bug to enable there too.

        Attachment #8932040 -
        Flags: review?(bugs) → review+

    
    

    
  
Created bug 1426786 for pointer event on android. We still have some works (e.g. fill the pointer event related attributes, fix failed test cases) before enabling it on android.

    
    

    
  
Pushed by sshih@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/0d6b0f5ffc8f
Keeping PointerEvent enabled in release build. r=smaug.

    
    

    
  
https://hg.mozilla.org/mozilla-central/rev/0d6b0f5ffc8f
Status: REOPENED → RESOLVED
Closed: 6 years ago → 6 years ago

          status-firefox59:
          --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla59

    
    

    
  
Ride the train on 59, won't fix for 58.

          status-firefox58:
          affected → wontfix

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