Fix various problems with work area change notifications so that we now (hope...

Fix various problems with work area change notifications so that we now (hopefully) move/resize correctly whenever the work area changes, no matter why.

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

[Ben Goodger (Google), 2009-04-10 19:59:03]

evil. OK.

http://codereview.chromium.org/67023/diff/1/2
File chrome/views/widget/widget_win.cc (right):

http://codereview.chromium.org/67023/diff/1/2#newcode722
Line 722: } else if (toplevel_) {
Can you just use !GetParent() here instead? I want to remove toplevel_. Maybe
you could add a note by the toplevel_ decl too. I think its flaky to have a
variable that incompletely maps to a property that's easily accessible from a
API call.

If you like using something called toplevel, feel free to add a method called
IsToplevel that just returns !GetParent().

[Peter Kasting, 2009-04-10 20:05:02]

http://codereview.chromium.org/67023/diff/1/2
File chrome/views/widget/widget_win.cc (right):

http://codereview.chromium.org/67023/diff/1/2#newcode722
Line 722: } else if (toplevel_) {
On 2009/04/10 19:59:03, Ben Goodger wrote:
> Can you just use !GetParent() here instead?

Yep, and I can just remove toplevel_ entirely, going one better than merely
commenting it :)