Closed
Bug 1076394
Opened 10 years ago
Closed 10 years ago
Perma-failing gaia integration tests
Categories
(Firefox OS Graveyard :: Gaia, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: kgrandon, Unassigned)
References
Details
https://tbpl.mozilla.org/php/getParsedLog.php?id=49333641&tree=Fx-Team#error4
It looks like manage_test.js and pick_test.js are now perma-red.
My hunch is that it's something in here: https://tbpl.mozilla.org/?tree=B2g-Inbound&showall=1&jobname=b2g_ubuntu64_vm%20b2g-inbound%20opt%20test%20gaia-integration&rev=bb23019a3bf1
Possibly Bug 1064669 on fx-team: https://tbpl.mozilla.org/?tree=Fx-Team&showall=1&rev=a1d7b16101c7
Reporter | ||
Comment 1•10 years ago
|
||
Nevermind, bug 1064669 only has java changes. That's probably not it.
Reporter | ||
Comment 2•10 years ago
|
||
Just got this stacktrace: JavaScriptError: (17) TypeError: setting a property that has only a getter
Remote Stack:
@app://ringtones.gaiamobile.org/js/tone_player.js, line 127
Links to: https://github.com/mozilla-b2g/gaia/blob/f79dcebc4a1cae49f3dfb88306a9ddcf8f7970aa/apps/ringtones/js/tone_player.js#L127
Seems to be a change in mozAudioChannelType that may be causing this.
Reporter | ||
Comment 3•10 years ago
|
||
This looks suspicious: https://hg.mozilla.org/integration/b2g-inbound/rev/11f95edf23a4
Baku or Roc - It looks like mozAudioChannelType was changed to readonly. Is setting this not a valid option for gaia? How do we differentiate between ringer or normal in gaia?
Reporter | ||
Comment 4•10 years ago
|
||
Just adding a ni? on Jim for awareness/feedback here.
Flags: needinfo?(squibblyflabbetydoo)
Reporter | ||
Comment 5•10 years ago
|
||
It looks like bug 1073615 was backed out and that should fix this issue.
Status: NEW → RESOLVED
Closed: 10 years ago
Flags: needinfo?(squibblyflabbetydoo)
Flags: needinfo?(roc)
Flags: needinfo?(amarchesini)
Resolution: --- → FIXED
Comment 6•10 years ago
|
||
(In reply to Kevin Grandon :kgrandon from comment #3)
> This looks suspicious:
> https://hg.mozilla.org/integration/b2g-inbound/rev/11f95edf23a4
>
> Baku or Roc - It looks like mozAudioChannelType was changed to readonl Is
> setting this not a valid option for gaia? How do we differentiate between
> ringer or normal in gaia?
The patch has been backed out but we are going to reland a new version of it soon.
You should use a new AudioChannel object instead setting a new value to mozAudioChannelType:
var ac1 = new AudioContext(); // normal by default.
var ac2 = new AudioContext('ringer');
Reporter | ||
Comment 7•10 years ago
|
||
(In reply to Andrea Marchesini (:baku) from comment #6)
> The patch has been backed out but we are going to reland a new version of it
> soon.
> You should use a new AudioChannel object instead setting a new value to
> mozAudioChannelType:
>
> var ac1 = new AudioContext(); // normal by default.
> var ac2 = new AudioContext('ringer');
Please make sure you do not land without the gaia patches, or you will break the phone.
You need to log in
before you can comment on or make changes to this bug.
Description
•