Closed Bug 477105 Opened 16 years ago Closed 16 years ago

improve zoom responsiveness by using _clippedPageDrawing mode

Categories

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

Product:
Firefox for Android Graveyard
Component:
Panning/Zooming
Platform:
x86
Linux
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(Not tracked)

Status:
RESOLVED FIXED

People

(Reporter: taras.mozilla, Assigned: taras.mozilla)

References

Details

Attachments

(3 files, 4 obsolete files)

1/2 - restrict draws to the visible part of the canvas
3.67 KB, patch
pavlov
: review+
Details | Diff | Splinter Review
bonus opts
3.72 KB, patch
pavlov
: review+
Details | Diff | Splinter Review
speed up zoomToElement, alternate version (fixed)
5.75 KB, patch
taras.mozilla
: review+
Details | Diff | Splinter Review
No description provided.
Attached patch step 1) avoid redundant draws (obsolete) — — Splinter Review
Depends on: 477708
Patch1/2 flushes the canvas on pageload(to speed up pre-pan flush), flushes before a pan and also freezes screen updates during panning.
Attachment #360989 - Attachment is obsolete: true
Attachment #363749 - Flags: review?(pavlov)
Attached patch 2/2 make zoomToElement fast (obsolete) — — Splinter Review
this avoids redraws during panning+zooming involved in doubletapping elements
Comment on attachment 363750 [details] [diff] [review] 2/2 make zoomToElement fast Gavin, I'm not too proud of the skipViewportHandler hack, got a better suggestion?
Attachment #363750 - Flags: review?(gavin.sharp)
Attached patch bonus opts — — Splinter Review
reduce js overhead of painting.
Attachment #364348 - Flags: review?(pavlov)
Attachment #363749 - Flags: review?(pavlov) → review+
Attachment #363750 - Flags: review?(gavin.sharp) → review+
Attached patch 2/2 make zoomToElement fast (obsolete) — — Splinter Review
moved the magic flag to CanvasBrowser
Attachment #363750 - Attachment is obsolete: true
Attachment #364357 - Flags: review?(gavin.sharp)
Attachment #364357 - Attachment is obsolete: true
Attachment #364401 - Flags: review?(tglek)
Attachment #364357 - Flags: review?(gavin.sharp)
Assignee: nobody → tglek
Attachment #364401 - Attachment is obsolete: true
Attachment #364416 - Flags: review?(tglek)
Attachment #364401 - Flags: review?(tglek)
Attachment #364416 - Flags: review?(tglek) → review+
Attachment #364348 - Flags: review?(pavlov) → review+
pushed http://hg.mozilla.org/mobile-browser/rev/44a16d1bdaf5
Status: NEW → RESOLVED
Closed: 16 years ago
Resolution: --- → FIXED
Component: General → Panning/Zooming
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: