837317 - [video] "[object Object] is now full screen" shows when playing a video

Status: VERIFIED FIXED

(Firefox OS Graveyard :: Gaia::System, defect)

Description

[Naoki Hirata :nhirata (please use needinfo instead of cc)]

Attachment: screenshot

## Environment :
Gecko  http://hg.mozilla.org/releases/mozilla-b2g18_v1_0_0/rev/361d9359f4f3
Gaia   0abc95774d0bbdfe314fa588e09fc92cac3e6427
BuildID 20130130113326
Version 18.0
Unagi
  
## Repro :
1. launch video
2. play a video

## Expected :
full screen and shows video

## Actual :
full screen message of  "[object Object] is now full screen"

## Note :
1. it's probably related to bug 837282 and bug 798647?
2. not sure if it has to do with changes to the sdcard?  need to do further testing.
3. I get the feeling it's more system wide than just browser.

Comment 1

[Naoki Hirata :nhirata (please use needinfo instead of cc)]

er. meant to say browser or video...

Comment 2

[David Flanagan [:djf]]

Recategorising this as a system bug. Its not specific to any one app.  But more likely it is a gecko or chrome bug.

In the past things like this have been something going wrong in a chrome object wrapper.  Cc'ing Fabrice because I think he's dealt with stuff like this before.

Thanks for filing this Naoki. I'd been seeing it today too, but hadn't filed a bug yet.

Comment 3

[David Flanagan [:djf]]

In this bug and the two related bugs mentioned in the description, the [object Object] text is supposed to be the origin of an app.  So somewhere an object is being stringified incorrectly. I'd look for recent changes to the system app and recent changes to the b2g chrome.

Comment 5

[Jason Smith [:jsmith]]

Nom - perm prompts pretty much have to always indicate the origin of the request (security requirement). This is likely a regression.

Comment 7

[[:fabrice] Fabrice Desré]

Attachment: patch

That's a regression from the BEP refactoring done in a performance related bug.

Comment 8

[Dale Harvey (:daleharvey)]

Comment on attachment 709996 [details] [diff] [review]
patch

Testing on device

Comment 9

[Misty Byrd]

Smoketest, regression:
This is still repros in:

Unagi build#20130207070202
Gecko: cf1b9d27345e70daf242514733f848e76241ea1d

Gaia:   7e54ca673277b20b1d91d18477dc44d6ad226761

Comment 10

[[:fabrice] Fabrice Desré]

https://hg.mozilla.org/integration/mozilla-inbound/rev/8ededceee28f

Comment 11

[Ed Morley [:emorley]]

https://hg.mozilla.org/mozilla-central/rev/8ededceee28f

Comment 12

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/releases/mozilla-b2g18/rev/55062e25e787
https://hg.mozilla.org/releases/mozilla-b2g18_v1_0_0/rev/07175f3445bb

Comment 13

[Chris Kreinbring [:CKreinbring]]

No longer appears in Unagi version 20130211070202.