990065 - Screen rotation feature is broken in packaged b2g desktop

Status: RESOLVED FIXED

(Firefox OS Graveyard :: Simulator, defect)

Description

[Alexandre Poirot [:ochameau]]

SimulatorScreen.js is missing in package-manifest.in, so that this recently added xpcom isn't added in b2g tarball. It ends up working in local builds, but not once released in packages...

Comment 1

[Alexandre Poirot [:ochameau]]

Attachment: Ensure shipping simulator screen xpcom in b2g desktop package

Comment 2

[Vivien Nicolas (:vingtetun) (:21) - (NOT reading bugmails, needinfo? please)]

Comment on attachment 8399488 [details] [diff] [review]
Ensure shipping simulator screen xpcom in b2g desktop package

Review of attachment 8399488 [details] [diff] [review]:
-----------------------------------------------------------------

Sigh.

Comment 3

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/integration/mozilla-inbound/rev/65338f03492b

Comment 4

[Ryan VanderMeulen [:RyanVM]]

Either this or bug 990057 made B2G desktop mochitests perma-fail. I backed both out for investigation.
https://hg.mozilla.org/integration/mozilla-inbound/rev/05792ab1e06f

https://tbpl.mozilla.org/php/getParsedLog.php?id=37084445&tree=Mozilla-Inbound

Comment 5

[Alexandre Poirot [:ochameau]]

Let's check if that's really failing because of these patches:
  https://tbpl.mozilla.org/?tree=Try&rev=a98f149fc65e

Comment 6

[Firefox Commit Bot]

The following changesets are now in Firefox Nightly:

> 65338f03492b Bug 990065 - Ensure shipping simulator screen xpcom in b2g desktop package. r=vingtetun
> 05792ab1e06f Backed out changeset 65338f03492b (bug 990065) and changeset 8c3b772518f2 (bug 990057) for B2G desktop mochitest failures.

Nightly Build Information:

        ID: 20140402030201
 Changeset: 4941a2ac0786109b08856738019b016a6c5a66a6
   Version: 31.0a1
      TBPL: https://tbpl.mozilla.org/?rev=4941a2ac0786
       URL: https://ftp.mozilla.org/pub/mozilla.org/firefox/nightly/2014/04/2014-04-02-03-02-01-mozilla-central

Download Links:

>         Linux x86: https://ftp.mozilla.org/pub/mozilla.org/firefox/nightly/2014/04/2014-04-02-03-02-01-mozilla-central/firefox-31.0a1.en-US.linux-i686.tar.bz2
>      Linux x86_64: https://ftp.mozilla.org/pub/mozilla.org/firefox/nightly/2014/04/2014-04-02-03-02-01-mozilla-central/firefox-31.0a1.en-US.linux-x86_64.tar.bz2
> Linux x86_64 ASAN: https://ftp.mozilla.org/pub/mozilla.org/firefox/nightly/2014/04/2014-04-02-03-02-01-mozilla-central/firefox-31.0a1.en-US.linux-x86_64-asan.tar.bz2
>               Mac: https://ftp.mozilla.org/pub/mozilla.org/firefox/nightly/2014/04/2014-04-02-03-02-01-mozilla-central/firefox-31.0a1.en-US.mac.dmg
>             Win32: https://ftp.mozilla.org/pub/mozilla.org/firefox/nightly/2014/04/2014-04-02-03-02-01-mozilla-central/firefox-31.0a1.en-US.win32.installer.exe
>             Win64: https://ftp.mozilla.org/pub/mozilla.org/firefox/nightly/2014/04/2014-04-02-03-02-01-mozilla-central/firefox-31.0a1.en-US.win64-x86_64.installer.exe

Previous Nightly Build Information:

        ID: 20140401030203
 Changeset: 1417d180a1d8665b1a91b897d1cc4cc31e7980d4
   Version: 31.0a1
      TBPL: https://tbpl.mozilla.org/?rev=1417d180a1d8
       URL: https://ftp.mozilla.org/pub/mozilla.org/firefox/nightly/2014/04/2014-04-01-03-02-03-mozilla-central

Comment 7

[Alexandre Poirot [:ochameau]]

I can't believe it is introducing such regression, but out of these two bug, I would most likely due to this one that is actually loaded into b2g desktop. The other is only shiped in simulator addon, that is never loaded in tests (so far).

Comment 8

[Alexandre Poirot [:ochameau]]

Attachment: patch

Ship simulator screen xpcom in b2g desktop package and also prevent it from breaking bfcache...

The xpcom was listener for unload event... which prevent bfcache to work...
And the shared worker mochitest was relying on bfcache to work.
So use observer notification instead.

https://tbpl.mozilla.org/?tree=Try&rev=f2c65186eebb

Comment 9

[Vivien Nicolas (:vingtetun) (:21) - (NOT reading bugmails, needinfo? please)]

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

Review of attachment 8404045 [details] [diff] [review]:
-----------------------------------------------------------------

Nice catch.

::: b2g/components/SimulatorScreen.js
@@ +108,5 @@
>  
>        case 'simulator-orientation-change':
>          this._windows.forEach(fireOrientationEvent);
>          break;
>      }

nit: You may consider a local |let windows = this._windows;| at the beginning of the switch.

Comment 10

[Alexandre Poirot [:ochameau]]

Attachment: patch

Comment 11

[Alexandre Poirot [:ochameau]]

Green try.

Comment 12

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/integration/b2g-inbound/rev/e1c19ed25b12

Comment 13

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/mozilla-central/rev/e1c19ed25b12

Comment 14

[Alexandre Poirot [:ochameau]]

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

See bug 976773 comment 30.
We would like to build 1.4 simulators and need to uplift mostly desktop-specific patches.

This patch modify only desktop codepath.

Comment 15

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/releases/mozilla-aurora/rev/2fd8d8f9d167