Add null check for MediaDevicesManager in MDDH destructor.

content/browser/renderer_host/media/media_devices_dispatcher_host.cc

 // Copyright 2016 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "content/browser/renderer_host/media/media_devices_dispatcher_host.h"
 
 #include <stddef.h>
 
 #include <utility>
 #include <vector>
@@ skipping 78 matching lines @@
       group_id_salt_(ResourceContext::CreateRandomMediaDeviceIDSalt()),
       media_stream_manager_(media_stream_manager),
       permission_checker_(base::MakeUnique<MediaDevicesPermissionChecker>()),
       weak_factory_(this) {
   DCHECK_CURRENTLY_ON(BrowserThread::IO);
 }
 
 MediaDevicesDispatcherHost::~MediaDevicesDispatcherHost() {
   DCHECK_CURRENTLY_ON(BrowserThread::IO);
   for (size_t i = 0; i < NUM_MEDIA_DEVICE_TYPES; ++i) {
-    if (!device_change_subscriptions_[i].empty()) {
+    if (media_stream_manager_->media_devices_manager() &&

[tommi (sloooow) - chröme, 2017/01/03 13:13:20]

would it make sense to do this check outside the for loop?

[Guido Urdaneta, 2017/01/03 13:27:17]

Done.

+        !device_change_subscriptions_[i].empty()) {
       media_stream_manager_->media_devices_manager()
           ->UnsubscribeDeviceChangeNotifications(
               static_cast<MediaDeviceType>(i), this);
     }
   }
 }
 
 void MediaDevicesDispatcherHost::EnumerateDevices(
     bool request_audio_input,
     bool request_video_input,
@@ skipping 177 matching lines @@
     for (const auto& device_info : enumeration[i]) {
       result[i].push_back(TranslateDeviceInfo(has_permissions[i],
                                               device_id_salt_, group_id_salt_,
                                               security_origin, device_info));
     }
   }
   client_callback.Run(result);
 }
 
 }  // namespace content