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

Unified Diff: chrome/browser/component_updater/component_updater_service.cc

Issue 10386063: Move URLFetcherDelegate to net/ and split URLFetcher between net/ and content/ (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: sync to head, fix win component build Created 8 years, 7 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/component_updater/component_updater_service.cc
diff --git a/chrome/browser/component_updater/component_updater_service.cc b/chrome/browser/component_updater/component_updater_service.cc
index b24d100597282e3c804e96643b920926b6a2db8a..99df4d6a80a4732d465347363472068731e837e0 100644
--- a/chrome/browser/component_updater/component_updater_service.cc
+++ b/chrome/browser/component_updater/component_updater_service.cc
@@ -28,8 +28,8 @@
#include "content/public/browser/notification_service.h"
#include "content/public/browser/utility_process_host.h"
#include "content/public/browser/utility_process_host_client.h"
-#include "content/public/common/url_fetcher_delegate.h"
#include "content/public/common/url_fetcher.h"
+#include "content/public/common/url_fetcher_delegate.h"
#include "googleurl/src/gurl.h"
#include "net/base/escape.h"
#include "net/base/load_flags.h"
@@ -117,7 +117,7 @@ class DelegateWithContext : public content::URLFetcherDelegate {
DelegateWithContext(Del* delegate, Ctx* context)
: delegate_(delegate), context_(context) {}
- virtual void OnURLFetchComplete(const content::URLFetcher* source) OVERRIDE {
+ virtual void OnURLFetchComplete(const net::URLFetcher* source) OVERRIDE {
delegate_->OnURLFetchComplete(source, context_);
delete this;
}
@@ -152,7 +152,7 @@ void StartFetch(content::URLFetcher* fetcher,
}
// Returs true if the url request of |fetcher| was succesful.
-bool FetchSuccess(const content::URLFetcher& fetcher) {
+bool FetchSuccess(const net::URLFetcher& fetcher) {
return (fetcher.GetStatus().status() == net::URLRequestStatus::SUCCESS) &&
(fetcher.GetResponseCode() == 200);
}
@@ -289,10 +289,10 @@ class CrxUpdateService : public ComponentUpdateService {
CRXContext() : installer(NULL) {}
};
- void OnURLFetchComplete(const content::URLFetcher* source,
+ void OnURLFetchComplete(const net::URLFetcher* source,
UpdateContext* context);
- void OnURLFetchComplete(const content::URLFetcher* source,
+ void OnURLFetchComplete(const net::URLFetcher* source,
CRXContext* context);
private:
@@ -574,7 +574,7 @@ void CrxUpdateService::ProcessPendingItems() {
// Caled when we got a response from the update server. It consists of an xml
// document following the omaha update scheme.
-void CrxUpdateService::OnURLFetchComplete(const content::URLFetcher* source,
+void CrxUpdateService::OnURLFetchComplete(const net::URLFetcher* source,
UpdateContext* context) {
DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
if (FetchSuccess(*source)) {
@@ -679,7 +679,7 @@ void CrxUpdateService::OnParseUpdateManifestFailed(
// Called when the CRX package has been downloaded to a temporary location.
// Here we fire the notifications and schedule the component-specific installer
// to be called in the file thread.
-void CrxUpdateService::OnURLFetchComplete(const content::URLFetcher* source,
+void CrxUpdateService::OnURLFetchComplete(const net::URLFetcher* source,
CRXContext* context) {
DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
base::PlatformFileError error_code;
« no previous file with comments | « chrome/browser/chromeos/imageburner/burn_manager.cc ('k') | chrome/browser/extensions/app_notify_channel_setup.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698