Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(22)

Side by Side Diff: chrome/browser/ui/cocoa/website_settings/website_settings_bubble_controller.mm

Issue 2448943002: Refactor SecurityStateModel/Clients for simplicity and reusability. (Closed)
Patch Set: fix DEPS. Created 4 years, 1 month ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 // Copyright 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #import "chrome/browser/ui/cocoa/website_settings/website_settings_bubble_contro ller.h" 5 #import "chrome/browser/ui/cocoa/website_settings/website_settings_bubble_contro ller.h"
6 6
7 #import <AppKit/AppKit.h> 7 #import <AppKit/AppKit.h>
8 8
9 #include <cmath> 9 #include <cmath>
10 10
(...skipping 1118 matching lines...) Expand 10 before | Expand all | Expand 10 after
1129 void WebsiteSettingsUIBridge::set_bubble_controller( 1129 void WebsiteSettingsUIBridge::set_bubble_controller(
1130 WebsiteSettingsBubbleController* controller) { 1130 WebsiteSettingsBubbleController* controller) {
1131 bubble_controller_ = controller; 1131 bubble_controller_ = controller;
1132 } 1132 }
1133 1133
1134 void WebsiteSettingsUIBridge::Show( 1134 void WebsiteSettingsUIBridge::Show(
1135 gfx::NativeWindow parent, 1135 gfx::NativeWindow parent,
1136 Profile* profile, 1136 Profile* profile,
1137 content::WebContents* web_contents, 1137 content::WebContents* web_contents,
1138 const GURL& virtual_url, 1138 const GURL& virtual_url,
1139 const security_state::SecurityStateModel::SecurityInfo& security_info) { 1139 const security_state::SecurityInfo& security_info) {
1140 if (chrome::ToolkitViewsWebUIDialogsEnabled()) { 1140 if (chrome::ToolkitViewsWebUIDialogsEnabled()) {
1141 chrome::ShowWebsiteSettingsBubbleViewsAtPoint( 1141 chrome::ShowWebsiteSettingsBubbleViewsAtPoint(
1142 gfx::ScreenPointFromNSPoint(AnchorPointForWindow(parent)), profile, 1142 gfx::ScreenPointFromNSPoint(AnchorPointForWindow(parent)), profile,
1143 web_contents, virtual_url, security_info); 1143 web_contents, virtual_url, security_info);
1144 return; 1144 return;
1145 } 1145 }
1146 1146
1147 // Don't show the popup if it's already being shown. Since this method is 1147 // Don't show the popup if it's already being shown. Since this method is
1148 // called each time the location icon is clicked, each click toggles the popup 1148 // called each time the location icon is clicked, each click toggles the popup
1149 // in and out. 1149 // in and out.
(...skipping 49 matching lines...) Expand 10 before | Expand all | Expand 10 after
1199 void WebsiteSettingsUIBridge::SetPermissionInfo( 1199 void WebsiteSettingsUIBridge::SetPermissionInfo(
1200 const PermissionInfoList& permission_info_list, 1200 const PermissionInfoList& permission_info_list,
1201 ChosenObjectInfoList chosen_object_info_list) { 1201 ChosenObjectInfoList chosen_object_info_list) {
1202 [bubble_controller_ setPermissionInfo:permission_info_list 1202 [bubble_controller_ setPermissionInfo:permission_info_list
1203 andChosenObjects:std::move(chosen_object_info_list)]; 1203 andChosenObjects:std::move(chosen_object_info_list)];
1204 } 1204 }
1205 1205
1206 void WebsiteSettingsUIBridge::SetSelectedTab(TabId tab_id) { 1206 void WebsiteSettingsUIBridge::SetSelectedTab(TabId tab_id) {
1207 // TODO(lgarron): Remove this from the interface. (crbug.com/571533) 1207 // TODO(lgarron): Remove this from the interface. (crbug.com/571533)
1208 } 1208 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698