[B2G][Tarako][PDF Viewer]Selecting PDF from Browser App attempts to load unsuccessfully

VERIFIED FIXED

Status

VERIFIED FIXED
5 years ago
5 years ago

People

(Reporter: mclemmons, Assigned: timdream)

Tracking

unspecified
ARM
Gonk (Firefox OS)

Firefox Tracking Flags

(blocking-b2g:1.3T+, b2g-v1.3T verified)

Details

(Whiteboard: [tarako-exploratory], URL)

Attachments

(2 attachments)

(Reporter)

Description

5 years ago
Created attachment 8435429 [details]
Firewatch log

According to bug 998099 PDF viewer is supposed to be removed from Tarako, however user taps Browser app and navigates to a URL that is a PDF. When user selects the URL, the page loads very slowly and demonstrates one of the following: Tab crash, device restart, or page fully loads but user interaction with the page is limited. Taps and swipes are not reflected as expected to go to that area of the page. 

Repro Steps: 
1) Update a Tarako to BuildID: 20140604014021
2) Tap Browser App and navigate to http://espn.go.com/pdf/2013/1004/arod-suit.pdf
3) Tap URL link
4) Observe device behavior 

Actual: 
Page loads slowly and either 1) completes after nearly 3 minutes load but user interaction is limited 2) device is restarted or 3) browser tab crash

Expected: 
No PDF attempt to load from browser

Notes: 
Repro frequency: (5/5, 100%) 

See attached: video = https://www.youtube.com/watch?v=O1b1hdLzw9I,firewatch

Environmental Variables:
Device: Tarako 1.3T
Build ID: 20140604014021
Gaia: 41a763154fbac34bef6baf17a201e50f52f2b72a
Gecko: fed0b4e6da6c
Version: 28.1 (1.3T) 
Firmware Version: SP6821a-Gonk-4.0-5-12
User Agent: Mozilla/5.0 (Mobile; rv:28.0) Gecko/28.0 Firefox/28.0
(Reporter)

Comment 1

5 years ago
QA Wanted for logcat and other device checks.
Keywords: qawanted
Looks like even though it's out of the list, it's being picked up somewhere else.
blocking-b2g: --- → 1.3T?
Flags: needinfo?(timdream)
Yes, I can verify the PDF Viewer app is still there :'(.
Assignee: nobody → timdream
Status: NEW → ASSIGNED
Flags: needinfo?(timdream)
Need to contact release ops to see the switches we are using for building tarako.

I suspect we are not using either apps-production.list or apps-engineering.list.
Depends on: 998099
Aki, could you tell me if our pvt tarako build call Gaia Makefile with any of the following variable?

GAIA_APP_TARGET
GAIA_APP_CONFIG
DISTRIBUTION_APP_CONFIG
PRODUCTION
DOGFOOD

Does pvt builds build with distribution dir?

Thanks!
Flags: needinfo?(aki)

Comment 7

5 years ago
The logs are all available here:
https://tbpl.mozilla.org/?tree=Mozilla-B2g28-v1.3t

I clicked on the latest Tarako 'B', which is a user, non-eng depend build, and clicked 'view full log'.

https://tbpl.mozilla.org/php/getParsedLog.php?id=41154626&tree=Mozilla-B2g28-v1.3t&full=1

Of the above env vars you're asking about, I do see 'PRODUCTION=1'.
I don't see a distribution dir.

