Implement a TODO in the media code: Change a "const MediaStreamRequest*" arg in various APIs to "co…

chrome/browser/media/media_stream_devices_controller.cc

 // Copyright (c) 2012 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 "chrome/browser/media/media_stream_devices_controller.h"
 
 #include "base/values.h"
 #include "chrome/browser/content_settings/content_settings_provider.h"
 #include "chrome/browser/content_settings/host_content_settings_map.h"
 #include "chrome/browser/extensions/api/tab_capture/tab_capture_registry.h"
@@ skipping 23 matching lines @@
   return !audio_devices.empty() || !video_devices.empty();
 };
 
 const char kAudioKey[] = "audio";
 const char kVideoKey[] = "video";
 
 }  // namespace
 
 MediaStreamDevicesController::MediaStreamDevicesController(
     Profile* profile,
-    const content::MediaStreamRequest* request,
+    const content::MediaStreamRequest& request,
     const content::MediaResponseCallback& callback)
     : profile_(profile),
-      request_(*request),
+      request_(request),
       callback_(callback),
-      has_audio_(content::IsAudioMediaType(request->audio_type) &&
+      has_audio_(content::IsAudioMediaType(request.audio_type) &&
                  !IsAudioDeviceBlockedByPolicy()),
-      has_video_(content::IsVideoMediaType(request->video_type) &&
+      has_video_(content::IsVideoMediaType(request.video_type) &&
                  !IsVideoDeviceBlockedByPolicy()) {
-  DCHECK(request);
 }
 
 MediaStreamDevicesController::~MediaStreamDevicesController() {}
 
 // static
 void MediaStreamDevicesController::RegisterUserPrefs(
     PrefServiceSyncable* prefs) {
   prefs->RegisterBooleanPref(prefs::kVideoCaptureAllowed,
                              true,
                              PrefServiceSyncable::UNSYNCABLE_PREF);
@@ skipping 203 matching lines @@
   }
   if (has_video_) {
     profile_->GetHostContentSettingsMap()->SetContentSetting(
         primary_pattern,
         ContentSettingsPattern::Wildcard(),
         CONTENT_SETTINGS_TYPE_MEDIASTREAM_CAMERA,
         std::string(),
         content_setting);
   }
 }