1166240 - Missing Pocket icon in reader view mode

Status: VERIFIED FIXED

(Firefox :: Pocket, defect, P1)

Description

[Petruta Horea [:phorea], Desktop Test Engineering]

Reproducible with Firefox 38.0.5 beta 3 under Win 7 (x64) and Ubuntu 14.04 (x64).
Not reproducible with:
 Firefox 38.0.5 beta 3 under Mac OS X 10.9.5.
 latest Aurora 40.0a2 2015-05-18 after enabling Pocket
 latest Nightly 41.0a1 2015-05-18

Steps to reproduce:
1. Open an article that can be viewed in Reader Mode (eg http://edition.cnn.com/2015/05/19/africa/cameroon-cinema-revival/index.html)
2. Select the Reader View icon in location bar

Actual results:
The Pocket icon from the left menu is blank. The button can be clicked and used, but has no icon on it: http://i.imgur.com/VzC67hn.png

Expected results:
The Pocket icon is displayed for all platforms.

Comment 1

[Andrei Vaida [:avaida]]

It actually seems that something went wrong in Bug 1155517 - the Pocket icon is missing from Reader View on both 38.0.5b2 and 38.0.5b3. Please not that this happens _only_ on Windows, tested with Windows 7 (x64), Windows 7 (x86), Windows 8.1 (x64).

Comment 2

[Jared Wein [:jaws] (please needinfo? me)]

Attachment: Patch

Approval Request Comment
[Feature/regressing bug #]: pocket
[User impact if declined]: missing pocket icon in reader mode for majority of windows users (button with no icon)
[Describe test coverage new/current, TreeHerder]: manual QA testing
[Risks and why]: very low risk, file was not added to separate manifest section
[String/UUID change made/needed]: none

Comment 3

[Jared Wein [:jaws] (please needinfo? me)]

Comment on attachment 8607556 [details] [diff] [review]
Patch

Or Gijs if he can get to the review sooner than dolske due to timezones.

Comment 4

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/releases/mozilla-beta/rev/2d0b4ea8ae1b

Comment 5

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/releases/mozilla-release/rev/58d8fb9fc5e3

Comment 6

[Andrei Vaida [:avaida]]

Verified fixed on RC 38.0.5-build3 (20150521175336), using Windows 7 (x64) and Windows 8.1 (x86). Also made sure that button still looks and works as expected on Mac OS X 10.9.5 and Ubuntu 14.04 (x64).