Closed
Bug 751678
Opened 9 years ago
Closed 9 years ago
Low-res screenshot happens when Fennec is in the background
Categories
(Firefox for Android Graveyard :: General, defect)
Tracking
(firefox14 verified, firefox15 verified, firefox16 verified, firefox17 verified, blocking-fennec1.0 +)
VERIFIED
FIXED
Firefox 15
People
(Reporter: mfinkle, Assigned: blassey)
Details
Attachments
(1 file)
1.75 KB,
patch
|
mfinkle
:
review+
mfinkle
:
approval-mozilla-aurora+
|
Details | Diff | Splinter Review |
We should probably turn-off low-res screenshot code when in the background.
Updated•9 years ago
|
OS: Linux → Android
Hardware: x86 → ARM
Version: unspecified → Trunk
Assignee | ||
Comment 1•9 years ago
|
||
Attachment #620947 -
Flags: review?(mark.finkle)
Reporter | ||
Comment 2•9 years ago
|
||
Comment on attachment 620947 [details] [diff] [review] patch >+ GeckoAppShell.sIsPaused = false; >+ GeckoAppShell.sIsPaused = true; >+ public static boolean sIsPaused = false; We already have a flag like this in all GeckoActivity classes, like GeckoApp. You should be able to use: GeckoApp.mAppContext.isApplicationInBackground() r+ if we don't need sIsPaused
Attachment #620947 -
Flags: review?(mark.finkle) → review+
Comment 3•9 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/def98285ad87
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 15
Reporter | ||
Comment 4•9 years ago
|
||
Does this bug need to be fixed on aurora?
Reporter | ||
Comment 6•9 years ago
|
||
this bug depends on bug 751690, which has been uplifted to aurora
Reporter | ||
Comment 7•9 years ago
|
||
Comment on attachment 620947 [details] [diff] [review] patch [Triage Comment]
Attachment #620947 -
Flags: approval-mozilla-aurora+
Reporter | ||
Comment 8•9 years ago
|
||
https://hg.mozilla.org/releases/mozilla-aurora/rev/61edcd0409bb
status-firefox15:
--- → fixed
Comment 9•9 years ago
|
||
WFM all around
Updated•2 months ago
|
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•