Closed Bug 1011833 (apz-e10s) Opened 10 years ago Closed 9 years ago

[e10s] Implement mac APZC for e10s

Categories

(Core :: Panning and Zooming, defect)

Product:
Core
Component:
Panning and Zooming
Platform:
All
macOS
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED FIXED
Milestone:
mozilla40
Tracking Flags:
Tracking Status
e10s + ---

People

(Reporter: BenWa, Unassigned)

References

Details

(Keywords: feature)

Attachments

(1 file, 1 obsolete file)

hacky prototype
(deleted), patch
Details | Diff | Splinter Review 
To enable mac APZC for e10s we need:

(1) (a) Decide how input events are routed. We can start off with routing them through the main thread of the compositor which is more portable and closer to b2g, or (b) we can use the approach of bug 944938 and use an event tap.
(2) Teach APZC how to deal with mouse wheel, desktop gesture. For e10s this also means routing these new events through TabParent.
Blocks: core-e10s
Attached patch hacky prototype (obsolete) (deleted) — — Splinter Review 
Here's a totally hacky proof of concept. This gets us APZC and is less invasive than bug 944938 but is a worse solution. We probably don't want to translate things like wheel events to touch events.
Attachment #8424286 - Flags: feedback?(mstange)
Attached patch hacky prototype (deleted) — — Splinter Review 

        Attachment #8424286 -
        Attachment is obsolete: true

        Attachment #8424286 -
        Flags: feedback?(mstange)

        Attachment #8424287 -
        Flags: feedback?(mstange)

    
    

    
  
Comment on attachment 8424287 [details] [diff] [review]
hacky prototype

For now let's wait for mstange' refactored patch queue in the other bug.

        Attachment #8424287 -
        Flags: feedback?(mstange)

    
  
Hardware: x86 → All
Depends on: 1148868

    
    

    
  
This was fixed by the work that got it working on Windows, and bug 1148868.
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla40
QA Contact: vasilica.mihasca
Depends on: 1210376

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






  

    
  







  

    

      
Attachment

      

      
      
      
      
    

    

      

        

          

            
General

            

              Creator: 



            

            
Created: 

            
Updated: 

            
Size: