Closed
Bug 588452
Opened 14 years ago
Closed 14 years ago
Multitouch: black boxes when zooming and a user can now pan
Categories
(Firefox for Android Graveyard :: Panning/Zooming, defect)
Tracking
(fennec2.0a1+)
VERIFIED
FIXED
Tracking | Status | |
---|---|---|
fennec | 2.0a1+ | --- |
People
(Reporter: aakashd, Assigned: mbrubeck)
Details
Attachments
(1 file, 1 obsolete file)
3.04 KB,
patch
|
stechz
:
review+
|
Details | Diff | Splinter Review |
Build id: Mozilla/5.0 (Android; U; Linux armv71; en-US; rv:2.0b4pre) Gecko/20100818 Namoroka/4.0b4pre Fennec/2.0a1pre Note: this is different from bug 584865. blassey mentions that bug was due to artifacts. This one seems to be some strange factoring Steps to Reproduce: 1. Go to addons.mozilla.org 2. Start a multitouch zoom 3. Move the bottom left finger up and down. Actual Results: Black boxes pop up in step 2 and the page starts panning in step 3 Expected Results: Black boxes should not pop up in step 2 and the page should not pan in step 3.
Reporter | ||
Updated•14 years ago
|
Summary: black boxes on zoom when zooming → Multitouch: black boxes when zooming and a user can now pan
Reporter | ||
Updated•14 years ago
|
tracking-fennec: --- → ?
Updated•14 years ago
|
tracking-fennec: ? → 2.0b1+
Updated•14 years ago
|
tracking-fennec: 2.0b1+ → 2.0a1+
Assignee | ||
Updated•14 years ago
|
Assignee: nobody → mbrubeck
Assignee | ||
Comment 1•14 years ago
|
||
If AnimatedZoom.finish() was called before the MozAsyncCanvasRender event was received, then (a) the event handler would fail because this.snapshot would be set to null, and (b) if a new AnimatedZoom object was created in the meantime, then it could render to the canvas while the previous render was still pending. This patch prevents any use of the snapshot canvas while the MozAsyncCanvasRender event is still pending, and correctly handles the case where finish() is called before handleEvent().
Attachment #467279 -
Flags: review?(webapps)
Assignee | ||
Comment 2•14 years ago
|
||
Fix an exception when this.bv is undefined when finish() is called.
Attachment #467279 -
Attachment is obsolete: true
Attachment #467287 -
Flags: review?(webapps)
Attachment #467279 -
Flags: review?(webapps)
Updated•14 years ago
|
Attachment #467287 -
Flags: review?(webapps) → review+
Assignee | ||
Comment 3•14 years ago
|
||
http://hg.mozilla.org/mobile-browser/rev/fa32126e5d30
Status: NEW → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
Reporter | ||
Comment 4•14 years ago
|
||
verified FIXED on build: Mozilla/5.0 (Android; U; Linux armv71; en-US; rv:2.0b4pre) Gecko/20100819 Namoroka/4.0b4pre Fennec/2.0a1pre
Status: RESOLVED → VERIFIED
You need to log in
before you can comment on or make changes to this bug.
Description
•