Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(6098)

Unified Diff: chrome/browser/ui/gtk/extensions/extension_popup_gtk.cc

Issue 10913243: extensions: Add ExtensionView interface. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: weak Created 8 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/ui/gtk/extensions/extension_popup_gtk.cc
diff --git a/chrome/browser/ui/gtk/extensions/extension_popup_gtk.cc b/chrome/browser/ui/gtk/extensions/extension_popup_gtk.cc
index 8ec192ae230a532df69df2b9eecd3d6c10a87e3f..556ad86a6640cbc303f7d091d331d4f403ded231 100644
--- a/chrome/browser/ui/gtk/extensions/extension_popup_gtk.cc
+++ b/chrome/browser/ui/gtk/extensions/extension_popup_gtk.cc
@@ -16,6 +16,7 @@
#include "chrome/browser/debugger/devtools_window.h"
#include "chrome/browser/extensions/extension_host.h"
#include "chrome/browser/extensions/extension_process_manager.h"
+#include "chrome/browser/extensions/extension_view.h"
#include "chrome/browser/profiles/profile.h"
#include "chrome/browser/ui/browser.h"
#include "chrome/browser/ui/browser_window.h"
@@ -48,7 +49,7 @@ ExtensionPopupGtk::ExtensionPopupGtk(Browser* browser,
host_(host),
anchor_(anchor),
weak_factory_(this) {
- host_->view()->SetContainer(this);
+ host_->GetExtensionView()->SetContainer(this);
being_inspected_ = show_action == SHOW_AND_INSPECT;
// If the host had somehow finished loading, then we'd miss the notification
@@ -138,14 +139,16 @@ void ExtensionPopupGtk::BubbleClosing(BubbleGtk* bubble,
delete this;
}
-void ExtensionPopupGtk::OnExtensionSizeChanged(
- ExtensionViewGtk* view,
- const gfx::Size& new_size) {
+void ExtensionPopupGtk::OnExtensionSizeChanged(ExtensionView* view,
+ const gfx::Size& new_size) {
int width = std::max(kMinWidth, std::min(kMaxWidth, new_size.width()));
int height = std::max(kMinHeight, std::min(kMaxHeight, new_size.height()));
- view->render_view_host()->GetView()->SetSize(gfx::Size(width, height));
- gtk_widget_set_size_request(view->native_view(), width, height);
+ view->GetRenderViewHost()->GetView()->SetSize(gfx::Size(width, height));
+ gtk_widget_set_size_request(view->GetNativeView(), width, height);
+}
+
+void ExtensionPopupGtk::OnExtensionViewDidShow(ExtensionView* view) {
}
bool ExtensionPopupGtk::DestroyPopup() {
@@ -179,7 +182,8 @@ void ExtensionPopupGtk::ShowPopup() {
// This border is necessary so the bubble's corners do not get cut off by the
// render view.
gtk_container_set_border_width(GTK_CONTAINER(border_box), 2);
- gtk_container_add(GTK_CONTAINER(border_box), host_->view()->native_view());
+ gtk_container_add(GTK_CONTAINER(border_box),
+ host_->GetExtensionView()->GetNativeView());
// We'll be in the upper-right corner of the window for LTR languages, so we
// want to put the arrow at the upper-right corner of the bubble to match the
@@ -204,6 +208,7 @@ void ExtensionPopupGtk::DestroyPopupWithoutResult() {
gfx::Rect ExtensionPopupGtk::GetViewBounds() {
GtkAllocation allocation;
- gtk_widget_get_allocation(host_->view()->native_view(), &allocation);
+ gtk_widget_get_allocation(host_->GetExtensionView()->GetNativeView(),
+ &allocation);
return gfx::Rect(allocation);
}
« no previous file with comments | « chrome/browser/ui/gtk/extensions/extension_popup_gtk.h ('k') | chrome/browser/ui/gtk/extensions/extension_view_gtk.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698