1001214 - Unnecessary size changes to background apps

Status: RESOLVED INVALID

(Core :: Layout, defect)

Description

[Botond Ballo [:botond]]

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.

Comment 1

[Timothy Nikkel (:tnikkel)]

Seems like bug 1003870 will fix this too.

Comment 2

[Jet Villegas (inactive)]

(In reply to Timothy Nikkel (:tn) from comment #1)
> Seems like bug 1003870 will fix this too.

Did it?

Comment 3

[Botond Ballo [:botond]]

Redirecting needinfo to Timothy while I'm away.

Comment 4

[Timothy Nikkel (: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.

Comment 5

[Botond Ballo [:botond]]

(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.

Comment 6

[Gregory Pappas [:gregp]]

Closing old B2G bugs