1160915 - [Status Bar] rocketbar is not shown after swiping down from screenshot preview page

Status: RESOLVED WONTFIX

(Firefox OS Graveyard :: Gaia::System::Status bar, Utility tray, Notification, defect)

Description

[Hermes Cheng (inactive after July 27, 2015)]

*STR:
1. take a screenshot
2. tap the notification to enter screenshot preview page
3. swipe down to show status bar

*Expected result:
After step 3, the rocketbar is shown on status bar.

*Actual result:
After step 3, the rocketbar is not shown on status bar.

*Test env:
**2.2
Build ID               20150503002500
Gaia Revision          8d14361337e608c8cdf165ea5034db5eda23b618
Gaia Date              2015-05-01 18:23:46
Gecko Revision         https://hg.mozilla.org/releases/mozilla-b2g37_v2_2/rev/cb7cb6597c91
Gecko Version          37.0
Device Name            flame
Firmware(Release)      4.4.2
Firmware(Incremental)  eng.cltbld.20150503.040203
Firmware Date          Sun May  3 04:02:15 EDT 2015
Bootloader             L1TC000118D0

**master
Build ID               20150503010201
Gaia Revision          e18cce173840d6ff07fb6f1f0e0ffb58b99aab3e
Gaia Date              2015-05-02 04:27:01
Gecko Revision         https://hg.mozilla.org/mozilla-central/rev/dc5f85980a82
Gecko Version          40.0a1
Device Name            flame
Firmware(Release)      4.4.2
Firmware(Incremental)  eng.cltbld.20150503.050218
Firmware Date          Sun May  3 05:02:30 EDT 2015
Bootloader             L1TC000118D0


**master on N5
Build ID               20150503160200
Gaia Revision          e18cce173840d6ff07fb6f1f0e0ffb58b99aab3e
Gaia Date              2015-05-02 04:27:01
Gecko Revision         https://hg.mozilla.org/mozilla-central/rev/dc5f85980a82
Gecko Version          40.0a1
Device Name            hammerhead
Firmware(Release)      5.1
Firmware(Incremental)  eng.cltbld.20150503.193743
Firmware Date          Sun May  3 19:38:00 EDT 2015
Bootloader             HHZ12f

Comment 1

[Hermes Cheng (inactive after July 27, 2015)]

This is a regression and function broken. nom 2.2 blocker.

Comment 2

[Jayme Mercado [:JMercado] (Inactive 3/4/2016)]

This has been like this since Bug 1043284.

B2g-inbound Regression Window

Last Working 
Environmental Variables:
Device: Flame 2.1
BuildID: 20140725073206
Gaia: dd8d9ad3e6ad7a0ed778353a745eda4cae44fcb4
Gecko: 8dc3c0b11f4d
Version: 34.0a1 (2.1) 
Firmware Version: v123
User Agent: Mozilla/5.0 (Mobile; rv:34.0) Gecko/34.0 Firefox/34.0

First Broken 
Environmental Variables:
Device: Flame 2.1
BuildID: 20140725084705
Gaia: f4174cf2bce6d530a9ef7d2a1c3b4df81d748fde
Gecko: a418eea48be4
Version: 34.0a1 (2.1) 
Firmware Version: v123
User Agent: Mozilla/5.0 (Mobile; rv:34.0) Gecko/34.0 Firefox/34.0

Last Working gaia / First Broken gecko - Issue does NOT occur
Gaia: dd8d9ad3e6ad7a0ed778353a745eda4cae44fcb4
Gecko: a418eea48be4

First Broken gaia / Last Working gecko - Issue DOES occur
Gaia: f4174cf2bce6d530a9ef7d2a1c3b4df81d748fde
Gecko: 8dc3c0b11f4d

Gaia Pushlog: https://github.com/mozilla-b2g/gaia/compare/dd8d9ad3e6ad7a0ed778353a745eda4cae44fcb4...f4174cf2bce6d530a9ef7d2a1c3b4df81d748fde

Comment 3

[KTucker [:KTucker][Inactive 3/4/2016]]

Etienne, can you take a look at this please? This might have been caused by the work done for bug 1043284.

Comment 4

[Gregor Wagner [:gwagner]]

Kevin said he will take it.

Comment 5

[GitHub Autolander Bot]

Attachment: [gaia] KevinGrandon:bug_1160915_default_combined_chrome > mozilla-b2g:master

Comment 6

[Kevin Grandon :kgrandon]

Comment on attachment 8605234 [details] [review]
[gaia] KevinGrandon:bug_1160915_default_combined_chrome > mozilla-b2g:master

Alive - could you take a look at this? Activities seem to not have a chrome attribute defined, and it seems our chrome logic on whether or not to use the combined chrome may be wrong. (The combined chrome is the only one that works as the shrunken rocketbar).

Comment 7

