1061682 - [Messages] Header shows wrong icons for back\close actions

Status: VERIFIED FIXED

(Firefox OS Graveyard :: Gaia::SMS, defect)

Description

[Oleg Zasypkin [:azasypkin]]

STR:
1. Open any message thread;
2. Observe "back" icon in the header - correct;
3. Long tap on message and go to the Report panel;
4. Observe "close" icon in the header - correct;
5. Go back to Thread panel and observe "back" icon.

Actual result: "close" icons is displayed (with <gaia-header action="back">)

Expected result: "back" icon should be displayed instead.

Looks like it was broken by patch for bug 1059833. It works fine with initial Messages gaia-header patch for bug 1011604 that was landed earlier.

[Blocking Requested - why for this release]: Regression

Comment 1

[Oleg Zasypkin [:azasypkin]]

Wilson, Yan could you please look into it?

Comment 2

[Yan Or]

Attachment: PR

Kevin, can you review this fix?  Wilson is out on PTO and you are the next person most familiar with this code.  Thanks.

Comment 3

[Kevin Grandon :kgrandon]

Comment on attachment 8482979 [details] [review]
PR

I didn't go through this specific STR, but since this is a simple patch I'll give it the benefit of the doubt. The code looks fine to me, thanks!

Comment 4

[Yan Or]

Tested the specific STR and behaves as expected with step #5 showing a back button.  Ready to land.

Comment 5

[Carsten Book [:Tomcat]]

master: https://github.com/mozilla-b2g/gaia/commit/8e117ae14d06fc71ec8859d23769a69f6abdb922

Comment 6

[Oleg Zasypkin [:azasypkin]]

Hey Yan,

Thanks for the fix, but I believe it has missed v2.1 train. Could you please request approval to uplift this fix to v2.1?

Thanks!

Comment 7

[Yan Or]

Stephany,

What's the process for uplifting this to 2.1?

Comment 8

[Stephany Wilkes]

Yan, in the Attachment - Details, please sent approval-gaia-v2.1? and complete the questions that appear in the comment box. This requests uplift.

Comment 9

[Yan Or]

Comment on attachment 8482979 [details] [review]
PR

[Approval Request Comment]
[Bug caused by] (feature/regressing bug #): 1059833
[User impact] if declined: regression, wrong icon showing on 'back' button
[Testing completed]: Manually tested on device, TBPL green
[Risk to taking this patch] (and alternatives if risky): minimal risk
[String changes made]: none

Comment 10

[Ryan VanderMeulen [:RyanVM]]

v2.1: https://github.com/mozilla-b2g/gaia/commit/ad2818e5fd3a51484f95724e1d43bc002f30bddf

Comment 11

[Coler]

This issue has been verified successfully on Flame2.1&2.2.
Reproducing rate: 0/5
See attachment: Verify_Flame_button.mp4

Flame2.1 build version:
Gaia-Rev        dbaf3e31c9ba9c3436e074381744f2971e15c7bf
Gecko-Rev       https://hg.mozilla.org/releases/mozilla-b2g34_v2_1/rev/ebce587d2194
Build-ID        20141203001205
Version         34.0

Flame2.2 build version:
Gaia-Rev        725685831f5336cf007e36d9a812aad689604695
Gecko-Rev       https://hg.mozilla.org/mozilla-central/rev/2c9781c3e9b5
Build-ID        20141203040207
Version         37.0a1

Comment 12

[Coler]

Attachment: Verify_Flame_button.MP4