NNTP: Fix window tooltip so that it is not shown out of place....

NNTP: Fix window tooltip so that it is not shown out of place.

BUG=18529
TEST=The bug is hard to reproduce but it involved moving the mouse
fast over the link.


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

[Glen Murphy, 2009-08-06 00:05:25]

OK + nit

http://codereview.chromium.org/164012/diff/1001/4
File chrome/browser/resources/new_new_tab.js (right):

http://codereview.chromium.org/164012/diff/1001/4#newcode1142
Line 1142: } else { // focus
two spaces before //, I think

[arv (Not doing code reviews), 2009-08-06 00:18:50]

http://codereview.chromium.org/164012/diff/1001/4
File chrome/browser/resources/new_new_tab.js (right):

http://codereview.chromium.org/164012/diff/1001/4#newcode1142
Line 1142: } else { // focus
On 2009/08/06 00:05:27, Glen Murphy wrote:
> two spaces before //, I think

I've never seen that in any style guide. There is other code that only have one
space before the comment. I can update all those comments when/if I find the
rule about this in the style guide.

[Glen Murphy, 2009-08-06 00:21:48]

Eh, it's something I just kept getting dinged on in my reviews, so I carried it
forward. If there's no source, LGTM.


On 2009/08/06 00:18:50, arv wrote:
> http://codereview.chromium.org/164012/diff/1001/4
> File chrome/browser/resources/new_new_tab.js (right):
> 
> http://codereview.chromium.org/164012/diff/1001/4#newcode1142
> Line 1142: } else { // focus
> On 2009/08/06 00:05:27, Glen Murphy wrote:
> > two spaces before //, I think
> 
> I've never seen that in any style guide. There is other code that only have
one
> space before the comment. I can update all those comments when/if I find the
> rule about this in the style guide.