Remove PermissionInfoBarDelegate from desktop builds.

chrome/browser/permissions/permission_queue_controller.cc

 // Copyright 2013 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/permissions/permission_queue_controller.h"
 
 #include "chrome/browser/chrome_notification_types.h"
 #include "chrome/browser/content_settings/host_content_settings_map_factory.h"
-#include "chrome/browser/geolocation/geolocation_infobar_delegate_android.h"
 #include "chrome/browser/infobars/infobar_service.h"
-#include "chrome/browser/media/midi_permission_infobar_delegate_android.h"
-#include "chrome/browser/media/protected_media_identifier_infobar_delegate_android.h"
-#include "chrome/browser/notifications/notification_permission_infobar_delegate.h"
 #include "chrome/browser/permissions/permission_infobar_delegate.h"
 #include "chrome/browser/permissions/permission_request.h"
 #include "chrome/browser/permissions/permission_request_id.h"
 #include "chrome/browser/permissions/permission_uma_util.h"
 #include "chrome/browser/profiles/profile.h"
 #include "chrome/browser/tab_contents/tab_util.h"
 #include "components/content_settings/core/browser/host_content_settings_map.h"
 #include "components/content_settings/core/common/content_settings.h"
 #include "components/infobars/core/infobar.h"
 #include "content/public/browser/browser_thread.h"
@@ skipping 92 matching lines @@
 
 void PermissionQueueController::PendingInfobarRequest::CreateInfoBar(
     PermissionQueueController* controller) {
   // Controller can be Unretained because the lifetime of the infobar
   // is tied to that of the queue controller. Before QueueController
   // is destroyed, all requests will be cancelled and so all delegates
   // will be destroyed.
   PermissionInfoBarDelegate::PermissionSetCallback callback = base::Bind(
       &PermissionQueueController::OnPermissionSet, base::Unretained(controller),
       id_, requesting_frame_, embedder_, user_gesture_);
-  switch (type_) {
-    case content::PermissionType::GEOLOCATION:
-      infobar_ = GeolocationInfoBarDelegateAndroid::Create(
-          GetInfoBarService(id_), requesting_frame_, user_gesture_, profile_,
-          callback);
-      break;
-#if defined(ENABLE_NOTIFICATIONS)
-    case content::PermissionType::NOTIFICATIONS:
-    case content::PermissionType::PUSH_MESSAGING:
-      infobar_ = NotificationPermissionInfoBarDelegate::Create(
-          GetInfoBarService(id_), requesting_frame_, user_gesture_, profile_,
-          callback);
-      break;
-#endif  // ENABLE_NOTIFICATIONS
-    case content::PermissionType::MIDI_SYSEX:
-      infobar_ = MidiPermissionInfoBarDelegateAndroid::Create(
-          GetInfoBarService(id_), requesting_frame_, user_gesture_, profile_,
-          callback);
-      break;
-    case content::PermissionType::PROTECTED_MEDIA_IDENTIFIER:
-      infobar_ = ProtectedMediaIdentifierInfoBarDelegateAndroid::Create(
-          GetInfoBarService(id_), requesting_frame_, user_gesture_, profile_,
-          callback);
-      break;
-    default:
-      NOTREACHED();
-      break;
-  }
+
+  DCHECK(type_ == content::PermissionType::GEOLOCATION ||
+         type_ == content::PermissionType::NOTIFICATIONS ||
+         type_ == content::PermissionType::PUSH_MESSAGING ||
+         type_ == content::PermissionType::MIDI_SYSEX ||
+         type_ == content::PermissionType::PROTECTED_MEDIA_IDENTIFIER);

[raymes, 2016/10/24 01:00:36]

nit: I'm not sure this is necessary as we should hit a NOTREACHED in
PermissionInfoBarDelegate::Create

[dominickn, 2016/10/24 01:42:00]

Done.

+  infobar_ = PermissionInfoBarDelegate::Create(type_, GetInfoBarService(id_),
+                                               requesting_frame_, user_gesture_,
+                                               profile_, callback);
 }
 
 PermissionQueueController::PermissionQueueController(
     Profile* profile,
     content::PermissionType permission_type,
     ContentSettingsType content_settings_type)
     : profile_(profile),
       permission_type_(permission_type),
       content_settings_type_(content_settings_type),
       in_shutdown_(false) {}
@@ skipping 249 matching lines @@
   }
 
   ContentSetting content_setting =
       (decision == GRANTED) ? CONTENT_SETTING_ALLOW : CONTENT_SETTING_BLOCK;
 
   HostContentSettingsMapFactory::GetForProfile(profile_)
       ->SetContentSettingDefaultScope(
           requesting_frame.GetOrigin(), embedder.GetOrigin(),
           content_settings_type_, std::string(), content_setting);
 }