1056989 - [PDF Viewer] Title and close (X) button is not present when viewing a PDF

Status: RESOLVED WORKSFORME

(Firefox OS Graveyard :: Gaia::PDF Viewer, defect)

Description

[Jordan de Geus(JordanD)]

Description:
When users are viewing a PDF file, upon viewing the top/header of the PDF, users will observe the PDF viewer lacks displaying the title of the PDF as well as the (X)/close button that allows the user to close the file. Users are required to enter task manager/card viewer to close the PDF.

Repro Steps:
1) Update a Flame to 20140820040203
2) Launch Browser> Search "PDF"> Select a link leading to a PDF such as the 5th link for "Bitcoin"
3) Once download completes tap notification
4) Observe header containing title of PDF and (X)/Exit/Close button is not present

Actual:
Header containing Title and Close/(X)/Exit button is not displaying when viewing PDF's

Expected:
Header contains Title and a Close/(X) button

Environmental Variables:
Device: Flame Master (319mb)
Build ID: 20140820040203
Gaia: df39c463259d348396ef7f143c2c780eeb8f02d8
Gecko: ffdd1a398105
Version: 34.0a1 (Master)
Firmware Version: v123
User Agent: Mozilla/5.0 (Mobile; rv:34.0) Gecko/34.0 Firefox/34.0

Repro frequency: 3/3
Link to failed test case: https://moztrap.mozilla.org/manage/case/4069/
See attached: 
Screenshot and logcat

Comment 1

[Jordan de Geus(JordanD)]

Attachment: PDF.png

Comment 2

[Jordan de Geus(JordanD)]

Attachment: logcat_20140821_1056.txt

Comment 3

[Jordan de Geus(JordanD)]

This issue DOES OCCUR on Flame 2.1 (512mb), Open C 2.1

Actual: Opening a downloaded PDF, users will not see a header containing a title or a (X)/Close button

Flame 2.1 (512mb)

Environmental Variables:
Device: Flame Master 
BuildID: 20140820040203193
Gaia: df39c463259d348396ef7f143c2c780eeb8f02d8
Gecko: ffdd1a398105
Version: 34.0a1 (Master)
Firmware Version: v123
User Agent: Mozilla/5.0 (Mobile; rv:34.0) Gecko/34.0 Firefox/34.0

Open C 2.1

Environmental Variables:
Device: Open_C Master
Build ID: 20140820040203
Gaia: df39c463259d348396ef7f143c2c780eeb8f02d8
Gecko: ffdd1a398105
Version: 34.0a1 (Master)
Firmware Version: P821A10V1.0.0B06_LOG_DL
User Agent: Mozilla/5.0 (Mobile; rv:34.0) Gecko/34.0 Firefox/34.0

-----------------------------------------------------------------

This issue DOES NOT occur on Flame 2.0 (319mb), Flame 1.4 (319mb), Open C 2.0, Open C 1.4

Actual: Users see a header containing both a (X)/Close button and title

Flame 2.0

Environmental Variables:
Device: Flame 2.0 (319mb)
BuildID: 20140820000201
Gaia: 88db39a0826086024631049d83ae6aa397f0918d
Gecko: 2092ac87eceb
Version: 32.0 (2.0)
Firmware Version: v123
User Agent: Mozilla/5.0 (Mobile; rv:32.0) Gecko/32.0 Firefox/32.0

Flame 1.4 (319mb)

Environmental Variables:
Device: Flame 1.4
Build ID: 20140820003001
Gaia: 4f92950e6d96326785a249e8acb704da3647616b
Gecko: e1de5a959089
Version: 30.0 (1.4)
Firmware Version: v123
User Agent: Mozilla/5.0 (Mobile; rv:30.0) Gecko/30.0 Firefox/30.0

Open C 2.0

