Rewrite enum in MACRO_STYLE

components/offline_pages/snapshot_controller.h

 // Copyright 2016 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.
 
 #ifndef COMPONENTS_OFFLINE_PAGES_SNAPSHOT_CONTROLLER_H_
 #define COMPONENTS_OFFLINE_PAGES_SNAPSHOT_CONTROLLER_H_
 
 #include "base/memory/ref_counted.h"
 #include "base/memory/weak_ptr.h"
 #include "base/single_thread_task_runner.h"
 
 namespace offline_pages {
 
 // Takes various signals and produces StartSnapshot calls following a specific
 // policy. Can request snapshots multiple times per 'session'. Session can be
 // ended and another one started by calling Reset().
 // Main invariants:
 // - It never starts overlapping snapshots, Client reports when previous
 //   snapshot is done.
 // - The currently worked on (pending) snapshot is always allowed to complete,
 //   the new attempts to start a snapshot are ignored until it does.
 // - Some signals prevent more snapshots to be taken.
 //   OnLoad is currently such signal.
 // - Once Reset() is called on the SnapshotController, the delayed tasks are
 //   reset so no StartSnapshot calls is made 'cross-session'.
 class SnapshotController {
  public:
-  // kStateReady - listening to input, will start snapshot when needed.
-  // kStateSnapshotPending - snapshot is in progress, don't start another.
-  // kStateStopped - terminal state, no snapshots until reset.
-  enum class State { kReady, kSnapshotPending, kStopped };
+  // READY - listening to input, will start snapshot when needed.
+  // SNAPSHOT_PENDING- snapshot is in progress, don't start another.
+  // STOPPED - terminal state, no snapshots until reset.
+  enum class State {
+    READY,

[fgorski, 2016/06/11 16:25:06]

Put documentation after the definition of the enum, like in other files please.
Make sure to align with the deepest comment (2 spaces after SNAPSHOT_PENDING,)
and use Sentence case, e.g.
Snapshot is in ... another.

[Vivian, 2016/06/13 17:07:04]

Done.

+    SNAPSHOT_PENDING,
+    STOPPED,
+  };
 
   // Client of the SnapshotController.
   class  Client {
    public:
     // Invoked at a good moment to start a snapshot. May be invoked multiple
     // times, but not in overlapping manner - waits until
     // PreviousSnapshotCompleted() before the next StatrSnapshot().
     // Client should overwrite the result of previous snapshot with the new one,
     // it is assumed that later snapshots are better then previous.
     virtual void StartSnapshot() = 0;
@@ skipping 36 matching lines @@
   SnapshotController::State state_;
 
   base::WeakPtrFactory<SnapshotController> weak_ptr_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(SnapshotController);
 };
 
 }  // namespace offline_pages
 
 #endif  // COMPONENTS_OFFLINE_PAGES_SNAPSHOT_CONTROLLER_H_