Closed Bug 773397 Opened 12 years ago Closed 12 years ago

Disable pdf.js prior to FF15 beta 5

Categories

(Firefox :: PDF Viewer, defect)

defect
Not set
normal

Tracking

()

VERIFIED FIXED
Firefox 15
Tracking Status
firefox15 + verified

People

(Reporter: akeybl, Assigned: bdahl)

References

Details

Attachments

(1 file, 1 obsolete file)

Asa let us know that we'll keep pdf.js on the test channels till at least FF16's release, so we'll need to disable pdf.js prior to FF15's release.
Summary: Disable pdf.js in our final Beta 14 build → Disable pdf.js in our final Beta 15 build
Assigning this to Brendan for now - Brendan are you the best person to have as the point person for disabling this when the time comes? If not, please re-assign to the person who will be.
Assignee: nobody → bdahl
I can create a patch to disable it, though I don't have commit access.  To disable it will require changing one line(http://mxr.mozilla.org/mozilla-beta/source/browser/app/profile/firefox.js#1165) to true.
(In reply to Brendan Dahl from comment #2)
> I can create a patch to disable it, though I don't have commit access.  To
> disable it will require changing one
> line(http://mxr.mozilla.org/mozilla-beta/source/browser/app/profile/firefox.
> js#1165) to true.

If you can prepare the patch, and get it reviewed, we can flag this as checkin-needed in time for the last beta.
Attached patch disable pdf.js (obsolete) — Splinter Review
Summary: Disable pdf.js in our final Beta 15 build → Disable pdf.js prior to FF15 beta 5
I think it would be more prudent to disable for our second to last beta, just in case.
Assignee: bdahl → gavin.sharp
Assignee: gavin.sharp → bdahl
Attachment #646630 - Flags: review+
Attachment #646630 - Flags: approval-mozilla-beta+
https://hg.mozilla.org/releases/mozilla-beta/rev/7488058a7c3f
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Backed out at Gavin's request for browser_pdfjs_main.js failures:
https://tbpl.mozilla.org/?tree=Mozilla-Beta&rev=7488058a7c3f

https://hg.mozilla.org/releases/mozilla-beta/rev/a9b08621c0f3
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Attached patch disable pdf.jsSplinter Review
Attachment #646630 - Attachment is obsolete: true
https://hg.mozilla.org/releases/mozilla-beta/rev/aff8c2ba16dd
Status: REOPENED → RESOLVED
Closed: 12 years ago12 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 15
Keywords: verifyme
Mozilla/5.0 (Macintosh; Intel Mac OS X 10.6; rv:15.0) Gecko/20100101 Firefox/15.0
Mozilla/5.0 (Windows NT 6.1; rv:15.0) Gecko/20100101 Firefox/15.0
Mozilla/5.0 (X11; Linux i686; rv:15.0) Gecko/20100101 Firefox/15.0

PDF.js was successfully disabled in Firefox 15.0beta5: "Preview in Firefox" option is not available for handling PDFs.
Status: RESOLVED → VERIFIED
Keywords: verifyme
QA Contact: moz.mihaelav
Blocks: 786012
When updating pdf.js 0.4.11 for Firefox 17, it lacked pdfjs/content/web/images/toolbarButton-fullscreen.png. And this lacking is continued in aurora 17.

At nightly 18, I sent a patch (bug 784718), and the patch will be checked-in within days.
Should we check-in it for aurora 17 too? Although disable pdf.js on aurora 17 by this bug's patch, I think that we keep a source-tree as cleanly. This filling does not cause any regression.
(In reply to OHZEKI Tetsuharu [:saneyuki_s] from comment #11)
> When updating pdf.js 0.4.11 for Firefox 17, it lacked
> pdfjs/content/web/images/toolbarButton-fullscreen.png. And this lacking is
> continued in aurora 17.
> 
> At nightly 18, I sent a patch (bug 784718), and the patch will be checked-in
> within days.
> Should we check-in it for aurora 17 too? Although disable pdf.js on aurora
> 17 by this bug's patch, I think that we keep a source-tree as cleanly. This
> filling does not cause any regression.

Opps! I mistake the bug number.
Please read as following:
bug 784718 -> bug 786219
(In reply to OHZEKI Tetsuharu [:saneyuki_s] from comment #11)
> When updating pdf.js 0.4.11 for Firefox 17, it lacked
> pdfjs/content/web/images/toolbarButton-fullscreen.png. And this lacking is
> continued in aurora 17.
> 
> At nightly 18, I sent a patch (bug 784718), and the patch will be checked-in
> within days.
> Should we check-in it for aurora 17 too? Although disable pdf.js on aurora
> 17 by this bug's patch, I think that we keep a source-tree as cleanly. This
> filling does not cause any regression.

PDF.js is now disabled in FF17 (bug 786012). No need to uplift.
Gavin, is there a bug filed (/plan) for the Nightly-only xpcshell failures on aurora due to aff8c2ba16dd ?
(In reply to Ed Morley [:edmorley] from comment #14)
> Gavin, is there a bug filed (/plan) for the Nightly-only xpcshell failures

Just noticed that the aurora commit landed with the wrong bug number, I'll move this discussion to bug 786012.
You need to log in before you can comment on or make changes to this bug.