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

Unified Diff: chrome/browser/autofill/autofill_download_unittest.cc

Issue 8351027: Reduce visibility of methods in AutofillManager and AutofillDownload. (Closed) Base URL: http://git.chromium.org/chromium/src.git@master
Patch Set: Rebase harder Created 9 years, 2 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 side-by-side diff with in-line comments
Download patch
Index: chrome/browser/autofill/autofill_download_unittest.cc
diff --git a/chrome/browser/autofill/autofill_download_unittest.cc b/chrome/browser/autofill/autofill_download_unittest.cc
index 05230ddc7c6665e0605ec2ca7394b44ce5ca7604..b7ca41de17f8091a6eec83e62846d2210c8280ba 100644
--- a/chrome/browser/autofill/autofill_download_unittest.cc
+++ b/chrome/browser/autofill/autofill_download_unittest.cc
@@ -64,7 +64,7 @@ class AutofillDownloadTest : public AutofillDownloadManager::Observer,
public testing::Test {
public:
AutofillDownloadTest()
- : download_manager_(&profile_),
+ : download_manager_(&profile_, this),
io_thread_(BrowserThread::IO) {
}
@@ -73,13 +73,11 @@ class AutofillDownloadTest : public AutofillDownloadManager::Observer,
options.message_loop_type = MessageLoop::TYPE_IO;
io_thread_.StartWithOptions(options);
profile_.CreateRequestContext();
- download_manager_.SetObserver(this);
}
virtual void TearDown() {
profile_.ResetRequestContext();
io_thread_.Stop();
- download_manager_.SetObserver(NULL);
}
void LimitCache(size_t cache_size) {
@@ -87,14 +85,15 @@ class AutofillDownloadTest : public AutofillDownloadManager::Observer,
}
// AutofillDownloadManager::Observer implementation.
- virtual void OnLoadedServerPredictions(const std::string& response_xml) {
+ virtual void OnLoadedServerPredictions(
+ const std::string& response_xml) OVERRIDE {
ResponseData response;
response.response = response_xml;
response.type_of_response = QUERY_SUCCESSFULL;
responses_.push_back(response);
}
- virtual void OnUploadedPossibleFieldTypes() {
+ virtual void OnUploadedPossibleFieldTypes() OVERRIDE {
ResponseData response;
response.type_of_response = UPLOAD_SUCCESSFULL;
responses_.push_back(response);
@@ -103,7 +102,7 @@ class AutofillDownloadTest : public AutofillDownloadManager::Observer,
virtual void OnServerRequestError(
const std::string& form_signature,
AutofillDownloadManager::AutofillRequestType request_type,
- int http_error) {
+ int http_error) OVERRIDE {
ResponseData response;
response.signature = form_signature;
response.error = http_error;
@@ -125,8 +124,8 @@ class AutofillDownloadTest : public AutofillDownloadManager::Observer,
int error;
std::string signature;
std::string response;
- ResponseData() : type_of_response(REQUEST_QUERY_FAILED), error(0) {
- }
+
+ ResponseData() : type_of_response(REQUEST_QUERY_FAILED), error(0) {}
};
std::list<ResponseData> responses_;

Powered by Google App Engine
This is Rietveld 408576698