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

Unified Diff: chrome/browser/component_updater/background_downloader_win.h

Issue 105853002: Implement a background downloader using BITS in Windows Chrome. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: trybots and Josh's feedback. Created 7 years 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/background_downloader_win.h
diff --git a/chrome/browser/component_updater/background_downloader_win.h b/chrome/browser/component_updater/background_downloader_win.h
new file mode 100644
index 0000000000000000000000000000000000000000..8e77c9f9a9438409b03561ddf1a891041616ab57
--- /dev/null
+++ b/chrome/browser/component_updater/background_downloader_win.h
@@ -0,0 +1,74 @@
+// Copyright 2013 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#ifndef CHROME_BROWSER_COMPONENT_UPDATER_BACKGROUND_DOWNLOADER_WIN_H_
+#define CHROME_BROWSER_COMPONENT_UPDATER_BACKGROUND_DOWNLOADER_WIN_H_
+
+#include "chrome/browser/component_updater/crx_downloader.h"
+#include <windows.h>
cpu_(ooo_6.6-7.5) 2013/12/05 19:32:43 I think you need a space between 8 and 9, I am not
Sorin Jianu 2013/12/05 22:34:36 Done.
+#include <bits.h>
+#include "base/strings/string16.h"
+#include "base/timer/timer.h"
+#include "base/win/scoped_comptr.h"
+
+namespace component_updater {
+
+// Implements a downloader in terms of the BITS service. The public interface
+// of this class and the CrxDownloader overrides are expected to be called
+// from the UI thread. The rest of the class code runs on the FILE thread in
+// a single threaded apartment.
+class BackgroundDownloader : public CrxDownloader {
+ protected:
+ friend class CrxDownloader;
+ BackgroundDownloader(scoped_ptr<CrxDownloader> successor,
+ net::URLRequestContextGetter* context_getter,
+ scoped_refptr<base::SequencedTaskRunner> task_runner,
+ const DownloadCallback& download_callback);
+ virtual ~BackgroundDownloader();
+
+ private:
+ // Overrides for CrxDownloader.
+ virtual void DoStartDownload(const GURL& url) OVERRIDE;
+
+ // Called asynchronously on the FILE thread at different stages during
+ // the download. |OnDownloading| can be called multiple times.
+ void BeginDownload(const GURL& url);
+ void OnDownloading();
+ void EndDownload(HRESULT hr);
+
+ // Handles the job state transitions to a final state.
+ void OnStateTransferred();
+ void OnStateError();
+ void OnStateCancelled();
+ void OnStateAcknowledged();
+
+ HRESULT QueueBitsJob(const GURL& url);
+ HRESULT CreateOrOpenJob(const GURL& url);
+ HRESULT InitializeNewJob(const GURL& url);
+
+ HRESULT InstallJobObserver();
+ HRESULT RemoveJobObserver();
+
+ static HRESULT CleanupStaleJobs(
+ base::win::ScopedComPtr<IBackgroundCopyManager> bits_manager);
+
+ net::URLRequestContextGetter* context_getter_;
+ scoped_refptr<base::SequencedTaskRunner> task_runner_;
+
+ scoped_ptr<base::OneShotTimer<BackgroundDownloader> > timer_;
+
+ base::win::ScopedComPtr<IBackgroundCopyManager> bits_manager_;
+
+ base::win::ScopedComPtr<IBackgroundCopyJob> job_;
+ base::win::ScopedComPtr<IBackgroundCopyCallback> job_observer_;
+
+ bool is_completed_;
+
+ DISALLOW_COPY_AND_ASSIGN(BackgroundDownloader);
+};
+
+} // namespace component_updater
+
+#endif // CHROME_BROWSER_COMPONENT_UPDATER_BACKGROUND_DOWNLOADER_WIN_H_
+

Powered by Google App Engine
This is Rietveld 408576698