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)

ARM
Android
defect
Not set
normal

Tracking

(firefox14 verified, firefox15 verified, firefox16 verified, firefox17 verified, blocking-fennec1.0 +)

VERIFIED FIXED
Firefox 15
Tracking Status
firefox14 --- verified
firefox15 --- verified
firefox16 --- verified
firefox17 --- verified
blocking-fennec1.0 --- +

People

(Reporter: mfinkle, Assigned: blassey)

Details

Attachments

(1 file)

We should probably turn-off low-res screenshot code when in the background.
OS: Linux → Android
Hardware: x86 → ARM
Version: unspecified → Trunk
Attached patch patchSplinter Review
Attachment #620947 - Flags: review?(mark.finkle)
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+
https://hg.mozilla.org/mozilla-central/rev/def98285ad87
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 15
Does this bug need to be fixed on aurora?
I see this on the current (14) beta build.
this bug depends on bug 751690, which has been uplifted to aurora
Comment on attachment 620947 [details] [diff] [review]
patch

[Triage Comment]
Attachment #620947 - Flags: approval-mozilla-aurora+
WFM all around
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in before you can comment on or make changes to this bug.