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

Side by Side Diff: chrome/browser/ui/webui/chromeos/login/signin_screen_handler.cc

Issue 10898002: Refactor BrowsingDataRemover creation for clarity. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Mac Created 8 years, 3 months 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 | Annotate | Revision Log
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 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/webui/chromeos/login/signin_screen_handler.h" 5 #include "chrome/browser/ui/webui/chromeos/login/signin_screen_handler.h"
6 6
7 #include "base/bind.h" 7 #include "base/bind.h"
8 #include "base/bind_helpers.h" 8 #include "base/bind_helpers.h"
9 #include "base/callback.h" 9 #include "base/callback.h"
10 #include "base/command_line.h" 10 #include "base/command_line.h"
(...skipping 1089 matching lines...) Expand 10 before | Expand all | Expand 10 after
1100 base::Bind(&SigninScreenHandler::OnDnsCleared, 1100 base::Bind(&SigninScreenHandler::OnDnsCleared,
1101 weak_factory_.GetWeakPtr())); 1101 weak_factory_.GetWeakPtr()));
1102 dns_clear_task_running_ = true; 1102 dns_clear_task_running_ = true;
1103 } 1103 }
1104 1104
1105 void SigninScreenHandler::StartClearingCookies() { 1105 void SigninScreenHandler::StartClearingCookies() {
1106 cookies_cleared_ = false; 1106 cookies_cleared_ = false;
1107 if (cookie_remover_) 1107 if (cookie_remover_)
1108 cookie_remover_->RemoveObserver(this); 1108 cookie_remover_->RemoveObserver(this);
1109 1109
1110 cookie_remover_ = new BrowsingDataRemover( 1110 cookie_remover_ = BrowsingDataRemover::CreateForUnboundedRange(
1111 Profile::FromWebUI(web_ui()), 1111 Profile::FromWebUI(web_ui()));
1112 BrowsingDataRemover::EVERYTHING,
1113 base::Time::Now());
1114 cookie_remover_->AddObserver(this); 1112 cookie_remover_->AddObserver(this);
1115 cookie_remover_->Remove(BrowsingDataRemover::REMOVE_SITE_DATA, 1113 cookie_remover_->Remove(BrowsingDataRemover::REMOVE_SITE_DATA,
1116 BrowsingDataHelper::UNPROTECTED_WEB); 1114 BrowsingDataHelper::UNPROTECTED_WEB);
1117 } 1115 }
1118 1116
1119 void SigninScreenHandler::MaybePreloadAuthExtension() { 1117 void SigninScreenHandler::MaybePreloadAuthExtension() {
1120 // Fetching of the extension is not started before account picker page is 1118 // Fetching of the extension is not started before account picker page is
1121 // loaded because it can affect the loading speed. Also if |cookie_remover_| 1119 // loaded because it can affect the loading speed. Also if |cookie_remover_|
1122 // or |dns_clear_task_running_| then auth extension showing has already been 1120 // or |dns_clear_task_running_| then auth extension showing has already been
1123 // initiated and preloading is senseless. 1121 // initiated and preloading is senseless.
(...skipping 28 matching lines...) Expand all
1152 } 1150 }
1153 for (UserList::const_iterator it = users.begin(); it != users.end(); ++it) { 1151 for (UserList::const_iterator it = users.begin(); it != users.end(); ++it) {
1154 if (!cros_settings->FindEmailInList(kAccountsPrefUsers, (*it)->email())) { 1152 if (!cros_settings->FindEmailInList(kAccountsPrefUsers, (*it)->email())) {
1155 return false; 1153 return false;
1156 } 1154 }
1157 } 1155 }
1158 return true; 1156 return true;
1159 } 1157 }
1160 1158
1161 } // namespace chromeos 1159 } // namespace chromeos
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698