Fix the uses of T* conversion operator from scoped_refptr<T> which is now removed.

chromecast/common/chromecast_config.cc

 // Copyright 2014 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 "chromecast/common/chromecast_config.h"
 
 #include <string>
 
 #include "base/command_line.h"
 #include "base/file_util.h"
@@ skipping 63 matching lines @@
   DCHECK(thread_checker_.CalledOnValidThread());
   VLOG(1) << "Loading config from " << config_path_.value();
   registry->RegisterIntegerPref(prefs::kRemoteDebuggingPort, 0);
 
   metrics::RegisterPrefs(registry);
   RegisterPlatformPrefs(registry);
 
   PersistentPrefStore::PrefReadError prefs_read_error =
       PersistentPrefStore::PREF_READ_ERROR_NONE;
   base::PrefServiceFactory prefServiceFactory;
-  prefServiceFactory.SetUserPrefsFile(config_path_,
-      JsonPrefStore::GetTaskRunnerForFile(config_path_, worker_pool_));
+  scoped_refptr<base::SequencedTaskRunner> task_runner =
+      JsonPrefStore::GetTaskRunnerForFile(config_path_, worker_pool_.get());
+  prefServiceFactory.SetUserPrefsFile(config_path_, task_runner.get());

[damienv1, 2014/09/08 16:27:21]

Is it safe ?
Ref count for task runner will be decremented at the end of this function.

[lcwu1, 2014/09/08 18:01:58]

Yes, it is. task_runner's raw pointer gets passed into JsonPrefStore's
constructor, which in turn (through a temporary scoped_refptr) assigns it to a
scoped_refptr data member.

   prefServiceFactory.set_async(false);
   prefServiceFactory.set_read_error_callback(
       base::Bind(&UserPrefsLoadError, &prefs_read_error));
   pref_service_ = prefServiceFactory.Create(registry);
 
   if (prefs_read_error == PersistentPrefStore::PREF_READ_ERROR_NONE) {
     return true;
   } else {
     LOG(ERROR) << "Cannot initialize chromecast config: "
                << config_path_.value()
@@ skipping 35 matching lines @@
   if (pref_service_->IsUserModifiablePreference(key.c_str())) {
     VLOG(1) << "Set config: key=" << key << ", value=" << value;
     pref_service_->SetInteger(key.c_str(), value);
   } else {
     LOG(ERROR) << "Cannot set read-only config: key=" << key
                << ", value=" << value;
   }
 }
 
 }  // namespace chromecast