Bug 1415342 - Part 3 - Clear MediaMmanager deviceIds when clearing cookies. r?esawin draft
authorJan Henning <jh+bugzilla@buttercookie.de>
Thu, 12 Apr 2018 20:37:21 +0200
changeset 785165 c813ea1d75271bc1f6fb1008a63e442a196d2c0f
parent 785164 56d6e3ac981a5e4aa37acdc4db1ac9064d401a21
child 785166 16e4d14c91654f8ba4b76874edeb381f3b1fe915
push id107162
push usermozilla@buttercookie.de
push dateThu, 19 Apr 2018 19:01:06 +0000
reviewersesawin
bugs1415342
milestone61.0a1
Bug 1415342 - Part 3 - Clear MediaMmanager deviceIds when clearing cookies. r?esawin There is at least one reference to the MediaManagerService being used within mobile in conjunction with WebRTC, so we should do the same thing that desktop Firefox is doing here. MozReview-Commit-ID: 1z1cLapxK2F
mobile/android/modules/Sanitizer.jsm
--- a/mobile/android/modules/Sanitizer.jsm
+++ b/mobile/android/modules/Sanitizer.jsm
@@ -98,16 +98,24 @@ Sanitizer.prototype = {
       clear: function() {
         return new Promise(function(resolve, reject) {
           let refObj = {};
           TelemetryStopwatch.start("FX_SANITIZE_COOKIES_2", refObj);
 
           Services.cookies.removeAll();
 
           TelemetryStopwatch.finish("FX_SANITIZE_COOKIES_2", refObj);
+
+          // Clear deviceIds. Done asynchronously (returns before complete).
+          try {
+            let mediaMgr = Cc["@mozilla.org/mediaManagerService;1"]
+                             .getService(Ci.nsIMediaManagerService);
+            mediaMgr.sanitizeDeviceIds(0);
+          } catch (er) { }
+
           resolve();
         });
       },
 
       get canClear() {
         return true;
       }
     },