[Cleanup] fix macro, comments and unused includes

fix macro, comments and unused includes

BUG=none

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=221499

[Daniel Erat, 2013-09-05 16:56:33]

https://codereview.chromium.org/23591028/diff/2001/ash/root_window_settings.h
File ash/root_window_settings.h (right):

https://codereview.chromium.org/23591028/diff/2001/ash/root_window_settings.h...
ash/root_window_settings.h:38: // Creates and returns RootWindowSettings for
|root|.
nit: either document ownership of the returned pointer or (better) make it
return a scoped_ptr

https://codereview.chromium.org/23591028/diff/2001/ash/root_window_settings.h...
ash/root_window_settings.h:39: RootWindowSettings*
InitRootWindowSettings(aura::RootWindow* root);
nit: renaming it to CreateRootWindowSettings might be more accurate

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

https://codereview.chromium.org/23591028/diff/2001/ash/root_window_settings.h
File ash/root_window_settings.h (right):

https://codereview.chromium.org/23591028/diff/2001/ash/root_window_settings.h...
ash/root_window_settings.h:38: // Creates and returns RootWindowSettings for
|root|.
On 2013/09/05 16:56:34, Daniel Erat wrote:
> nit: either document ownership of the returned pointer or (better) make it
> return a scoped_ptr

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

https://codereview.chromium.org/23591028/diff/2001/ash/root_window_settings.h...
ash/root_window_settings.h:39: RootWindowSettings*
InitRootWindowSettings(aura::RootWindow* root);
On 2013/09/05 16:56:34, Daniel Erat wrote:
> nit: renaming it to CreateRootWindowSettings might be more accurate

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

[Daniel Erat, 2013-09-05 17:22:57]

lgtm

[commit-bot: I haz the power, 2013-09-05 17:29:05]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/oshima@chromium.org/23591028/10001

[commit-bot: I haz the power, 2013-09-05 20:14:04]

Change committed as 221499