Base: Make TimeDelta constructors deal with overflow.

chrome/browser/sync/sync_stopped_reporter.cc

 // Copyright 2015 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.
 
 #include "chrome/browser/sync/sync_stopped_reporter.h"
 
 #include "base/location.h"
 #include "base/logging.h"
 #include "base/single_thread_task_runner.h"
 #include "base/strings/stringprintf.h"
 #include "base/thread_task_runner_handle.h"
 #include "base/timer/timer.h"
 #include "chrome/browser/profiles/profile.h"
 #include "chrome/browser/sync/glue/local_device_info_provider_impl.h"
 #include "chrome/common/chrome_version_info.h"
 #include "net/base/load_flags.h"
 #include "net/http/http_status_code.h"
 #include "net/url_request/url_fetcher.h"
 #include "net/url_request/url_request_context_getter.h"
 #include "sync/protocol/sync.pb.h"
 
 namespace {
 
 const char kEventEndpoint[] = "event";
 
 // The request is tiny, so even on poor connections 10 seconds should be
 // plenty of time. Since sync is off when this request is started, we don't
 // want anything sync-related hanging around for very long from a human
 // perspective either. This seems like a good compromise.
-const base::TimeDelta kRequestTimeout = base::TimeDelta::FromSeconds(10);
+const int kRequestTimeoutSeconds = 10;
 
 std::string GetUserAgent() {
   chrome::VersionInfo version_info;
   return browser_sync::LocalDeviceInfoProviderImpl::MakeUserAgentForSyncApi(
       version_info);
 }
 
 }  // namespace
 
 namespace browser_sync {
@@ skipping 37 matching lines @@
       access_token.c_str()));
   fetcher_->AddExtraRequestHeader(base::StringPrintf(
       "%s: %s", net::HttpRequestHeaders::kUserAgent, user_agent_.c_str()));
   fetcher_->SetRequestContext(request_context_.get());
   fetcher_->SetUploadData("application/octet-stream", msg);
   fetcher_->SetLoadFlags(net::LOAD_BYPASS_CACHE |
                          net::LOAD_DISABLE_CACHE |
                          net::LOAD_DO_NOT_SAVE_COOKIES |
                          net::LOAD_DO_NOT_SEND_COOKIES);
   fetcher_->Start();
-  timer_.Start(FROM_HERE, kRequestTimeout, this,
-               &SyncStoppedReporter::OnTimeout);
+  timer_.Start(FROM_HERE, base::TimeDelta::FromSeconds(kRequestTimeoutSeconds),
+               this, &SyncStoppedReporter::OnTimeout);
 }
 
 void SyncStoppedReporter::OnURLFetchComplete(const net::URLFetcher* source) {
   Result result = source->GetResponseCode() == net::HTTP_OK
       ? RESULT_SUCCESS : RESULT_ERROR;
   fetcher_.reset();
   timer_.Stop();
   if (!callback_.is_null()) {
     base::ThreadTaskRunnerHandle::Get()->PostTask(
         FROM_HERE, base::Bind(callback_, result));
@@ skipping 19 matching lines @@
   replacements.SetPathStr(path);
   return sync_service_url.ReplaceComponents(replacements);
 }
 
 void SyncStoppedReporter::SetTimerTaskRunnerForTest(
     const scoped_refptr<base::SingleThreadTaskRunner>& task_runner) {
   timer_.SetTaskRunner(task_runner);
 }
 
 }  // namespace browser_sync