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

Side by Side Diff: components/webdata/common/web_database_service.cc

Issue 17760007: components/webdata: Cleanup callback usage in WebDatabaseService. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: disallow Created 7 years, 6 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
« no previous file with comments | « components/webdata/common/web_database_service.h ('k') | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright 2013 The Chromium Authors. All rights reserved. 1 // Copyright 2013 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 "components/webdata/common/web_database_service.h" 5 #include "components/webdata/common/web_database_service.h"
6 6
7 #include "base/bind.h" 7 #include "base/bind.h"
8 #include "base/location.h" 8 #include "base/location.h"
9 #include "components/webdata/common/web_data_request_manager.h" 9 #include "components/webdata/common/web_data_request_manager.h"
10 #include "components/webdata/common/web_data_results.h" 10 #include "components/webdata/common/web_data_results.h"
(...skipping 132 matching lines...) Expand 10 before | Expand all | Expand 10 after
143 return handle; 143 return handle;
144 } 144 }
145 145
146 void WebDatabaseService::CancelRequest(WebDataServiceBase::Handle h) { 146 void WebDatabaseService::CancelRequest(WebDataServiceBase::Handle h) {
147 if (!wds_backend_.get()) 147 if (!wds_backend_.get())
148 return; 148 return;
149 wds_backend_->request_manager()->CancelRequest(h); 149 wds_backend_->request_manager()->CancelRequest(h);
150 } 150 }
151 151
152 void WebDatabaseService::RegisterDBLoadedCallback( 152 void WebDatabaseService::RegisterDBLoadedCallback(
153 const base::Callback<void(void)>& callback) { 153 const DBLoadedCallback& callback) {
154 loaded_callbacks_.push_back(callback); 154 loaded_callbacks_.push_back(callback);
155 } 155 }
156 156
157 void WebDatabaseService::RegisterDBErrorCallback( 157 void WebDatabaseService::RegisterDBErrorCallback(
158 const base::Callback<void(sql::InitStatus)>& callback) { 158 const DBLoadErrorCallback& callback) {
159 error_callbacks_.push_back(callback); 159 error_callbacks_.push_back(callback);
160 } 160 }
161 161
162 void WebDatabaseService::OnDatabaseLoadDone(sql::InitStatus status) { 162 void WebDatabaseService::OnDatabaseLoadDone(sql::InitStatus status) {
163 if (status == sql::INIT_OK) { 163 if (status == sql::INIT_OK) {
164 db_loaded_ = true; 164 db_loaded_ = true;
165 165
166 for (size_t i = 0; i < loaded_callbacks_.size(); i++) { 166 for (size_t i = 0; i < loaded_callbacks_.size(); i++) {
167 if (!loaded_callbacks_[i].is_null()) 167 if (!loaded_callbacks_[i].is_null())
168 loaded_callbacks_[i].Run(); 168 loaded_callbacks_[i].Run();
169 } 169 }
170 170
171 loaded_callbacks_.clear(); 171 loaded_callbacks_.clear();
172 } else { 172 } else {
173 // Notify that the database load failed. 173 // Notify that the database load failed.
174 for (size_t i = 0; i < error_callbacks_.size(); i++) { 174 for (size_t i = 0; i < error_callbacks_.size(); i++) {
175 if (!error_callbacks_[i].is_null()) 175 if (!error_callbacks_[i].is_null())
176 error_callbacks_[i].Run(status); 176 error_callbacks_[i].Run(status);
177 } 177 }
178 178
179 error_callbacks_.clear(); 179 error_callbacks_.clear();
180 } 180 }
181 } 181 }
OLDNEW
« no previous file with comments | « components/webdata/common/web_database_service.h ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698