Environmental Variables:
Device: Open_C 2.0
BuildID: 20140820000201
Gaia: 88db39a0826086024631049d83ae6aa397f0918d
Gecko: 2092ac87eceb
Version: 32.0 (2.0)
Firmware Version: P821A10V1.0.0B06_LOG_DL
User Agent: Mozilla/5.0 (Mobile; rv:32.0) Gecko/32.0 Firefox/32.0

Open C 1.4

Environmental Variables:
Device: Open_C 1.4
Build ID: 20140820003001
Gaia: 4f92950e6d96326785a249e8acb704da3647616b
Gecko: e1de5a959089
Version: 30.0 (1.4)
Firmware Version: P821A10V1.0.0B06_LOG_DL
User Agent: Mozilla/5.0 (Mobile; rv:30.0) Gecko/30.0 Firefox/30.0

Comment 4

[KTucker [:KTucker][Inactive 3/4/2016]]

[Blocking Requested - why for this release]:

This is a regression from 2.0 and is a bad user flow so nominating 2.1?

Comment 5

[Rachel Pribble [:rpribble]]

B2G Inbound Last Working:
Device: Flame 2.1 Master
BuildID: 20140818002315
Gaia: d3b5e42a58597ca2f4c02107c915b79c10b851e3
Gecko: 4867ddbf5474
Version: 34.0a1 (2.1 Master)
Firmware: V123
User Agent: Mozilla/5.0 (Mobile; rv:33.0) Gecko/33.0 Firefox/33.0

B2G Inbound First Broken:
Device: Flame 2.1 Master
BuildID: 20140818012716
Gaia: 988392a0e186aebe67358fd80189a2810f1b0150
Gecko: fbcd09b9e3b5
Version: 34.0a1 (2.1 Master)
Firmware: V123
User Agent: Mozilla/5.0 (Mobile; rv:33.0) Gecko/33.0 Firefox/33.0

Swap:
First broken gecko/Last working gaia - Issue DOES NOT reproduce
Gaia: d3b5e42a58597ca2f4c02107c915b79c10b851e3
Gecko: 988392a0e186aebe67358fd80189a2810f1b0150

First broken gaia/Last working gecko - Issue DOES reproduce
Gaia: 988392a0e186aebe67358fd80189a2810f1b0150
Gecko: 4867ddbf5474

Gaia pushlog:
https://github.com/mozilla-b2g/gaia/compare/d3b5e42a58597ca2f4c02107c915b79c10b851e3...988392a0e186aebe67358fd80189a2810f1b0150

Possibly broken by bug 1017328.

Comment 6

[Joshua Mitchell (Inactive)]

Broken by bug 1017328?

Comment 7

[yifan [:yifan][:yliao]]

bug 1017328 consists of 2 commits which updated pdf.js to 1.0.524 and 1.0.700. This bug could be reproduced on pdfjs version 1.0.700 . Tried the latest 1.0.717 but failed either.

I'll revert it back to the working 1.0.524.

Comment 8

[Kevin Grandon :kgrandon]

This is getting really confusing now as there are mulitple open bugs in various states. I see a backout in bug 1017328 comment 18, so I'm not sure of the current state. 

I'm just going to add a qawanted keyword to see if *this* issue reproduces, or has been resolved by backouts in other bugs. TBH - I am a little lost without digging in further here.

Comment 9

[Kevin Grandon :kgrandon]

And you can see how confused I am because that comment was supposed to be for bug 1016817 :)

Comment 10

[Joshua Mitchell (Inactive)]

Kevin - this issue no longer repro's in the latest 2.1

Actual Results - pdf files have titles and the X (to close)

Device: Flame Master
Build ID: 20140825085054
Gaia: a25ae14dbd2fe3e25144a7064efc0cc4e31042b8
Gecko: 4ca2bd0722d9
Version: 34.0a1 (Master)
Firmware Version: v123
User Agent: Mozilla/5.0 (Mobile; rv:34.0) Gecko/34.0 Firefox/34.0



I didn't know if you wanted to close this as a specific fixed by so I did not close as WFM

Comment 11

[Kevin Grandon :kgrandon]

Resolved WFM seems to be fine for now. Thanks!