Clean up a few bits of files that I'm about to modify. The only visible chan...

Clean up a few bits of files that I'm about to modify.  The only visible change is the removal of some extra spacing Ben erroneously added to the link infobar a while back (clients already do their own spacing as needed, so this just added extra whitespace in mid-sentence).

BUG=none
TEST=none

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

[sky, 2009-09-25 00:03:51]

One nit, LGTM otherwise.

http://codereview.chromium.org/223029/diff/1/5
File views/controls/label.cc (right):

http://codereview.chromium.org/223029/diff/1/5#newcode21
Line 21: // static
nit: no need to indent this.

[Peter Kasting, 2009-09-25 00:08:17]

On 2009/09/25 00:03:51, sky wrote:
> http://codereview.chromium.org/223029/diff/1/5#newcode21
> Line 21: // static
> nit: no need to indent this.

Stupid MSVC "fixing" my indentation :(

[whywhat, 2010-10-27 12:28:12]

http://codereview.chromium.org/223029/diff/1/2
File views/controls/link.cc (right):

http://codereview.chromium.org/223029/diff/1/2#newcode156
views/controls/link.cc:156: normal_color_ = color;
Is this a typo? Shouldn't be highlighted_color_ = color?
Related to http://codereview.chromium.org/4179001

[Peter Kasting, 2010-10-27 18:23:31]

http://codereview.chromium.org/223029/diff/1/2
File views/controls/link.cc (right):

http://codereview.chromium.org/223029/diff/1/2#newcode156
views/controls/link.cc:156: normal_color_ = color;
On 2010/10/27 12:28:12, whywhat wrote:
> Is this a typo? Shouldn't be highlighted_color_ = color?
> Related to http://codereview.chromium.org/4179001

If you look more closely you'll notice that I was merely moving functions from
one place in the file to another.  This is indeed wrong, but the bug didn't
originate in this review.

Since this is fixed anyway by the change you linked, I'm not sure why it
matters.

[whywhat, 2010-10-28 11:35:23]

http://codereview.chromium.org/223029/diff/1/2
File views/controls/link.cc (right):

http://codereview.chromium.org/223029/diff/1/2#newcode156
views/controls/link.cc:156: normal_color_ = color;
On 2010/10/27 18:23:32, Peter Kasting wrote:
> On 2010/10/27 12:28:12, whywhat wrote:
> > Is this a typo? Shouldn't be highlighted_color_ = color?
> > Related to http://codereview.chromium.org/4179001
> 
> If you look more closely you'll notice that I was merely moving functions from
> one place in the file to another.  This is indeed wrong, but the bug didn't
> originate in this review.
> 
> Since this is fixed anyway by the change you linked, I'm not sure why it
> matters.

I wasn't about to blame anyone :) I just found someone who touched this code and
who might know if this typo was intentional somehow (you never know, right?).
It's too easy to think you're fixing something obvious while you're actually
breaking something not commented well-enough.

[Peter Kasting, 2010-10-28 18:26:37]

On 2010/10/28 11:35:23, whywhat wrote:
> I wasn't about to blame anyone :) I just found someone who touched this code
and
> who might know if this typo was intentional somehow (you never know, right?).
> It's too easy to think you're fixing something obvious while you're actually
> breaking something not commented well-enough.

Ah, sorry, I didn't even connect the dots that you reviewed the other change. 
Also I saw "whywhat" and thought "who is this totally random outside person?". 
So I couldn't figure out why you were interested.

If possible, I suggest changing your "display name" to something akin to your
email or real name, to reduce future confusion...