Task manager stays on top, even when "opened" multiple times.

Task manager stays on top, even when "opened" multiple times.

BUG=7227

[brettw, 2009-02-26 00:09:10]

LGTM with a minor comment fix.

http://codereview.chromium.org/28150/diff/3/4
File chrome/views/widget_win.cc (right):

http://codereview.chromium.org/28150/diff/3/4#newcode244
Line 244: // Check if the window is topmost
This comment should have a period at the end. I would probably add a blank line
before it as well, but that's not a requirement.

[Peter Kasting, 2009-02-26 00:44:46]

LGTM with my nit and Brett's

http://codereview.chromium.org/28150/diff/3/4
File chrome/views/widget_win.cc (right):

http://codereview.chromium.org/28150/diff/3/4#newcode245
Line 245: bool top_most = false;
Nit: Avoid the bool, just do:

...
HWND hwnd = (wi.dwExStyle & WS_EX_TOPMOST) ? HWND_TOPMOST : HWND_NOTOPMOST;
SetWindowPos(hwnd, 0, 0, 0, 0, flags);

[Peter Kasting, 2009-02-26 20:59:25]

I don't think you're yet in the AUTHORS file (please correct me if needed).  Can
you get the contributor's agreement signed and in to us, and upload a version
that adds yourself to AUTHORS as well?

[Xharze, 2009-02-26 22:43:07]

I have now completed the contributor's agreement online, and uploaded the
AUTHORS file with my name and email.

[Peter Kasting, 2009-02-27 01:13:06]

Landed in r10558.