Closed Bug 1346121 Opened 7 years ago Closed 7 years ago

In ScrollFrameHelper::DecideScrollableLayer we should check for a displayport again after we call MaybeCreateDisplayPort

Categories

(Core :: Panning and Zooming, defect, P3)

Product:
Core
Component:
Panning and Zooming
Type:
defect

Tracking

()

Status:
RESOLVED FIXED
Milestone:
mozilla55
Tracking Flags:
Tracking Status
firefox55 --- fixed

People

(Reporter: tnikkel, Assigned: tnikkel)

References

Details

(Whiteboard: [gfx-noted])

Attachments

(1 file)

patch
1.57 KB, patch
botond
: review+
Details | Diff | Splinter Review 
I even noticed this during review in bug 1246443 (bug 1246443, comment 14), wrote another patch so the main patch could fix this, and then seemed to forget to actually fix the problem there! Let's fix it now.
Attached patch patch — — Splinter Review 

        Attachment #8845752 -
        Flags: review?(botond)
Priority: -- → P3
Whiteboard: [gfx-noted]

        Attachment #8845752 -
        Flags: review?(botond) → review+

    
    

    
  
Pushed by tnikkel@gmail.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/b316fdd7d8fc
In ScrollFrameHelper::DecideScrollableLayer we should check for a displayport again after we call MaybeCreateDisplayPort. r=botond

    
    

    
  
https://hg.mozilla.org/mozilla-central/rev/b316fdd7d8fc
Status: NEW → RESOLVED
Closed: 7 years ago

          status-firefox55:
          affected → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla55

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






  

    
  







  

    

      
Attachment

      

      
      
      
      
    

    

      

        

          

            
General

            

              Creator: 



            

            
Created: 

            
Updated: 

            
Size: