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

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: sync. 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 1156 matching lines...) Expand 10 before | Expand all | Expand 10 after
1167 void WebsiteSettingsUIBridge::set_bubble_controller( 1167 void WebsiteSettingsUIBridge::set_bubble_controller(
1168 WebsiteSettingsBubbleController* controller) { 1168 WebsiteSettingsBubbleController* controller) {
1169 bubble_controller_ = controller; 1169 bubble_controller_ = controller;
1170 } 1170 }
1171 1171
1172 void WebsiteSettingsUIBridge::Show( 1172 void WebsiteSettingsUIBridge::Show(
1173 gfx::NativeWindow parent, 1173 gfx::NativeWindow parent,
1174 Profile* profile, 1174 Profile* profile,
1175 content::WebContents* web_contents, 1175 content::WebContents* web_contents,
1176 const GURL& virtual_url, 1176 const GURL& virtual_url,
1177 const security_state::SecurityStateModel::SecurityInfo& security_info) { 1177 const security_state::SecurityInfo& security_info) {
1178 if (chrome::ToolkitViewsWebUIDialogsEnabled()) { 1178 if (chrome::ToolkitViewsWebUIDialogsEnabled()) {
1179 chrome::ShowWebsiteSettingsBubbleViewsAtPoint( 1179 chrome::ShowWebsiteSettingsBubbleViewsAtPoint(
1180 gfx::ScreenPointFromNSPoint(AnchorPointForWindow(parent)), profile, 1180 gfx::ScreenPointFromNSPoint(AnchorPointForWindow(parent)), profile,
1181 web_contents, virtual_url, security_info); 1181 web_contents, virtual_url, security_info);
1182 return; 1182 return;
1183 } 1183 }
1184 1184
1185 // Don't show the popup if it's already being shown. Since this method is 1185 // Don't show the popup if it's already being shown. Since this method is
1186 // called each time the location icon is clicked, each click toggles the popup 1186 // called each time the location icon is clicked, each click toggles the popup
1187 // in and out. 1187 // in and out.
(...skipping 49 matching lines...) Expand 10 before | Expand all | Expand 10 after
1237 void WebsiteSettingsUIBridge::SetPermissionInfo( 1237 void WebsiteSettingsUIBridge::SetPermissionInfo(
1238 const PermissionInfoList& permission_info_list, 1238 const PermissionInfoList& permission_info_list,
1239 ChosenObjectInfoList chosen_object_info_list) { 1239 ChosenObjectInfoList chosen_object_info_list) {
1240 [bubble_controller_ setPermissionInfo:permission_info_list 1240 [bubble_controller_ setPermissionInfo:permission_info_list
1241 andChosenObjects:std::move(chosen_object_info_list)]; 1241 andChosenObjects:std::move(chosen_object_info_list)];
1242 } 1242 }
1243 1243
1244 void WebsiteSettingsUIBridge::SetSelectedTab(TabId tab_id) { 1244 void WebsiteSettingsUIBridge::SetSelectedTab(TabId tab_id) {
1245 // TODO(lgarron): Remove this from the interface. (crbug.com/571533) 1245 // TODO(lgarron): Remove this from the interface. (crbug.com/571533)
1246 } 1246 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698