base: Put DISALLOW_COPY_AND_ASSIGN inside private section of classes.

This macro should be private to make the constructors hidden, making
them fail at compile time if called, instead of at link time.

R=Nico
BUG=464816,447156

Review URL: https://codereview.chromium.org/986623004

Cr-Commit-Position: refs/heads/master@{#319568}

base/containers/stack_container.h

@@ -176,6 +176,7 @@ class StackContainer {
   Allocator allocator_;
   ContainerType container_;
 
+ private:
   DISALLOW_COPY_AND_ASSIGN(StackContainer);
 };
 

base/files/file_path_watcher_browsertest.cc

@@ -192,6 +192,7 @@ class FilePathWatcherTest : public testing::Test {
   ScopedTempDir temp_dir_;
   scoped_refptr<NotificationCollector> collector_;
 
+ private:
   DISALLOW_COPY_AND_ASSIGN(FilePathWatcherTest);
 };
 

base/observer_list_threadsafe.h

@@ -199,6 +199,7 @@ class ObserverListThreadSafe
     scoped_refptr<base::MessageLoopProxy> loop;
     ObserverList<ObserverType> list;
 
+   private:
     DISALLOW_COPY_AND_ASSIGN(ObserverListContext);
   };
 

base/prefs/pref_member.h

@@ -273,6 +273,7 @@ class PrefMember : public subtle::PrefMemberBase {
     // tree.
     mutable ValueType value_;
 
+   private:
     DISALLOW_COPY_AND_ASSIGN(Internal);
   };
 

base/system_monitor/system_monitor_unittest.cc

@@ -23,6 +23,7 @@ class SystemMonitorTest : public testing::Test {
   MessageLoop message_loop_;
   scoped_ptr<SystemMonitor> system_monitor_;
 
+ private:
   DISALLOW_COPY_AND_ASSIGN(SystemMonitorTest);
 };
 

base/test/mock_devices_changed_observer.h

@@ -21,6 +21,7 @@ class MockDevicesChangedObserver
   MOCK_METHOD1(OnDevicesChanged,
                void(base::SystemMonitor::DeviceType device_type));
 
+ private:
   DISALLOW_COPY_AND_ASSIGN(MockDevicesChangedObserver);
 };
 

base/threading/thread_local_storage.h

@@ -136,6 +136,7 @@ class BASE_EXPORT ThreadLocalStorage {
     DISALLOW_COPY_AND_ASSIGN(Slot);
   };
 
+ private:
   DISALLOW_COPY_AND_ASSIGN(ThreadLocalStorage);
 };