Closed Bug 1076394 Opened 5 years ago Closed 5 years ago

Perma-failing gaia integration tests

Categories

(Firefox OS Graveyard :: Gaia, defect)

x86
macOS
defect
Not set

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: kgrandon, Unassigned)

References

Details

Nevermind, bug 1064669 only has java changes. That's probably not it.
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.
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?
Blocks: 1073615
Flags: needinfo?(roc)
Flags: needinfo?(amarchesini)
Just adding a ni? on Jim for awareness/feedback here.
Flags: needinfo?(squibblyflabbetydoo)
It looks like bug 1073615 was backed out and that should fix this issue.
Status: NEW → RESOLVED
Closed: 5 years ago
Flags: needinfo?(squibblyflabbetydoo)
Flags: needinfo?(roc)
Flags: needinfo?(amarchesini)
Resolution: --- → FIXED
(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');
(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.