summaryrefslogtreecommitdiffstats
path: root/chrome/browser/safe_browsing
diff options
context:
space:
mode:
Diffstat (limited to 'chrome/browser/safe_browsing')
-rw-r--r--chrome/browser/safe_browsing/protocol_manager.cc31
-rw-r--r--chrome/browser/safe_browsing/protocol_manager.h3
2 files changed, 5 insertions, 29 deletions
diff --git a/chrome/browser/safe_browsing/protocol_manager.cc b/chrome/browser/safe_browsing/protocol_manager.cc
index c1ed1b4..e66afc5 100644
--- a/chrome/browser/safe_browsing/protocol_manager.cc
+++ b/chrome/browser/safe_browsing/protocol_manager.cc
@@ -49,22 +49,6 @@ static const int kSbClientMinorVersion = 0;
static const int kSbMaxBackOff = 8;
-// Periodic update task --------------------------------------------------------
-class SafeBrowsingProtocolUpdateTask : public Task {
- public:
- explicit SafeBrowsingProtocolUpdateTask(SafeBrowsingProtocolManager* manager)
- : manager_(manager) {
- }
-
- void Run() {
- manager_->GetNextUpdate();
- }
-
- private:
- SafeBrowsingProtocolManager* manager_;
-};
-
-
// SafeBrowsingProtocolManager implementation ----------------------------------
SafeBrowsingProtocolManager::SafeBrowsingProtocolManager(
@@ -93,9 +77,6 @@ SafeBrowsingProtocolManager::SafeBrowsingProtocolManager(
}
SafeBrowsingProtocolManager::~SafeBrowsingProtocolManager() {
- if (update_timer_.get())
- MessageLoop::current()->timer_manager()->StopTimer(update_timer_.get());
-
// Delete in-progress SafeBrowsing requests.
STLDeleteContainerPairFirstPointers(hash_requests_.begin(),
hash_requests_.end());
@@ -399,17 +380,13 @@ void SafeBrowsingProtocolManager::Initialize() {
void SafeBrowsingProtocolManager::ScheduleNextUpdate(bool back_off) {
DCHECK(next_update_sec_ > 0);
- if (!update_task_.get())
- update_task_.reset(new SafeBrowsingProtocolUpdateTask(this));
-
- // Unschedule any current timer & task.
- TimerManager* tm = MessageLoop::current()->timer_manager();
- if (update_timer_.get())
- tm->StopTimer(update_timer_.get());
+ // Unschedule any current timer.
+ update_timer_.Stop();
// Reschedule with the new update.
const int next_update = GetNextUpdateTime(back_off);
- update_timer_.reset(tm->StartTimer(next_update, update_task_.get(), false));
+ update_timer_.Start(TimeDelta::FromMilliseconds(next_update), this,
+ &SafeBrowsingProtocolManager::GetNextUpdate);
}
// According to section 5 of the SafeBrowsing protocol specification, we must
diff --git a/chrome/browser/safe_browsing/protocol_manager.h b/chrome/browser/safe_browsing/protocol_manager.h
index 1b12d5c6..c7276c6 100644
--- a/chrome/browser/safe_browsing/protocol_manager.h
+++ b/chrome/browser/safe_browsing/protocol_manager.h
@@ -159,8 +159,7 @@ class SafeBrowsingProtocolManager : public URLFetcher::Delegate {
// For managing the next earliest time to query the SafeBrowsing servers for
// updates.
int next_update_sec_;
- scoped_ptr<Task> update_task_;
- scoped_ptr<Timer> update_timer_;
+ base::OneShotTimer<SafeBrowsingProtocolManager> update_timer_;
// All chunk requests that need to be made, along with their MAC.
std::deque<ChunkUrl> chunk_request_urls_;