remoting: Remove use of ALLOW_THIS_IN_INITIALIZER_LIST.

remoting/host/policy_hack/policy_watcher_linux.cc

 // Copyright (c) 2012 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.
 
 // Most of this code is copied from various classes in
 // src/chrome/browser/policy. In particular, look at
 //
 //   file_based_policy_loader.{h,cc}
 //   config_dir_policy_provider.{h,cc}
 //
@@ skipping 32 matching lines @@
 const int kSettleIntervalSeconds = 5;
 
 }  // namespace
 
 class PolicyWatcherLinux : public PolicyWatcher {
  public:
   PolicyWatcherLinux(scoped_refptr<base::SingleThreadTaskRunner> task_runner,
                      const base::FilePath& config_dir)
       : PolicyWatcher(task_runner),
         config_dir_(config_dir),
-        ALLOW_THIS_IN_INITIALIZER_LIST(weak_factory_(this)) {
+        (weak_factory_(this) {

[Sergey Ulanov, 2013/04/26 20:48:22]

typo: remove (

[scherkus (not reviewing), 2013/04/26 20:52:55]

Done. Need to tweak my script :)

     // Detach the factory because we ensure that only the policy thread ever
     // calls methods on this. Also, the API contract of having to call
     // StopWatching() (which signals completion) after StartWatching()
     // before this object can be destructed ensures there are no users of
     // this object before it is destructed.
     weak_factory_.DetachFromThread();
   }
 
   virtual ~PolicyWatcherLinux() {}
 
@@ skipping 182 matching lines @@
 };
 
 PolicyWatcher* PolicyWatcher::Create(
     scoped_refptr<base::SingleThreadTaskRunner> task_runner) {
   base::FilePath policy_dir(kPolicyDir);
   return new PolicyWatcherLinux(task_runner, policy_dir);
 }
 
 }  // namespace policy_hack
 }  // namespace remoting