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

Unified Diff: chrome/browser/chromeos/login/ui/simple_web_view_dialog.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 side-by-side diff with in-line comments
Download patch
Index: chrome/browser/chromeos/login/ui/simple_web_view_dialog.cc
diff --git a/chrome/browser/chromeos/login/ui/simple_web_view_dialog.cc b/chrome/browser/chromeos/login/ui/simple_web_view_dialog.cc
index 2a0af7c27975b1e4e7b1c35deacc0fbaf476d505..2063a7110d908f0f267f35f20c506776adc7a191 100644
--- a/chrome/browser/chromeos/login/ui/simple_web_view_dialog.cc
+++ b/chrome/browser/chromeos/login/ui/simple_web_view_dialog.cc
@@ -15,7 +15,7 @@
#include "chrome/browser/command_updater.h"
#include "chrome/browser/password_manager/chrome_password_manager_client.h"
#include "chrome/browser/profiles/profile.h"
-#include "chrome/browser/ssl/chrome_security_state_model_client.h"
+#include "chrome/browser/ssl/security_state_tab_helper.h"
#include "chrome/browser/ui/autofill/chrome_autofill_client.h"
#include "chrome/browser/ui/browser.h"
#include "chrome/browser/ui/content_settings/content_setting_bubble_model_delegate.h"
@@ -24,7 +24,6 @@
#include "chrome/grit/generated_resources.h"
#include "chrome/grit/theme_resources.h"
#include "components/password_manager/core/browser/password_manager.h"
-#include "components/security_state/security_state_model.h"
#include "components/strings/grit/components_strings.h"
#include "components/toolbar/toolbar_model_impl.h"
#include "content/public/browser/navigation_controller.h"
@@ -165,8 +164,7 @@ void SimpleWebViewDialog::StartLoad(const GURL& url) {
void SimpleWebViewDialog::Init() {
// Create the security state model that the toolbar model needs.
if (web_view_->GetWebContents())
- ChromeSecurityStateModelClient::CreateForWebContents(
- web_view_->GetWebContents());
+ SecurityStateTabHelper::CreateForWebContents(web_view_->GetWebContents());
toolbar_model_.reset(
new ToolbarModelImpl(this, content::kMaxURLDisplayChars));
@@ -300,7 +298,7 @@ SimpleWebViewDialog::GetContentSettingBubbleModelDelegate() {
void SimpleWebViewDialog::ShowWebsiteSettings(
content::WebContents* web_contents,
const GURL& virtual_url,
- const security_state::SecurityStateModel::SecurityInfo& security_info) {
+ const security_state::SecurityInfo& security_info) {
NOTIMPLEMENTED();
// TODO (markusheintz@): implement this
}

Powered by Google App Engine
This is Rietveld 408576698