688689 - New throbber animation for Fennec

Status: RESOLVED WONTFIX

(Firefox for Android Graveyard :: General, defect)

Description

[Matt Brubeck (:mbrubeck)]

Attachment: part 1: new throbber asset

+++ This bug was initially created as a clone of Bug #679927 +++

Since the two-stage throbber in bug 679927 requires some more code changes, as an initial step we will just replace the current single-stage throbber animation with the new animated icon from that bug.

The first patch just replaces the current throbber icon.  We will also need new backgrounds for endcaps on SSL sites that work better with the dark blue color of the new throbber.

Comment 1

[Mark Finkle (:mfinkle) (use needinfo?)]

Comment on attachment 561964 [details] [diff] [review]
part 1: new throbber asset

Did you try pngcrush on the files to remove any cruft?

Comment 2

[Matt Brubeck (:mbrubeck)]

Attachment: part 2: new SSL endcaps

This uses the new endcap assets and colors from bug 679927.  It also removes some unused images from the honeycomb theme folder.

All images in both patches have been pngcrushed.

Comment 3

[Matt Brubeck (:mbrubeck)]

https://hg.mozilla.org/integration/mozilla-inbound/rev/6e370a64b8d1
https://hg.mozilla.org/integration/mozilla-inbound/rev/96b970e6291b

Comment 4

[Matt Brubeck (:mbrubeck)]

https://hg.mozilla.org/mozilla-central/rev/96b970e6291b
https://hg.mozilla.org/mozilla-central/rev/6e370a64b8d1

Comment 5

[Aaron Train [:aaronmt]]

Mozilla/5.0 (Android; Linux armv7l; rv:9.0a1) Gecko/20110925 Firefox/9.0a1 Fennec/9.0a1

Comment 6

[Matt Brubeck (:mbrubeck)]

Backed out because of Maemo perf regressions (bug 689578).  This can re-land after we verify that it does not regress performance on Android (waiting on bug 689350).

http://hg.mozilla.org/mozilla-central/rev/115e3576acf5

Comment 7

[Matt Brubeck (:mbrubeck)]

We did not see similar regressions on Android, but Mark points out that this might be because the Android Tegra machines are dual-core while the Maemo N900 machines are single-core.  We need performance tests from single-core Android devices to make sure we are not regressing them.

Comment 8

[Sriram Ramasubramanian [:sriram]]

Gone native!