1026488 - PDF viewer OK in FF30 Does not work in FF31b1 if disabled cookies

Status: VERIFIED FIXED

(Firefox :: PDF Viewer, defect)

Description

[RobertJ]

User Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.9; rv:31.0) Gecko/20100101 Firefox/31.0 (Beta/Release)
Build ID: 20140610163407

Steps to reproduce:

Went to this site

http://cdn-www.myharmony.com/files/harmony650-usermanual-en.pdf




Actual results:

In FF30 the document is displayed. In FF31b1 nothing happens.

Also tested this using other sites with the same result.

Comment 1

[RobertJ]

Just checked this in Aurora and it works fine.

Also noticed that in about:config there is an extra preference in 31b

pdfjs.ifAvailableShowOutlineOnLoad --> false

Also in Aurora this preference is there but it is not in 31b

pdfjs.sidebarViewOnLoad --> 0

Comment 2

[RobertJ]

Just checked in Nightly and it works just fine.

Comment 3

[RobertJ]

Just checked this in FF31b2. The problem also exists there.

Comment 4

[RobertJ]

Have run some tests and not sure what is going on. Using this for testing

http://cdn-www.myharmony.com/files/harmony650-usermanual-en.pdf

1) Run FF30.0 from default profile -> OK

2) Run FF31b2 from default profile -> DOES NOT DISPLAY

3) Run FF31b2 from default profile in safe mode -> DOES NOT DISPLAY

4) Run FF31b2 with new blank profile -> OK

1) and 2) together would suggest it is not a profile issue. 

However, 3) and 4) would suggest it not an extension issue; but, is a profile issue.

At this point will leave this open and go to mozillazine to look for ideas.

Comment 5

[RobertJ]

There was something in the prefs.js file that FF31b did not like. After an hour rebuilt and all seems good.

Comment 6

[RobertJ]

I have found the issue which causes PDF Viewer to not work in FF31b. 

In Preferences > History when using custom settings, if you DO NOT CHECK "Accept cookies from sites" the PDF Viewer does not render. Note, this is true EVEN IF THERE ARE NO COOKIES REQUIRED. Not sure if this is unique to Mac or not.

This is not an issue for FF30, FF32 (Aurora) and FF33 (Nightly).

Comment 7

[Alice0775 White]

I can reproduce the problem in 31b1 if disabled cookies.

I think that it is worth to fix this in 31beta for next ESR31.

Comment 8

[Alice0775 White]

Regression window(m-c)
Good:
https://hg.mozilla.org/mozilla-central/rev/3d34a3b6443a
Mozilla/5.0 (Windows NT 6.1; WOW64; rv:31.0) Gecko/20100101 Firefox/31.0 ID:20140415061201
Bad:
https://hg.mozilla.org/mozilla-central/rev/d969fb1b3c24
Mozilla/5.0 (Windows NT 6.1; WOW64; rv:31.0) Gecko/20100101 Firefox/31.0 ID:20140415091804
Pushlog:
http://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=3d34a3b6443a&tochange=d969fb1b3c24

Regressed by: 
076498ab4476	Brendan Dahl — Bug 995431 - Update pdf.js to version 1.0.21. r=yury, r=Mossop, f=RyanVM

Comment 9

[Ryan VanderMeulen [:RyanVM]]

Alice, can you bisect to what pdf.js update in Fx32 fixed it? Very much agreed that this needs fixing on 31.

Comment 10

[Alice0775 White]

Progression window(m-c)
Bad:
https://hg.mozilla.org/mozilla-central/rev/420f4c65a67f
Mozilla/5.0 (Windows NT 6.1; WOW64; rv:32.0) Gecko/20100101 Firefox/32.0 ID:20140509051431
Fixed:
https://hg.mozilla.org/mozilla-central/rev/007380afa681
Mozilla/5.0 (Windows NT 6.1; WOW64; rv:32.0) Gecko/20100101 Firefox/32.0 ID:20140509123830
Pushlog:
http://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=420f4c65a67f&tochange=007380afa681

Seemed fixed by:
970b90975e1c	Ryan VanderMeulen — Bug 1007627 - Update pdf.js to version 1.0.162

Comment 11

[Ryan VanderMeulen [:RyanVM]]

<3 as always :)

Looks like this might be a dupe of bug 1000777, which was fixed in that update.

Comment 12

[Sylvestre Ledru [:Sylvestre]]

Critical regression. Tracking.

Comment 13

[Yury Delendik (:yury)]

Attachment: Uplift patch for FF31

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

Comment 14

[Yury Delendik (:yury)]

Comment on attachment 8442790 [details] [diff] [review]
Uplift patch for FF31

[Approval Request Comment]
Bug caused by (feature/regressing bug #): Bug 1007627
User impact if declined: Users that disabled cookies via about:config will not be able view PDF files.
Testing completed (on m-c, etc.): on m-c and aurora
Risk to taking this patch (and alternatives if risky): really low and only affects PDF viewer
String or IDL/UUID changes made by this patch: none

Comment 15

[RobertJ]

Not familiar with the process so please excuse me. What beta release of FF31 is this likely to hit?

Comment 16

[Sylvestre Ledru [:Sylvestre]]

Probably beta 3 if it gets uplifted soon (gtb soon). Beta 4 otherwise.

Comment 17

[Liz Henry (:lizzard)]

This should get verified once it lands so I'm adding the tag now while I remember to do it.

Comment 18

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

https://hg.mozilla.org/releases/mozilla-beta/rev/77362ce24f4f

Comment 19

[RobertJ]

So does that mean FF31 release or FF31b4?

Comment 20

[Sylvestre Ledru [:Sylvestre]]

RobertJ, that means that it should be part of the 31 beta4 release.

Comment 21

[[SV Manager] Florin Mezei, QA (:FlorinMezei)]

Reproduced the issue based on comment 0 and comment 6, with Firefox 31 Beta 1, 2 or 3.
Verified as fixed on Firefox 31 Beta 4 (Build ID: 20140623175014) on Windows 7 x64, Windows 8 x64, Ubuntu 13.04 x64 and Mac OS X 10.6.