Change the panel using the improved way of doing autosizing.

Change the panel using the improved way of doing autosizing.

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

[jennb, 2011-12-01 21:56:56]

LGTM

http://codereview.chromium.org/8774016/diff/1/chrome/browser/ui/panels/panel.cc
File chrome/browser/ui/panels/panel.cc (right):

http://codereview.chromium.org/8774016/diff/1/chrome/browser/ui/panels/panel....
chrome/browser/ui/panels/panel.cc:618: void
Panel::EnableRendererAutoResize(RenderViewHost* render_view_host) {
Only reason we had this helper func was because it was a 2 step process before.
You can get rid of this and use RVH's EnableAutoResize direclty everywhere.

[levin, 2011-12-01 22:41:11]

How's this?

[levin, 2011-12-01 23:03:51]

On 2011/12/01 22:41:11, levin wrote:
> How's this?

Comment from Jenn:
"GetRenderViewHost() uses browser's GetSelectedTabContents. At two of the call
sites, the tab contents might not be the same as the browser's selected tab
contents. e.g. usually not inserted yet when TabInserted is called."

[levin, 2011-12-01 23:29:25]

On 2011/12/01 23:03:51, levin wrote:
> On 2011/12/01 22:41:11, levin wrote:
> > How's this?
> 
> Comment from Jenn:
> "GetRenderViewHost() uses browser's GetSelectedTabContents. At two of the call
> sites, the tab contents might not be the same as the browser's selected tab
> contents. e.g. usually not inserted yet when TabInserted is called."
Fixed.

[jennb, 2011-12-01 23:46:31]

LGTM

Nice.