Closed Bug 1001214 Opened 11 years ago Closed 2 years ago

Unnecessary size changes to background apps

Categories

(Core :: Layout, defect)

ARM
Gonk (Firefox OS)
defect

Tracking

()

RESOLVED INVALID

People

(Reporter: botond, Unassigned)

References

Details

STR (taken from bug 993930):
  1. Open the Settings app on B2G.
  2. Switch to the Browser app.
  3. Go to m.youtube.com, play a video, and make it fullscreen.

Actual results:
  When the video is made fullscreen, the Setting app's iframe
  calls nsFrameLoader::UpdatePositionAndSize.

Expected results:
  nsFrameLoader::UpdatePositionAndSize is not called for
  iframes of apps in the background.
Seems like bug 1003870 will fix this too.
Depends on: 1003870
(In reply to Timothy Nikkel (:tn) from comment #1)
> Seems like bug 1003870 will fix this too.

Did it?
Flags: needinfo?(botond)
Redirecting needinfo to Timothy while I'm away.
Flags: needinfo?(tnikkel)
I didn't get around to making the required b2g builds (one before the fixes to confirm that I see the same problem as Botond, one after to confirm it's gone), and Botond is back next week so I'll just let him verify.
Flags: needinfo?(tnikkel)
(In reply to Jet Villegas (:jet) from comment #2)
> (In reply to Timothy Nikkel (:tn) from comment #1)
> > Seems like bug 1003870 will fix this too.
> 
> Did it?

I still see the Settings app's iframe calling nsFrameLoader::UpdatePositionAndSize() whenever a youtube video is made fullscreen or brought out of fullscreen using the STR in comment 0.
Flags: needinfo?(botond)
Severity: normal → S3

Closing old B2G bugs

Status: NEW → RESOLVED
Closed: 2 years ago
Resolution: --- → INVALID
You need to log in before you can comment on or make changes to this bug.