Some code style cleanup under src/app.

Some code style cleanup under src/app.

BUG=None
TEST=None

Patch contributed by thiago.farina@gmail.com

Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=24859

[tfarina (gmail-do not use), 2009-08-24 00:22:20]

Hi Mohamed, this is just a little style fix, could you take a look, please?

[Mark Mentovai, 2009-08-24 01:05:50]

Quick drive-by nit.

http://codereview.chromium.org/173260/diff/1/5
File app/gfx/color_utils.h (right):

http://codereview.chromium.org/173260/diff/1/5#newcode17
Line 17: double Y;   // luminance
Our style guide requires two spaces between the end of a line and its in-line
comment.  In some cases, to make things line up vertically, more than two spaces
are permitted.  You've got three here, but there's no reason for more than two
in this case.

http://google-styleguide.googlecode.com/svn/trunk/cppguide.xml?showone=Implem...

[tfarina (gmail-do not use), 2009-08-24 01:15:23]

On 2009/08/24 01:05:50, Mark Mentovai wrote:
> Quick drive-by nit.
> 
> http://codereview.chromium.org/173260/diff/1/5
> File app/gfx/color_utils.h (right):
> 
> http://codereview.chromium.org/173260/diff/1/5#newcode17
> Line 17: double Y;   // luminance
You've got three here, but there's no reason for more than two
> in this case.
> 
>
http://google-styleguide.googlecode.com/svn/trunk/cppguide.xml?showone=Implem...
Sorry, I pressed tab instead of space in this case. Fixed.

[tfarina (gmail-do not use), 2009-08-24 01:15:35]

http://codereview.chromium.org/173260/diff/1/5
File app/gfx/color_utils.h (right):

http://codereview.chromium.org/173260/diff/1/5#newcode17
Line 17: double Y;   // luminance
On 2009/08/24 01:05:50, Mark Mentovai wrote:
> Our style guide requires two spaces between the end of a line and its in-line
> comment.  In some cases, to make things line up vertically, more than two
spaces
> are permitted.  You've got three here, but there's no reason for more than two
> in this case.
> 
>
http://google-styleguide.googlecode.com/svn/trunk/cppguide.xml?showone=Implem...

Done.

[tfarina (gmail-do not use), 2009-08-25 18:44:13]

Ping

[Mohamed Mansour, 2009-08-26 02:30:16]

LGTM

[tfarina (gmail-do not use), 2009-08-27 19:00:12]

On 2009/08/26 02:30:16, Mohamed Mansour wrote:
> LGTM
Thanks Mohamed, can you land it to me?