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

Side by Side Diff: chrome/browser/ui/views/website_settings/website_settings_popup_view_unittest.cc

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 2015 The Chromium Authors. All rights reserved. 1 // Copyright 2015 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 #include "chrome/browser/ui/views/website_settings/website_settings_popup_view.h " 5 #include "chrome/browser/ui/views/website_settings/website_settings_popup_view.h "
6 6
7 #include "base/macros.h" 7 #include "base/macros.h"
8 #include "base/strings/utf_string_conversions.h" 8 #include "base/strings/utf_string_conversions.h"
9 #include "chrome/browser/ui/exclusive_access/exclusive_access_manager.h" 9 #include "chrome/browser/ui/exclusive_access/exclusive_access_manager.h"
10 #include "chrome/browser/ui/views/website_settings/chosen_object_row.h" 10 #include "chrome/browser/ui/views/website_settings/chosen_object_row.h"
(...skipping 27 matching lines...) Expand all
38 parent_(parent), 38 parent_(parent),
39 profile_(profile), 39 profile_(profile),
40 web_contents_(web_contents) { 40 web_contents_(web_contents) {
41 CreateView(); 41 CreateView();
42 } 42 }
43 43
44 void CreateView() { 44 void CreateView() {
45 if (view_) 45 if (view_)
46 view_->GetWidget()->CloseNow(); 46 view_->GetWidget()->CloseNow();
47 47
48 security_state::SecurityStateModel::SecurityInfo security_info; 48 security_state::SecurityInfo security_info;
49 views::View* anchor_view = nullptr; 49 views::View* anchor_view = nullptr;
50 view_ = 50 view_ =
51 new WebsiteSettingsPopupView(anchor_view, parent_, profile_, 51 new WebsiteSettingsPopupView(anchor_view, parent_, profile_,
52 web_contents_, GURL(kUrl), security_info); 52 web_contents_, GURL(kUrl), security_info);
53 } 53 }
54 54
55 WebsiteSettingsPopupView* view() { return view_; } 55 WebsiteSettingsPopupView* view() { return view_; }
56 views::View* permissions_view() { return view_->permissions_view_; } 56 views::View* permissions_view() { return view_->permissions_view_; }
57 57
58 PermissionSelectorRow* GetPermissionSelectorAt(int index) { 58 PermissionSelectorRow* GetPermissionSelectorAt(int index) {
(...skipping 192 matching lines...) Expand 10 before | Expand all | Expand 10 after
251 251
252 const ui::MouseEvent event(ui::ET_MOUSE_PRESSED, gfx::Point(), gfx::Point(), 252 const ui::MouseEvent event(ui::ET_MOUSE_PRESSED, gfx::Point(), gfx::Point(),
253 ui::EventTimeForNow(), 0, 0); 253 ui::EventTimeForNow(), 0, 0);
254 views::ButtonListener* button_listener = 254 views::ButtonListener* button_listener =
255 static_cast<views::ButtonListener*>(object_view); 255 static_cast<views::ButtonListener*>(object_view);
256 button_listener->ButtonPressed(button, event); 256 button_listener->ButtonPressed(button, event);
257 api_->SetPermissionInfo(list); 257 api_->SetPermissionInfo(list);
258 EXPECT_EQ(kExpectedChildren, api_->permissions_view()->child_count()); 258 EXPECT_EQ(kExpectedChildren, api_->permissions_view()->child_count());
259 EXPECT_FALSE(store->HasDevicePermission(origin, origin, device)); 259 EXPECT_FALSE(store->HasDevicePermission(origin, origin, device));
260 } 260 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698