[Cleanup] fix macro, comments and unused includes

ash/root_window_settings.h

Index: ash/root_window_settings.h
diff --git a/ash/root_window_settings.h b/ash/root_window_settings.h
index 5dc3363f1e35b474df77ff2b5b052364ef20de16..adb28aa1fdf0586a6604c176bffe49cf83054f11 100644
--- a/ash/root_window_settings.h
+++ b/ash/root_window_settings.h
@@ -2,8 +2,8 @@
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
-#ifndef ASH_WM_ROOT_WINDOW_SETTINGS_H_
-#define ASH_WM_ROOT_WINDOW_SETTINGS_H_
+#ifndef ASH_ROOT_WINDOW_SETTINGS_H_
+#define ASH_ROOT_WINDOW_SETTINGS_H_
 
 #include "ash/ash_export.h"
 #include "base/basictypes.h"
@@ -35,8 +35,7 @@ struct RootWindowSettings {
   RootWindowController* controller;
 };
 
-// Initializes and creates RootWindowSettings for |root|, and returns
-// the property.
+// Creates and returns RootWindowSettings for |root|.

[Daniel Erat, 2013/09/05 16:56:34]

nit: either document ownership of the returned pointer or (better) make it
return a scoped_ptr

[oshima, 2013/09/05 17:05:54]

Done. it is owned by the root window, so scoped_ptr isn't necessary.

 RootWindowSettings* InitRootWindowSettings(aura::RootWindow* root);

[Daniel Erat, 2013/09/05 16:56:34]

nit: renaming it to CreateRootWindowSettings might be more accurate

[oshima, 2013/09/05 17:05:54]

The object is owned by the root, so I think Init is better.

 
 // Returns the RootWindowSettings for |root|.
@@ -49,4 +48,4 @@ GetRootWindowSettings(const aura::RootWindow* root);
 }  // namespace internal
 }  // namespace ash
 
-#endif  // ASH_WM_ROOT_WINDOW_SETTINGS_H_
+#endif  // ASH_ROOT_WINDOW_SETTINGS_H_