Last Comment Bug 714710 - LayerRenderer.checkFrameRateMonitorEnabled should use start() instead of run()
: LayerRenderer.checkFrameRateMonitorEnabled should use start() instead of run()
Product: Firefox for Android
Classification: Client Software
Component: General (show other bugs)
: unspecified
: x86 Linux
: P2 normal (vote)
: Firefox 12
Assigned To: Mark Finkle (:mfinkle) (use needinfo?)
: Sebastian Kaspari (:sebastian)
Depends on:
  Show dependency treegraph
Reported: 2012-01-02 20:54 PST by Mark Finkle (:mfinkle) (use needinfo?)
Modified: 2012-01-06 22:13 PST (History)
2 users (show)
See Also:
Crash Signature:
QA Whiteboard:
Iteration: ---
Points: ---
Has Regression Range: ---
Has STR: ---

patch (1.00 KB, patch)
2012-01-02 21:11 PST, Mark Finkle (:mfinkle) (use needinfo?)
doug.turner: review+
akeybl: approval‑mozilla‑aurora+
Details | Diff | Splinter Review

Description Mark Finkle (:mfinkle) (use needinfo?) 2012-01-02 20:54:21 PST

Calling .run() doesn't actually start the Thread, so we are hurting the main thread IO

We should be calling start() here
Comment 1 Mark Finkle (:mfinkle) (use needinfo?) 2012-01-02 21:11:18 PST
Created attachment 585351 [details] [diff] [review]

simple patch
Comment 2 Mark Finkle (:mfinkle) (use needinfo?) 2012-01-02 22:01:31 PST
Comment 3 Marco Bonardo [::mak] 2012-01-04 04:39:54 PST
Comment 4 Mark Finkle (:mfinkle) (use needinfo?) 2012-01-06 06:26:23 PST
Comment on attachment 585351 [details] [diff] [review]

[Approval Request Comment]
Code cleanup. No risk

Small chance of a the startup improvement, but mainly for correctness.
Comment 5 Alex Keybl [:akeybl] 2012-01-06 11:06:45 PST
Comment on attachment 585351 [details] [diff] [review]

[Triage Comment]
Mobile only - approving for Aurora given the possible perf improvement, otherwise this likely wouldn't meet the criteria for Aurora.
Comment 6 Mark Finkle (:mfinkle) (use needinfo?) 2012-01-06 22:13:03 PST

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