Move extension pop-ups and notifications to the new auto-resize code.

Move extension pop-ups and notifications to the new auto-resize code.

This should fix the "flicker" issues where autosize ends up flipping
back and forth between two size in rapid succession.

The change in render_widget_host is needed to avoid cases where the OnMsgUpdateRect happens while processing a resize message which may result in recursion back into the resize code which is undesirable.

BUG=58816, 56693
TEST=Try out auto resize in notifications and extensions. Best to do with windows that change
     size and hit the maximum as well as minimum sizes on the 3 desktop platforms for pop-ups.


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

[levin, 2012-03-15 19:30:11]

John, please look at the balloon changes in here.

This change is all about moving to the new resize mechanism which doesn't have
the feedback loops that happened with the previous mechanism.

Thanks!

[Aaron Boodman, 2012-03-15 19:32:07]

+finnur

Finnur, can you please look at this from an extensions point of view?

[johnnyg, 2012-03-15 19:52:04]

LGTM

Notification balloon stuff looks good.  Sorry, can't figure out how to become
johnnyg@chromium.org.

[Finnur, 2012-03-16 13:31:02]

My Mac and GTK foo is not very strong but those changes look simple and in line
with the rest.

So, with that: LGTM. Looking very much forward to seeing this flicker bug go
away.

Go levin! :)

[levin, 2012-03-20 16:01:23]

Adding more folks (from owners files)...

sesek would you look over the chrome/browser/ui/cocoa changes?

sky would you look over the chrome/browser/ui/android, chrome/browser/ui/views,
content/browser/render_host changes?

estade would you look over the content/browser/ui/gtk changes?

Thanks,
dave

[sky, 2012-03-20 16:15:06]

LGTM

http://codereview.chromium.org/9702068/diff/8004/content/browser/renderer_hos...
File content/browser/renderer_host/render_widget_host_impl.cc (right):

http://codereview.chromium.org/9702068/diff/8004/content/browser/renderer_hos...
content/browser/renderer_host/render_widget_host_impl.cc:1189: bool
post_callback = new_auto_size_.IsEmpty();
Add a comment as to why you need to do this.

[Evan Stade, 2012-03-20 17:50:09]

gtk lgtm

[Robert Sesek, 2012-03-20 18:17:02]

cocoa lgtm

[commit-bot: I haz the power, 2012-03-20 21:20:17]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/levin@chromium.org/9702068/18001

[commit-bot: I haz the power, 2012-03-20 23:07:08]

Change committed as 127824