There are also lines like
15:57:33     INFO - Running command: ['mock_mozilla', '-r', u'mozilla-centos6-i386', '-q', '--cwd', '/builds/slave/b2g_m-b28_v1_3t_tko_dep-000000/build', '--unpriv', '--shell', u'/usr/bin/env MOZILLA_MLS_KEY_FILE=/builds/slave/b2g_m-b28_v1_3t_tko_dep-000000/build/mls.key VARIANT=user MOZ_AUTOMATION=1 GAIA_OPTIMIZE=1 "LESSOPEN=|/usr/bin/lesspipe.sh %s" TMOUT=86400 CVS_RSH=ssh LOGNAME=cltbld USER=cltbld PATH=/tools/python27/bin:/usr/local/bin:/usr/lib64/ccache:/usr/local/bin:/bin:/usr/bin:/usr/local/sbin:/usr/sbin:/sbin:/home/cltbld/bin:/builds/slave/b2g_m-b28_v1_3t_tko_dep-000000/build/compare-locales/scripts SYMBOL_SERVER_USER=b2gbld "WGET_OPTS=-c -q" CCACHE_UMASK=002 LANG=en_US.UTF-8 CCACHE_HASHDIR= TERM=linux SHELL=/bin/bash MOZ_BUILD_DATE=20140605153400 ENABLE_ADB_ROOT=1 SHLVL=1 G_BROKEN_FILENAMES=1 HISTSIZE=1000 SYMBOL_SERVER_PATH=/mnt/netapp/breakpad/symbols_b2g/ GAIA_MEMORY_PROFILE=low LOCALES_FILE=/builds/slave/b2g_m-b28_v1_3t_tko_dep-000000/build/gaia/locales/languages_dev.json SYMBOL_SERVER_HOST=symbolpush.mozilla.org CCACHE_DIR=/builds/ccache PYTHONPATH=:/builds/slave/b2g_m-b28_v1_3t_tko_dep-000000/build/compare-locales/lib L10NBASEDIR=/builds/slave/b2g_m-b28_v1_3t_tko_dep-000000/build/gecko-l10n TINDERBOX_OUTPUT=1 MOZILLA_OFFICIAL=1 "MOZ_CHROME_MULTILOCALE=es-ES pl pt-BR" MAIL=/var/spool/mail/cltbld CCACHE_COMPRESS=1 _=/tools/buildbot/bin/python CCACHE_BASEDIR=/builds/slave/b2g_m-b28_v1_3t_tko_dep-000000/build JS_BINARY=/builds/slave/b2g_m-b28_v1_3t_tko_dep-000000/build/js HOSTNAME=bld-linux64-spot-178.build.releng.use1.mozilla.com SYMBOL_SERVER_SSH_KEY=/home/mock_mozilla/.ssh/b2gbld_dsa LOCALE_BASEDIR=/builds/slave/b2g_m-b28_v1_3t_tko_dep-000000/build/gaia-l10n HISTCONTROL=ignoredups POST_SYMBOL_UPLOAD_CMD=/usr/local/bin/post-symbol-upload.py PWD=/builds/slave/b2g_m-b28_v1_3t_tko_dep-000000 PROPERTIES_FILE=/builds/slave/b2g_m-b28_v1_3t_tko_dep-000000/buildprops.json MOZ_CRASHREPORTER_NO_REPORT=1 B2G_UPDATER=1 B2G_UPDATE_CHANNEL=tarako/1.3.0t/default ./build.sh ""'] in /builds/slave/b2g_m-b28_v1_3t_tko_dep-000000/build

that show the running env.
Flags: needinfo?(aki)
Thank you Aki
This is pretty weird. So I looked at the log above and 

https://pvtbuilds.mozilla.org/pvt/mozilla.org/b2gotoro/nightly/mozilla-b2g28_v1_3t-tarako-eng/latest/logs/log_info.log

And I don't see anything strange that make the build includes pdfjs. I run the same command locally and verified the app is indeed removed too.

Without spending more time on this, I will simply going to purge the pdfjs app from the tree.
Attachment #8435606 - Flags: review?(21)
ni? myself to keep an eye on this to see if this might cause an issue w/ pvtbuild
Flags: needinfo?(nhirata.bugzilla)
v1.3t: https://github.com/mozilla-b2g/gaia/commit/df364e84fefa6f09c6a6828c21126441f424ef91
Status: ASSIGNED → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
Keywords: qawanted
blocking-b2g: 1.3T? → 1.3T+
Verified that pdf viewer doesn't appear at all.

Gaia      5611a68f5c039f62ae74b828feff1530ab066d3a
Gecko     https://hg.mozilla.org/releases/mozilla-b2g28_v1_3t/rev/7011129d1abd
BuildID   20140612014001
Version   28.1
ro.build.version.incremental=eng.cltbld.20140612.051428
ro.build.date=Thu Jun 12 05:14:36 EDT 2014
Tarako
Status: RESOLVED → VERIFIED
Flags: needinfo?(nhirata.bugzilla)
status-b2g-v1.3T: affected → verified
You need to log in before you can comment on or make changes to this bug.