ChromeOS: Implement periodic timezone refresh on geolocation data.

chrome/browser/supervised_user/supervised_user_service.cc

Index: chrome/browser/supervised_user/supervised_user_service.cc
diff --git a/chrome/browser/supervised_user/supervised_user_service.cc b/chrome/browser/supervised_user/supervised_user_service.cc
index a2781ef342ab5905b0fe1386b74dcbbdf4d10fff..f2fb1e7404a2f15cf26ced560c7f52d85ea15e29 100644
--- a/chrome/browser/supervised_user/supervised_user_service.cc
+++ b/chrome/browser/supervised_user/supervised_user_service.cc
@@ -43,6 +43,7 @@
 #include "components/signin/core/browser/signin_manager_base.h"
 #include "content/public/browser/browser_thread.h"
 #include "content/public/browser/user_metrics.h"
+#include "net/url_request/url_request_context_getter.h"
 #include "ui/base/l10n/l10n_util.h"
 
 #if defined(OS_CHROMEOS)
@@ -173,7 +174,7 @@ void SupervisedUserService::URLFilterContext::InitAsyncURLChecker(
       BrowserThread::IO,
       FROM_HERE,
       base::Bind(&SupervisedUserURLFilter::InitAsyncURLChecker,
-                 io_url_filter_, context, cx));
+                 io_url_filter_, base::Unretained(context), cx));

[Bernhard Bauer, 2015/01/05 10:22:33]

Is base::Unretained necessary for raw pointers after the first one now? And is
there a reason why this change is made in this CL?

[Alexander Alekseev, 2015/01/15 18:59:02]

This is actually a refcounted pointer, but here it was always bound as raw
pointer.

My previous patch set introduced some change to headers, so I had to add proper
include and force the pointer to be bound as Unretained to keep compatibility.

I see that now it is not required (I've changed something and error doesn't
happen any more). So I've removed this change, though it is a bug. Probably we
should fix it with a separate CL.

 }
 
 SupervisedUserService::SupervisedUserService(Profile* profile)