[(inactive after 6/18) Alive Kuo [:alive]]

Comment on attachment 8605234 [details] [review]
[gaia] KevinGrandon:bug_1160915_default_combined_chrome > mozilla-b2g:master

Looks fine, thanks.

Comment 8

[GitHub Autolander Bot]

Pull request has landed in master: https://github.com/mozilla-b2g/gaia/commit/c5b3aa4fc662d961eba79ef24d1378a243109e29

Comment 9

[Kevin Grandon :kgrandon]

Backed out for causing bug: 1165950

https://github.com/mozilla-b2g/gaia/commit/1030116046e434e21b6cae6b8f4af1fac10205e0

Comment 10

[Kevin Grandon :kgrandon]

I took a look here and I don't think this is a regression - I didn't see this behavior on 2.1. Adding qawanted to do branch checks and verify if it's a regression.

Also it seems pretty late for this patch as it's risky and has already caused regressions. I'm renominating it for blocking with the suggestion that we don't block.

Comment 11

[sunny]

Attachment: logcat_1854.txt

This bug has been failed verified on latest Nightly Flame v2.2 and Nexus5 v2.2.
See attachments:logcat_1854.txt and 1854.mp4

Reproduce rate: 5/5
Repro STR:
1. take a screenshot
2. tap the notification to enter screenshot preview page
3. swipe down to show status bar

Actual result:
After step 3, the rocketbar is not shown on status bar.

Device: Flame 2.2 build(Fail)
Build ID               20150519162501
Gaia Revision          63e9eeec3032318f8a240f80b6a184fa4b50b6e1
Gaia Date              2015-05-19 17:52:15
Gecko Revision         https://hg.mozilla.org/releases/mozilla-b2g37_v2_2/rev/4e078e1364d3
Gecko Version          37.0
Device Name            flame
Firmware(Release)      4.4.2
Firmware(Incremental)  eng.cltbld.20150519.200807
Firmware Date          Tue May 19 20:08:18 EDT 2015
Bootloader             L1TC000118D0

Device: Nexus5 2.2 build(Fail)
Build ID               20150519162501
Gaia Revision          63e9eeec3032318f8a240f80b6a184fa4b50b6e1
Gaia Date              2015-05-19 17:52:15
Gecko Revision         https://hg.mozilla.org/releases/mozilla-b2g37_v2_2/rev/4e078e1364d3
Gecko Version          37.0
Device Name            hammerhead
Firmware(Release)      5.1
Firmware(Incremental)  eng.cltbld.20150519.195445
Firmware Date          Tue May 19 19:55:01 EDT 2015
Bootloader             HHZ12f

Comment 12

[sunny]

Attachment: 1854.mp4

Comment 13

[sunny]

This bug has been failed verified on latest Nightly Flame v2.2 and Nexus5 v2.2.
See attachments:logcat_1854.txt and 1854.mp4

Reproduce rate: 5/5
Repro STR:
1. take a screenshot
2. tap the notification to enter screenshot preview page
3. swipe down to show status bar

Actual result:
After step 3, the rocketbar is not shown on status bar.

Device: Flame 2.2 build(Fail)
Build ID               20150519162501
Gaia Revision          63e9eeec3032318f8a240f80b6a184fa4b50b6e1
Gaia Date              2015-05-19 17:52:15
Gecko Revision         https://hg.mozilla.org/releases/mozilla-b2g37_v2_2/rev/4e078e1364d3
Gecko Version          37.0
Device Name            flame
Firmware(Release)      4.4.2
Firmware(Incremental)  eng.cltbld.20150519.200807
Firmware Date          Tue May 19 20:08:18 EDT 2015
Bootloader             L1TC000118D0

Device: Nexus5 2.2 build(Fail)
Build ID               20150519162501
Gaia Revision          63e9eeec3032318f8a240f80b6a184fa4b50b6e1
Gaia Date              2015-05-19 17:52:15
Gecko Revision         https://hg.mozilla.org/releases/mozilla-b2g37_v2_2/rev/4e078e1364d3
Gecko Version          37.0
Device Name            hammerhead
Firmware(Release)      5.1
Firmware(Incremental)  eng.cltbld.20150519.195445
Firmware Date          Tue May 19 19:55:01 EDT 2015
Bootloader             HHZ12f

Comment 14

[Gregor Wagner [:gwagner]]

Kevin, are you still working on this?

Comment 15

[Kevin Grandon :kgrandon]

I'm going to suggest that we close this as wontfix. The reason being: the screenshot preview page is an activity, we can either choose to show the rocketbar on activity screens or not, but it should be consistent. The last time this landed bug 1165950 was opened due to showing rocketbar inside activities, which is consistent with the fix. We can either hide rocketbar, as it is now, or show it everywhere.

We can change this - but I think that would be more feature work, and not a bug.