983971 - [system] the "download" indicator is not displayed

Status: RESOLVED FIXED

(Core :: Graphics: Layers, defect)

Description

[Julien Wajsberg [:julienw]]

We used to have a "network active" indicator when the phone was downloading/uploading. It's not displayed anymore in 1.4 (tested on Peak).

I don't know if that's expected but that's was certainly useful.

Comment 1

[Julien Wajsberg [:julienw]]

Works fine in a 1.3 build (still on the peak).

QAWanted for testing on other devices.

Comment 2

[Jason Smith [:jsmith]]

We need clearer STR here.

Comment 3

[Josh Schmitt [Joshs]]

Attachment: Download.png

Repro steps:
1) Proceed to the Marketplace
2) Download any app (I used Wikipedia)
3) Select the 'Free' button
4) Select 'Install'
5) Observe the notifications bar when downloading

I have included a screenshot with the 1.3 build and the 1.4 build.

The 1.3 build has the download icon that is animating, on 1.4 there is no icon.

Comment 4

[Josh Schmitt [Joshs]]

1.4 
Environmental Variables:
Device: Buri 1.4 MOZ
BuildID: 20140318000203
Gaia: c03a6af9028c4b74a84b5a98085bbb0c07261175
Gecko: 3776f72f1967
Version: 30.0a2
Firmware Version: V1.2-device.cfg

1.3
Environmental Variables:
Device: Buri 1.3 MOZ
BuildID: 20140319004003
Gaia: f2f2be55dcca1d6775592ef478948045935a91ed
Gecko: 87d90ecfe616
Version: 28.0
Firmware Version: V1.2-device.cfg

Comment 5

[Natalya Kot [:nkot]]

The following regression window was found using b2g-inbound builds:

~ Last Working ~
BuildID: 20140213063024
Gaia: 3c2e748e4d57551527366eba00428a0f5ccc4012
Gecko: 3fd6a4150f85
Version: 30.0a1
Firmware: V1.2-device.cfg

~ First Broken ~ 
BuildID: 20140213064824
Gaia: 3c2e748e4d57551527366eba00428a0f5ccc4012
Gecko: 935cb4d15364
Version: 30.0a1
Firmware: V1.2-device.cfg

Please note, the Gaia is the same so this is a Gecko regression!

Pushlog:
http://hg.mozilla.org/integration/b2g-inbound/pushloghtml?fromchange=3fd6a4150f85&tochange=935cb4d15364

Comment 6

[Jason Smith [:jsmith]]

Buri doesn't have seccomp enabled, so this has to be caused by bug 946720.

Comment 7

[Jason Smith [:jsmith]]

Needs a logcat.

Comment 8

[Natalya Kot [:nkot]]

Attachment: logcat

Here is the logcat... though it seems like there are some other issues with Marketplace. Intermittently, I was unable to install the app at all, should investigate that more before filing a bug.

Comment 9

[Julien Wajsberg [:julienw]]

In comment 0, I was only talking about the network indicator, which should be displayed for any network activity, eg fetching a webpage in the browser.

From the comment 3 I see this is not the only indicator failing though.

Comment 10

[Kai-Zhen Li [:seinlin][:kli]]

Attachment: 2014-03-24-04-44-02.png

I am working on dolphin + 1.4 branch and I notice that both network indicator and network activity can be displayed.

1.4 Environmental Variables:
Gaia      a10224e21a5c2acca929508edaad35e471413ff0
Gecko     4159b06a14f441302348a33dded34a988ce6ce7a
BuildID   20140325144735
Version   30.0a2

Comment 11

[Milan Sreckovic [:milan] (needinfo for best results)]

I'm not sure dolphin is :), but if this is Buri/Peak only so far, perhaps its ICS specific. Assigning to Sotaro based on Comment 5.

Comment 12

[Sotaro Ikeda [:sotaro]]

The indocator are drawn as synchronous Image layer. In Bug 946720, I also changed to use gralloc buffer for Image Layer. It seems to affect it. Adreno 200 has a bug that can not render gralloc buffers of size from 9px to 16px. See Bug 873937. It might be affected.

Comment 13

[Sotaro Ikeda [:sotaro]]

Attachment: patch - Do not use gralloc for small size on ICS gonk

Confirmed the patch fixed the problem.

Comment 15

[Sotaro Ikeda [:sotaro]]

https://hg.mozilla.org/integration/b2g-inbound/rev/42751d107303

Comment 16

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

https://hg.mozilla.org/mozilla-central/rev/42751d107303

Comment 17

[Ryan VanderMeulen [:RyanVM]]

Needs a branch patch for Aurora uplift.

Comment 18

[Sotaro Ikeda [:sotaro]]

Attachment: patch for b2gv1.4 - Do not use gralloc for small size on ICS gonk

patch for aurora(b2gv1.4).

Comment 19

[Sotaro Ikeda [:sotaro]]

Comment on attachment 8399856 [details] [diff] [review]
patch for b2gv1.4 - Do not use gralloc for small size on ICS gonk

Carry "r=nical".

Comment 20

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/releases/mozilla-aurora/rev/4823a86eb16c