Bug 1309723 - Remove DOM/voicemail and related code. r=Ehsan

MozReview-Commit-ID: JDSqQnYD6xL --HG-- extra : rebase_source : 43008cd2eaf97d77d32a6faa9fb4854e3d1f29a8
dothq · Oct 13, 2016 · 42dd1f7 · 42dd1f7
1 parent 88cf93f
commit 42dd1f7
Show file tree

Hide file tree

Showing 44 changed files with 1 addition and 2,424 deletions.
diff --git a/b2g/chrome/content/settings.js b/b2g/chrome/content/settings.js
@@ -130,7 +130,7 @@ SettingsListener.observe('language.current', 'en-US', function(value) {
 // =================== RIL ====================
 (function RILSettingsToPrefs() {
   // DSDS default service IDs
-  ['mms', 'sms', 'telephony', 'voicemail'].forEach(function(key) {
+  ['mms', 'sms', 'telephony'].forEach(function(key) {
     SettingsListener.observe('ril.' + key + '.defaultServiceId', 0,
                              function(value) {
       if (value != null) {

diff --git a/b2g/installer/package-manifest.in b/b2g/installer/package-manifest.in
@@ -203,7 +203,6 @@
 @RESPATH@/components/dom_tv.xpt
 @RESPATH@/components/dom_inputport.xpt
 @RESPATH@/components/dom_views.xpt
-@RESPATH@/components/dom_voicemail.xpt
 #ifdef MOZ_WEBSPEECH
 @RESPATH@/components/dom_webspeechrecognition.xpt
 #endif
@@ -490,8 +489,6 @@
 #ifndef DISABLE_MOZ_RIL_GEOLOC
 @RESPATH@/components/TelephonyService.js
 @RESPATH@/components/TelephonyService.manifest
-@RESPATH@/components/VoicemailService.js
-@RESPATH@/components/VoicemailService.manifest
 #endif
 #endif // MOZ_WIDGET_GONK && MOZ_B2G_RIL
 

diff --git a/browser/installer/package-manifest.in b/browser/installer/package-manifest.in
@@ -213,7 +213,6 @@
 @RESPATH@/components/dom_stylesheets.xpt
 @RESPATH@/components/dom_telephony.xpt
 @RESPATH@/components/dom_traversal.xpt
-@RESPATH@/components/dom_voicemail.xpt
 #ifdef MOZ_WEBSPEECH
 @RESPATH@/components/dom_webspeechrecognition.xpt
 #endif

diff --git a/dom/apps/PermissionsTable.jsm b/dom/apps/PermissionsTable.jsm
@@ -253,11 +253,6 @@ this.PermissionsTable =  { geolocation: {
                              privileged: ALLOW_ACTION,
                              certified: ALLOW_ACTION
                            },
-                           "voicemail": {
-                             app: DENY_ACTION,
-                             privileged: DENY_ACTION,
-                             certified: ALLOW_ACTION
-                           },
                            "idle": {
                              app: DENY_ACTION,
                              privileged: DENY_ACTION,

diff --git a/dom/base/Navigator.cpp b/dom/base/Navigator.cpp
@@ -49,7 +49,6 @@
 #include "mozilla/dom/StorageManager.h"
 #include "mozilla/dom/TCPSocket.h"
 #include "mozilla/dom/Telephony.h"
-#include "mozilla/dom/Voicemail.h"
 #include "mozilla/dom/VRDisplay.h"
 #include "mozilla/dom/workers/RuntimeService.h"
 #include "mozilla/Hal.h"
@@ -219,7 +218,6 @@ NS_IMPL_CYCLE_COLLECTION_TRAVERSE_BEGIN(Navigator)
   NS_IMPL_CYCLE_COLLECTION_TRAVERSE(mIccManager)
   NS_IMPL_CYCLE_COLLECTION_TRAVERSE(mMobileMessageManager)
   NS_IMPL_CYCLE_COLLECTION_TRAVERSE(mTelephony)
-  NS_IMPL_CYCLE_COLLECTION_TRAVERSE(mVoicemail)
   NS_IMPL_CYCLE_COLLECTION_TRAVERSE(mInputPortManager)
   NS_IMPL_CYCLE_COLLECTION_TRAVERSE(mConnection)
   NS_IMPL_CYCLE_COLLECTION_TRAVERSE(mStorageManager)
@@ -303,11 +301,6 @@ Navigator::Invalidate()
     mTelephony = nullptr;
   }
 
-  if (mVoicemail) {
-    mVoicemail->Shutdown();
-    mVoicemail = nullptr;
-  }
-
   if (mInputPortManager) {
     mInputPortManager = nullptr;
   }
@@ -1707,21 +1700,6 @@ Navigator::GetMozMobileConnections(ErrorResult& aRv)
 
 #endif // MOZ_B2G_RIL
 
-Voicemail*
-Navigator::GetMozVoicemail(ErrorResult& aRv)
-{
-  if (!mVoicemail) {
-    if (!mWindow) {
-      aRv.Throw(NS_ERROR_UNEXPECTED);
-      return nullptr;
-    }
-
-    mVoicemail = Voicemail::Create(mWindow, aRv);
-  }
-
-  return mVoicemail;
-}
-
 IccManager*
 Navigator::GetMozIccManager(ErrorResult& aRv)
 {

diff --git a/dom/base/Navigator.h b/dom/base/Navigator.h
@@ -86,7 +86,6 @@ class MobileConnectionArray;
 class PowerManager;
 class IccManager;
 class Telephony;
-class Voicemail;
 class InputPortManager;
 class DeviceStorageAreaListener;
 class Presentation;
@@ -225,7 +224,6 @@ class Navigator final : public nsIDOMNavigator
   IccManager* GetMozIccManager(ErrorResult& aRv);
   MobileMessageManager* GetMozMobileMessage();
   Telephony* GetMozTelephony(ErrorResult& aRv);
-  Voicemail* GetMozVoicemail(ErrorResult& aRv);
   InputPortManager* GetInputPortManager(ErrorResult& aRv);
   already_AddRefed<LegacyMozTCPSocket> MozTCPSocket();
   network::Connection* GetConnection(ErrorResult& aRv);
@@ -331,7 +329,6 @@ class Navigator final : public nsIDOMNavigator
   RefPtr<IccManager> mIccManager;
   RefPtr<MobileMessageManager> mMobileMessageManager;
   RefPtr<Telephony> mTelephony;
-  RefPtr<Voicemail> mVoicemail;
   RefPtr<InputPortManager> mInputPortManager;
   RefPtr<network::Connection> mConnection;
 #ifdef MOZ_B2G_RIL

diff --git a/dom/bindings/Bindings.conf b/dom/bindings/Bindings.conf
@@ -711,14 +711,6 @@ DOMInterfaces = {
     'nativeType': 'mozilla::dom::time::TimeManager',
 },
 
-'MozVoicemail': {
-    'nativeType': 'mozilla::dom::Voicemail',
-},
-
-'MozVoicemailStatus': {
-    'nativeType': 'mozilla::dom::VoicemailStatus',
-},
-
 'MutationObserver': {
     'nativeType': 'nsDOMMutationObserver',
 },

diff --git a/dom/events/test/test_all_synthetic_events.html b/dom/events/test/test_all_synthetic_events.html
@@ -364,10 +364,6 @@
                                                           return new MozStkCommandEvent(aName, aProps);
                                                        },
                                              },
-  MozVoicemailEvent:                         { create: function (aName, aProps) {
-                                                          return new MozVoicemailEvent(aName, aProps);
-                                                       },
-                                             },
   MozWifiConnectionInfoEvent:                { create: function (aName, aProps) {
                                                          return new MozWifiConnectionInfoEvent(aName, aProps);
                                                        },

diff --git a/dom/ipc/ContentChild.cpp b/dom/ipc/ContentChild.cpp
@@ -192,7 +192,6 @@
 #include "mozilla/dom/ipc/StructuredCloneData.h"
 #include "mozilla/dom/telephony/PTelephonyChild.h"
 #include "mozilla/dom/time/DateCacheCleaner.h"
-#include "mozilla/dom/voicemail/VoicemailIPCService.h"
 #include "mozilla/net/NeckoMessageUtils.h"
 #include "mozilla/widget/PuppetBidiKeyboard.h"
 #include "mozilla/RemoteSpellCheckEngineChild.h"
@@ -211,7 +210,6 @@ using namespace mozilla::dom::ipc;
 using namespace mozilla::dom::mobileconnection;
 using namespace mozilla::dom::mobilemessage;
 using namespace mozilla::dom::telephony;
-using namespace mozilla::dom::voicemail;
 using namespace mozilla::dom::workers;
 using namespace mozilla::media;
 using namespace mozilla::embedding;
@@ -2005,30 +2003,6 @@ ContentChild::DeallocPTelephonyChild(PTelephonyChild* aActor)
   return true;
 }
 
-PVoicemailChild*
-ContentChild::AllocPVoicemailChild()
-{
-  MOZ_CRASH("No one should be allocating PVoicemailChild actors");
-}
-
-PVoicemailChild*
-ContentChild::SendPVoicemailConstructor(PVoicemailChild* aActor)
-{
-  aActor = PContentChild::SendPVoicemailConstructor(aActor);
-  if (aActor) {
-    static_cast<VoicemailIPCService*>(aActor)->AddRef();
-  }
-
-  return aActor;
-}
-
-bool
-ContentChild::DeallocPVoicemailChild(PVoicemailChild* aActor)
-{
-  static_cast<VoicemailIPCService*>(aActor)->Release();
-  return true;
-}
-
 media::PMediaChild*
 ContentChild::AllocPMediaChild()
 {

diff --git a/dom/ipc/ContentChild.h b/dom/ipc/ContentChild.h
@@ -343,12 +343,6 @@ class ContentChild final : public PContentChild
 
   virtual bool DeallocPTelephonyChild(PTelephonyChild*) override;
 
-  virtual PVoicemailChild* AllocPVoicemailChild() override;
-
-  PVoicemailChild* SendPVoicemailConstructor(PVoicemailChild* aActor);
-
-  virtual bool DeallocPVoicemailChild(PVoicemailChild*) override;
-
   virtual PMediaChild* AllocPMediaChild() override;
 
   virtual bool DeallocPMediaChild(PMediaChild* aActor) override;

diff --git a/dom/ipc/ContentParent.cpp b/dom/ipc/ContentParent.cpp
@@ -70,7 +70,6 @@
 #include "mozilla/dom/quota/QuotaManagerService.h"
 #include "mozilla/dom/telephony/TelephonyParent.h"
 #include "mozilla/dom/time/DateCacheCleaner.h"
-#include "mozilla/dom/voicemail/VoicemailParent.h"
 #include "mozilla/embedding/printingui/PrintingParent.h"
 #include "mozilla/gfx/gfxVars.h"
 #include "mozilla/gfx/GPUProcessManager.h"
@@ -294,7 +293,6 @@ using namespace mozilla::dom::power;
 using namespace mozilla::dom::mobileconnection;
 using namespace mozilla::dom::mobilemessage;
 using namespace mozilla::dom::telephony;
-using namespace mozilla::dom::voicemail;
 using namespace mozilla::media;
 using namespace mozilla::embedding;
 using namespace mozilla::gfx;
@@ -3475,31 +3473,6 @@ ContentParent::DeallocPTelephonyParent(PTelephonyParent* aActor)
   return true;
 }
 
-PVoicemailParent*
-ContentParent::AllocPVoicemailParent()
-{
-  if (!AssertAppProcessPermission(this, "voicemail")) {
-    return nullptr;
-  }
-
-  VoicemailParent* actor = new VoicemailParent();
-  actor->AddRef();
-  return actor;
-}
-
-bool
-ContentParent::RecvPVoicemailConstructor(PVoicemailParent* aActor)
-{
-  return static_cast<VoicemailParent*>(aActor)->Init();
-}
-
-bool
-ContentParent::DeallocPVoicemailParent(PVoicemailParent* aActor)
-{
-  static_cast<VoicemailParent*>(aActor)->Release();
-  return true;
-}
-
 media::PMediaParent*
 ContentParent::AllocPMediaParent()
 {

diff --git a/dom/ipc/ContentParent.h b/dom/ipc/ContentParent.h
@@ -851,12 +851,6 @@ class ContentParent final : public PContentParent
 
   virtual bool DeallocPTelephonyParent(PTelephonyParent*) override;
 
-  virtual PVoicemailParent* AllocPVoicemailParent() override;
-
-  virtual bool RecvPVoicemailConstructor(PVoicemailParent* aActor) override;
-
-  virtual bool DeallocPVoicemailParent(PVoicemailParent* aActor) override;
-
   virtual PMediaParent* AllocPMediaParent() override;
 
   virtual bool DeallocPMediaParent(PMediaParent* aActor) override;

diff --git a/dom/ipc/PContent.ipdl b/dom/ipc/PContent.ipdl
@@ -47,7 +47,6 @@ include protocol PSpeechSynthesis;
 include protocol PStorage;
 include protocol PTelephony;
 include protocol PTestShell;
-include protocol PVoicemail;
 include protocol PJavaScript;
 include protocol PRemoteSpellcheckEngine;
 include protocol PWebBrowserPersistDocument;
@@ -377,7 +376,6 @@ nested(upto inside_cpow) sync protocol PContent
     manages PStorage;
     manages PTelephony;
     manages PTestShell;
-    manages PVoicemail;
     manages PJavaScript;
     manages PRemoteSpellcheckEngine;
     manages PWebBrowserPersistDocument;
@@ -827,8 +825,6 @@ parent:
 
     async PTelephony();
 
-    async PVoicemail();
-
     async PMedia();
 
     async PBluetooth();

diff --git a/dom/moz.build b/dom/moz.build
@@ -96,7 +96,6 @@ DIRS += [
     'smil',
     'telephony',
     'url',
-    'voicemail',
     'inputmethod',
     'webidl',
     'xbl',

diff --git a/dom/permission/tests/mochitest-ril.ini b/dom/permission/tests/mochitest-ril.ini
@@ -2,4 +2,3 @@
 [test_mobileconnection.html]
 [test_networkstats-manage.html]
 [test_telephony.html]
-[test_voicemail.html]
diff --git a/dom/permission/tests/test_voicemail.html b/dom/permission/tests/test_voicemail.html