Closed Bug 587814 Opened 14 years ago Closed 14 years ago

Smoother panning using mozAnimationFrame

Categories

(Firefox for Android Graveyard :: Panning/Zooming, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: stechz, Assigned: stechz)

Details

Attachments

(1 file)

Attached patch PatchSplinter Review
      No description provided.
Attachment #466441 - Attachment is patch: true
Attachment #466441 - Attachment mime type: application/octet-stream → text/plain
Attachment #466441 - Flags: review?(mark.finkle)
Comment on attachment 466441 [details] [diff] [review]
Patch


> AnimatedZoom.prototype.finish = function() {
>   try {
>+    Elements.viewBuffer.style.display = "none";
>+

What is this doing?

>     // resume live rendering
>     this.bv.resumeRendering(true);
> 
>     // if we actually zoomed somewhere, clean up the UI to normal
>     if (this.zoomRect)
>       Browser.setVisibleRect(this.zoomRect);
>+
>   }

No blank line

r+ with question answered
Attachment #466441 - Flags: review?(mark.finkle) → review+
OS: Mac OS X → All
Hardware: x86 → All
Sorry, both of those were from another experiment.  They can be removed.
Pushed http://hg.mozilla.org/mobile-browser/rev/3e452a03bf0f
Status: NEW → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
Looks like part of this was lost in a merge.  Pushed:
http://hg.mozilla.org/mobile-browser/rev/9e68083047aa
Flags: in-testsuite?
bugspam
Assignee: nobody → ben
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: