Closed
Bug 1009963
Opened 11 years ago
Closed 11 years ago
[B2G][1.4][Open_C] Cut the rope - Fast-tapping sound icon for replay icon brings the pause menu.
Categories
(Firefox OS Graveyard :: Gaia::System, defect)
Tracking
(blocking-b2g:1.4+, b2g-v1.3 unaffected, b2g-v1.4 verified, b2g-v2.0 verified)
Tracking | Status | |
---|---|---|
b2g-v1.3 | --- | unaffected |
b2g-v1.4 | --- | verified |
b2g-v2.0 | --- | verified |
People
(Reporter: ychung, Assigned: etienne)
References
Details
(Keywords: regression, Whiteboard: [openc-1.4-exploratory])
Attachments
(3 files)
Description:
When the user taps the sound or replay icon during the gameplay screen, the pause menu is displayed.
Repro Steps:
1) Update a Open_C to BuildID: 20140513000208.
2) Download "Cut the rope" from the Marketplace.
3) Launch the application.
4) Select "Play".
5) Select "Cardboard Box"
6) Select any level available.
7) Tap the sound icon or the replay icon next to "MENU".
8) Observe the pause menu is displayed.
Actual:
The pause menu is displayed, after being shown in the up-side-down vertical view for a second.
Expected:
The sound setting turns on/off when the user selects the sound button, or the game is reloaded on the current level when the user selects the replay button.
1.4F Environmental Variables:
Device: Open_C 1.4F MOZ
BuildID: 20140513000208
Gaia: b40103dec34a147c9018a1af76eb21c3184f2f93
Gecko: c140bcd02b9b
Version: 30.0
Firmware Version: P821A10V1.0.0B06_LOG_DL
Repro frequency: 90%
Link to failed test case: N/A
See attached: Video, logcat
Reporter | ||
Comment 1•11 years ago
|
||
This issue DOES occur in Buri 1.4.
1.4 Environmental Variables:
Device: Buri 1.4 MOZ
BuildID: 20140513000208
Gaia: b40103dec34a147c9018a1af76eb21c3184f2f93
Gecko: c140bcd02b9b
Version: 30.0
Firmware Version: v1.2-device.cfg
This issue DOES occur in Flame 1.4.
1.4 Environmental Variables:
Device: Flame 1.4 MOZ
BuildID: 20140512000204
Gaia: 17fb44880e95bc7ae363a609d811bf5a9a067b5b
Gecko: ec24f847e7c0
Version: 30.0
Firmware Version: v10F
Reporter | ||
Updated•11 years ago
|
status-b2g-v1.4:
--- → affected
Reporter | ||
Comment 2•11 years ago
|
||
Reporter | ||
Comment 3•11 years ago
|
||
This issues does NOT occur in Open_C 1.3
1.3 Environmental Variables:
Device: Open_C 1.3 MOZ
BuildID: 20140505052400
Gaia: Unknown Git commit; build date shown here.
Gecko:
Version: 28.0
Firmware Version: P821A10V1.0.0B06_LOG_DL
Reporter | ||
Updated•11 years ago
|
status-b2g-v1.3:
--- → unaffected
Comment 4•11 years ago
|
||
Can we get a clarification of what behavior is exactly seen on 1.3?
Reporter | ||
Comment 5•11 years ago
|
||
The sound button and the replay button responded correctly on 1.3, without bringing the pause menu. When the user selects the sound button, the button changes to on or off. When the user selects the replay button, the game restarts on the same level. The buttons responded correctly regardless the speed of the tapping by the user.
Comment 6•11 years ago
|
||
Maybe a DOM bug?
blocking-b2g: --- → 1.4?
Component: General → DOM
Keywords: regression,
regressionwindow-wanted
Product: Firefox OS → Core
Version: unspecified → 30 Branch
Updated•11 years ago
|
Priority: P2 → --
Comment 7•11 years ago
|
||
Issue occurs on the earliest available tinderbox build for Open C 1.4.
Result: The pause menu is displayed, after being shown in the up-side-down vertical view for a second.
Environmental Variables:
Device: Open C v1.4 Tinderbox
BuildID: 20140424123005
Gaia: fc95009476fac9ce205a59b237d146ca7f6f42e7
Gecko: 37237034e45c
Version: 30.0a2
Firmware Version: P821A10V1.0.0B06_LOG_DL
Keywords: regressionwindow-wanted
Does this happen on master?
Comment 10•11 years ago
|
||
Switching to qawanted to check 2.0.
Keywords: regressionwindow-wanted → qawanted
Updated•11 years ago
|
blocking-b2g: 1.4? → 1.4+
Updated•11 years ago
|
Keywords: regressionwindow-wanted
Updated•11 years ago
|
QA Contact: ekramer
Comment 12•11 years ago
|
||
We do not have b2b-inbound builds available for this period of time. Therefore, the regression window was found using Tinderbox builds.
Last Working Environmental Variables:
Device: Buri
BuildID: 20140203150906
Gaia: 75e9691f02b9d18585c18a5434beeff39ee7ea20
Gecko: 23555d9a4a17
Version: 30.0a1
Firmware Version: v1.2-device.cfg
First Broken Environmental Variables:
Device: Buri
BuildID: 20140203151506
Gaia: 75e9691f02b9d18585c18a5434beeff39ee7ea20
Gecko: c150845d077d
Version: 30.0a1
Firmware Version: v1.2-device.cfg
Gaia is the same on both builds. This indicates that it is a Gecko issue.
Gecko Pushlog: http://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=23555d9a4a17&tochange=c150845d077d
Keywords: regressionwindow-wanted
Comment 13•11 years ago
|
||
The window here doesn't make any sense. There's only one gecko bug in that range & that bug doesn't seem relevant to the root cause here.
Keywords: regressionwindow-wanted
Comment 14•11 years ago
|
||
I have worked with Erik and we determined that the two builds above do not actually have the same Gaia.
Mozilla Central Hamachi Regression window:
Last Working Environmental Variables:
Device: Buri
BuildID: 20140203150906
Gaia: 53483b08ac780d19c085874006763bb1ef45c83d
Gecko: 23555d9a4a17
Version: 30.0a1
First Broken Environmental Variables:
Device: Buri
BuildID: 20140203151506
Gaia: 75e9691f02b9d18585c18a5434beeff39ee7ea20
Gecko: c150845d077d
Version: 30.0a1
Last Working gaia / First Broken gekko - Issue does not occur
Gaia: 53483b08ac780d19c085874006763bb1ef45c83d
Gecko: c150845d077d
First Broken gaia / Last Working gekko - Issue occurs - This indicates it is a Gaia issue
Gaia: 75e9691f02b9d18585c18a5434beeff39ee7ea20
Gecko: 23555d9a4a17
Gaia Pushlog: https://github.com/mozilla-b2g/gaia/compare/53483b08ac780d19c085874006763bb1ef45c83d...75e9691f02b9d18585c18a5434beeff39ee7ea20
B2g inbound builds are not available as far back as necessary to get a more precise window.
Keywords: regressionwindow-wanted
Comment 15•11 years ago
|
||
So there's a bunch of touch handling changes in the system app in this regression range. Maybe the root cause is due to one of those changes?
Tim - What do you think?
Component: DOM → Gaia::System::Window Mgmt
Flags: needinfo?(timdream)
Product: Core → Firefox OS
Version: 30 Branch → unspecified
Comment 16•11 years ago
|
||
Judging by the area touched, I suspect this is a regression of bug 958584.
Component: Gaia::System::Window Mgmt → Gaia::System
Depends on: 958584
Flags: needinfo?(timdream)
Flags: needinfo?(etienne)
Flags: needinfo?(21)
Comment 18•11 years ago
|
||
Can't do that. The range in comment 14 is as deep as we can go on tinderbox.
Keywords: qawanted
Assignee | ||
Comment 19•11 years ago
|
||
(In reply to Tim Guan-tin Chien [:timdream] (MoCo-TPE) (please ni?) from comment #16)
> Judging by the area touched, I suspect this is a regression of bug 958584.
Yes probably, having a look.
Flags: needinfo?(etienne)
Assignee | ||
Updated•11 years ago
|
Assignee: nobody → etienne
Assignee | ||
Comment 20•11 years ago
|
||
Nice one, the event forwarding works perfectly, but since we don't prevent default the system-app-only mouse events the cut the rope frame get's blurred and decides to show the menu.
Patch incoming :)
Flags: needinfo?(21)
Assignee | ||
Comment 21•11 years ago
|
||
Attachment #8425447 -
Flags: review?(21)
Comment 22•11 years ago
|
||
So that's basically a workaround for bug 1005815 ?
Comment 23•11 years ago
|
||
Comment on attachment 8425447 [details]
Gaia PR
Didn't realized this is a 1.4+. So let's land the workaround in 1.4 and we should really move forward on 1.5 with the real fix.
r+ with a comment referencing the real issue.
Attachment #8425447 -
Flags: review?(21) → review+
Comment 24•11 years ago
|
||
(In reply to Vivien Nicolas (:vingtetun) (:21) (NOT reading bugmails, needinfo? please) from comment #23)
> Comment on attachment 8425447 [details]
> Gaia PR
>
> Didn't realized this is a 1.4+. So let's land the workaround in 1.4 and we
> should really move forward on 1.5 with the real fix.
>
> r+ with a comment referencing the real issue.
Note - I'd rather that we would land the workaround on all branches & file a followup for making the real fix. The reason for this is that landing an upper branch workaround only causes us to carry over blockers across releases. We also have to remember to land the workaround again if the real fix doesn't make it into 2.0 in time.
Assignee | ||
Comment 25•11 years ago
|
||
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Assignee | ||
Comment 26•11 years ago
|
||
(In reply to Jason Smith [:jsmith] from comment #24)
> (In reply to Vivien Nicolas (:vingtetun) (:21) (NOT reading bugmails,
> needinfo? please) from comment #23)
> > Comment on attachment 8425447 [details]
> > Gaia PR
> >
> > Didn't realized this is a 1.4+. So let's land the workaround in 1.4 and we
> > should really move forward on 1.5 with the real fix.
> >
> > r+ with a comment referencing the real issue.
>
> Note - I'd rather that we would land the workaround on all branches & file a
> followup for making the real fix. The reason for this is that landing an
> upper branch workaround only causes us to carry over blockers across
> releases. We also have to remember to land the workaround again if the real
> fix doesn't make it into 2.0 in time.
Makes sense, this patch landed on master. The real fix is tracked in a separate bug and mentioned in the code so everything should be ok.
Comment 27•11 years ago
|
||
status-b2g-v2.0:
--- → fixed
Target Milestone: --- → 2.0 S2 (23may)
Comment 28•11 years ago
|
||
Verified as fixed on 1.4 in the following devices:
1.4F Environmental Variables:
Device: Flame 1.4F
BuildID: 20140522000200
Gaia: 233dd43b3b976e66a619dbc1b4044ed1e3ca3e34
Gecko: c3c0c57daef8
Version: 30.0
Firmware Version: v10F-3
1.4 Environmental Variables:
Device: Open_C 1.4
BuildID: 20140522000200
Gaia: 233dd43b3b976e66a619dbc1b4044ed1e3ca3e34
Gecko: c3c0c57daef8
Version: 30.0
Firmware Version: P821A10V1.0.0B06_LOG_DL
1.4 Environmental Variables:
Device: Buri 1.4 MOZ
BuildID: 20140522000200
Gaia: 233dd43b3b976e66a619dbc1b4044ed1e3ca3e34
Gecko: c3c0c57daef8
Version: 30.0
Firmware Version: v1.2-device.cfg
Comment 29•11 years ago
|
||
Verified as fixed on master
2.0 Environmental Variables:
Device: Open_C 2.0
BuildID: 20140522040230
Gaia: ef66efa34ed8a559c8998bde688fae88eb943a7a
Gecko: b40296602083
Version: 32.0a1
Firmware Version: P821A10V1.0.0B06_LOG_DL
You need to log in
before you can comment on or make changes to this bug.
Description
•