Make PermissionRequest::GetIconId return different types

chrome/browser/media/webrtc/media_stream_capture_indicator.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/webrtc/media_stream_capture_indicator.h"
 
 #include <stddef.h>
 
 #include <memory>
 #include <string>
@@ skipping 338 matching lines @@
   DCHECK(!image.isNull());
   DCHECK(!tool_tip.empty());
 
   status_icon_ = status_tray->CreateStatusIcon(
       StatusTray::MEDIA_STREAM_CAPTURE_ICON, image, tool_tip);
 }
 
 void MediaStreamCaptureIndicator::EnsureStatusTrayIconResources() {
   DCHECK_CURRENTLY_ON(BrowserThread::UI);
 
+  // TODO(estade): these should use vector icons.

[Peter Kasting, 2016/10/06 07:10:29]

Nit: crbug.com/505953?  Or is there another one?

[Evan Stade, 2016/10/06 15:33:50]

there is not another one filed. Dunno if you wanted me to link to that one, but
it doesn't seem all that useful (easy enough to find resources that still need
removing by looking in chrome/app/theme)

   if (!mic_image_) {
     mic_image_ = ResourceBundle::GetSharedInstance().GetImageSkiaNamed(
         IDR_INFOBAR_MEDIA_STREAM_MIC);
   }
   if (!camera_image_) {
     camera_image_ = ResourceBundle::GetSharedInstance().GetImageSkiaNamed(
         IDR_INFOBAR_MEDIA_STREAM_CAMERA);
   }
   DCHECK(mic_image_);
   DCHECK(camera_image_);
@@ skipping 86 matching lines @@
   } else if (audio && !video) {
     message_id = IDS_MEDIA_STREAM_STATUS_TRAY_TEXT_AUDIO_ONLY;
     *image = *mic_image_;
   } else if (!audio && video) {
     message_id = IDS_MEDIA_STREAM_STATUS_TRAY_TEXT_VIDEO_ONLY;
     *image = *camera_image_;
   }
 
   *tool_tip = l10n_util::GetStringUTF16(message_id);
 }