Remove ref-counting from FilePathWatcher.

base/files/file_path_watcher_stub.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.
 
 // This file exists for Unix systems which don't have the inotify headers, and
 // thus cannot build file_watcher_inotify.cc
 
 #include "base/files/file_path_watcher.h"
 
+#include "base/macros.h"
+#include "base/memory/ptr_util.h"
+
 namespace base {
 
 namespace {
 
 class FilePathWatcherImpl : public FilePathWatcher::PlatformDelegate {
  public:
+  ~FilePathWatcherImpl() override = default;
+
   bool Watch(const FilePath& path,
              bool recursive,
              const FilePathWatcher::Callback& callback) override {
     return false;
   }
 
   void Cancel() override {}
 
- protected:
-  ~FilePathWatcherImpl() override {}
+ private:
+  DISALLOW_COPY_AND_ASSIGN(FilePathWatcherImpl);

[gab, 2017/01/05 21:22:53]

Probably need an explicit public default constructor because of this?

[fdoray, 2017/01/05 23:04:11]

Done.

 };
 
 }  // namespace
 
 FilePathWatcher::FilePathWatcher() {
   sequence_checker_.DetachFromSequence();
-  impl_ = new FilePathWatcherImpl();
+  impl_ = MakeUnique<FilePathWatcherImpl>();
 }
 
 }  // namespace base