952846 - Don't ship unused PDF.js files in metro installer

Status: VERIFIED FIXED

(Firefox for Metro Graveyard :: Build Config, defect, P2)

Description

[Matt Brubeck (:mbrubeck)]

PDF.js is disabled in Metro Firefox (bug 859579), but we are still shipping an extra copy of the extension in Metro-enabled Windows builds (bug 952844) which adds an extra 1MB or so to our installer size.

On Aurora 28, we don't have any code that uses PDF.js, so we should just stop shipping the extension there.

On Nightly 29, PDF.js code is present but disabled by default.  We should keep that true in nightly builds, but remove the code and extension in release builds at least until the footprint issue is fixed.

Comment 1

[Matt Brubeck (:mbrubeck)]

Attachment: patch for Aurora

[Approval Request Comment]
Bug caused by (feature/regressing bug #): enabling Metro

User impact if declined: Windows installer size increased by ~1MB

Testing completed (on m-c, etc.): Aurora-only patch.  Pushed to Try:
https://tbpl.mozilla.org/?tree=Try&rev=97ad30ae511d

Risk to taking this patch (and alternatives if risky): Low risk patch that only removes unused code from the Metro package directory.

String or IDL/UUID changes made by this patch: None.

Comment 2

[Matt Brubeck (:mbrubeck)]

Attachment: patch for trunk

This patch builds PDF.js code for Metro only when NIGHTLY_BUILD is true.  We can remove this when PDF.js for Metro is ready to ship.

This also catches one line of Shumway code that I missed in bug 923596.

Comment 3

[Matt Brubeck (:mbrubeck)]

https://hg.mozilla.org/integration/fx-team/rev/ae27b82c41ee

Comment 4

[(PTO June 19-July 7) Ryan VanderMeulen]

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

Comment 5

[(PTO June 19-July 7) Ryan VanderMeulen]

https://hg.mozilla.org/releases/mozilla-aurora/rev/8c10d0e5d85d

Comment 6

[Matt Brubeck (:mbrubeck)]

I accidentally ifdef'ed the wrong lines in package-manifest.in.  Philor noticed that this broke the build when pushing m-c to Try with Aurora build config.  Push a fix to fx-team:
https://hg.mozilla.org/integration/fx-team/rev/211c36cf894a

Comment 7

[Tim Taubert [:ttaubert] (inactive)]

https://hg.mozilla.org/mozilla-central/rev/211c36cf894a

Comment 8

[Manuela Muntean [Away]]

For iteration #22, verified as fixed with latest Nightly and Aurora on Win 8.1 64-bit: .pdf files can't be viewed inline in Metro mode.