diff options
| -rwxr-xr-x | chrome/browser/extensions/extension_view.cc | 40 | ||||
| -rw-r--r-- | chrome/common/temp_scaffolding_stubs.h | 2 |
2 files changed, 15 insertions, 27 deletions
diff --git a/chrome/browser/extensions/extension_view.cc b/chrome/browser/extensions/extension_view.cc index 8caf259..960765a 100755 --- a/chrome/browser/extensions/extension_view.cc +++ b/chrome/browser/extensions/extension_view.cc @@ -50,32 +50,22 @@ void ExtensionView::DidStopLoading(RenderViewHost* render_view_host, void ExtensionView::DidContentsPreferredWidthChange(const int pref_width) { if (pref_width > 0) { - gfx::Size pref_size = GetPreferredSize(); - // This seems to get called a fair amount, so early out if preferred width - // is unchanged. - if (pref_size.width() != pref_width) { - // SchedulePaint first because new_width may be smaller and we want - // the Parent to paint the vacated space. - SchedulePaint(); - set_preferred_size(gfx::Size(pref_width, 100)); - SizeToPreferredSize(); - - // TODO(rafaelw): This assumes that the extension view is a child of an - // ExtensionToolstrip, which is a child of the BookmarkBarView. There - // should be a way to do this where the ExtensionView doesn't have to know - // its containment hierarchy. - if (GetParent() != NULL && GetParent()->GetParent() != NULL) { - GetParent()->GetParent()->Layout(); - } - - // Also SchedulePaint after because new_width may be larger, and we need - // to draw the new part of the view. - SchedulePaint(); - - // Need to tell the RenderWidgetHostView about the new size since it's - // not part of the view hierarchy. - render_view_host()->view()->SetSize(size()); + // SchedulePaint first because new_width may be smaller and we want + // the Parent to paint the vacated space. + SchedulePaint(); + set_preferred_size(gfx::Size(pref_width, 100)); + SizeToPreferredSize(); + + // TODO(rafaelw): This assumes that the extension view is a child of an + // ExtensionToolstrip, which is a child of the BookmarkBarView. There should + // be a way to do this where the ExtensionView doesn't have to know it's + // containment hierarchy. + if (GetParent() != NULL && GetParent()->GetParent() != NULL) { + GetParent()->GetParent()->Layout(); } + + SchedulePaint(); + render_view_host()->WasResized(); } } diff --git a/chrome/common/temp_scaffolding_stubs.h b/chrome/common/temp_scaffolding_stubs.h index 5578bb1..cac23ea 100644 --- a/chrome/common/temp_scaffolding_stubs.h +++ b/chrome/common/temp_scaffolding_stubs.h @@ -514,8 +514,6 @@ class HWNDHtmlView { void Layout() { NOTIMPLEMENTED(); } void SchedulePaint() { NOTIMPLEMENTED(); } SiteInstance* site_instance() { NOTIMPLEMENTED(); return NULL; } - gfx::Size GetPreferredSize() { NOTIMPLEMENTED(); return gfx::Size(); } - gfx::Size size() { NOTIMPLEMENTED(); return gfx::Size(); } }; #endif // CHROME_COMMON_TEMP_SCAFFOLDING_STUBS_H_ |