Bug 904084 - fix voicemail test case test_voicemail_statuschanged.js permanent failure. r=yoshi
authorVicamo Yang <vyang@mozilla.com>
Wed, 14 Aug 2013 10:23:58 +0800
changeset 142454 333d778a06191f63715ce5698643c864037176ba
parent 142453 cb9e18eebcc7c569e36fc1dec0585687ae2d42da
child 142455 f8f60ec05deb61bbd8b457538871cc28e920d50e
push id228
push uservyang@mozilla.com
push dateWed, 14 Aug 2013 02:24:14 +0000
treeherderb2g-inbound@333d778a0619 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersyoshi
bugs904084
milestone26.0a1
Bug 904084 - fix voicemail test case test_voicemail_statuschanged.js permanent failure. r=yoshi
dom/system/gonk/RILContentHelper.js
dom/voicemail/test/marionette/manifest.ini
dom/voicemail/test/marionette/pdu_builder.js
--- a/dom/system/gonk/RILContentHelper.js
+++ b/dom/system/gonk/RILContentHelper.js
@@ -1742,17 +1742,17 @@ RILContentHelper.prototype = {
 
     if (this.voicemailStatus.returnMessage != message.returnMessage) {
       changed = true;
       this.voicemailStatus.returnMessage = message.returnMessage;
     }
 
     if (changed) {
       this._deliverEvent("_voicemailListeners",
-                         "voicemailNotification",
+                         "notifyStatusChanged",
                          [this.voicemailStatus]);
     }
   },
 
   _cfRulesToMobileCfInfo: function _cfRulesToMobileCfInfo(rules) {
     for (let i = 0; i < rules.length; i++) {
       let rule = rules[i];
       let info = new MobileCFInfo();
--- a/dom/voicemail/test/marionette/manifest.ini
+++ b/dom/voicemail/test/marionette/manifest.ini
@@ -1,8 +1,7 @@
 [DEFAULT]
 b2g = true
 browser = false
 qemu = true
 
 [test_voicemail_statuschanged.py]
-disabled = Bug 806138
 [test_voicemail_number.js]
--- a/dom/voicemail/test/marionette/pdu_builder.js
+++ b/dom/voicemail/test/marionette/pdu_builder.js
@@ -1,16 +1,14 @@
 /* Any copyright is dedicated to the Public Domain.
  * http://creativecommons.org/publicdomain/zero/1.0/ */
 "use strict";
 
-let Cu = SpecialPowers.wrap(Components).utils;
 let RIL = {};
-
-Cu.import("resource://gre/modules/ril_consts.js", RIL);
+SpecialPowers.Cu.import("resource://gre/modules/ril_consts.js", RIL);
 
 // Only bring in what we need from ril_worker/RadioInterfaceLayer here. Reusing
 // that code turns out to be a nightmare, so there is some code duplication.
 let PDUBuilder = {
   toHexString: function toHexString(n, length) {
     let str = n.toString(16);
     if (str.length < length) {
       for (let i = 0; i < length - str.length; i++) {