Oleg Zasypkin [:azasypkin] Reporter
	Description • 10 years ago

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

	Oleg Zasypkin [:azasypkin] Reporter
	Comment 1 • 10 years ago

Wilson, Yan could you please look into it?

	Yan Or Assignee
	Comment 2 • 10 years ago

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

	Kevin Grandon :kgrandon
	Comment 3 • 10 years ago

Comment on attachment 8482979 [details]
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!

	Yan Or Assignee
	Comment 4 • 10 years ago

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

	Carsten Book [:Tomcat]
	Comment 5 • 10 years ago

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

	Oleg Zasypkin [:azasypkin] Reporter
	Comment 6 • 10 years ago

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!

	Yan Or Assignee
	Comment 7 • 10 years ago

Stephany,

What's the process for uplifting this to 2.1?

	Stephany Wilkes
	Comment 8 • 10 years ago

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

	Yan Or Assignee
	Comment 9 • 10 years ago

Comment on attachment 8482979 [details]
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

	Ryan VanderMeulen [:RyanVM]
	Comment 10 • 10 years ago

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

	Coler
	Comment 11 • 10 years ago

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