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

Unified Diff: blimp/client/core/contents/blimp_contents_impl.cc

Issue 2292723003: Move remaining Blimp feature code to core. (Closed)
Patch Set: Fix build break Created 4 years, 4 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: blimp/client/core/contents/blimp_contents_impl.cc
diff --git a/blimp/client/core/contents/blimp_contents_impl.cc b/blimp/client/core/contents/blimp_contents_impl.cc
index 76e486f0a68630842444918a99ba2c8c6e801329..2790c6a5a527b20d5136b62ac5b9ea69a40ba7d8 100644
--- a/blimp/client/core/contents/blimp_contents_impl.cc
+++ b/blimp/client/core/contents/blimp_contents_impl.cc
@@ -24,14 +24,19 @@ const char kBlimpContentsImplAndroidKey[] = "blimp_contents_impl_android";
#endif // OS_ANDROID
}
-BlimpContentsImpl::BlimpContentsImpl(int id,
- ImeFeature* ime_feature,
- NavigationFeature* navigation_feature,
- TabControlFeature* tab_control_feature)
+BlimpContentsImpl::BlimpContentsImpl(
+ int id,
+ BlimpCompositorDependencies* compositor_deps,
+ ImeFeature* ime_feature,
+ NavigationFeature* navigation_feature,
+ RenderWidgetFeature* render_widget_feature,
+ TabControlFeature* tab_control_feature)
: navigation_controller_(this, navigation_feature),
+ compositor_manager_(render_widget_feature, compositor_deps),
id_(id),
tab_control_feature_(tab_control_feature) {
- blimp_contents_view_ = BlimpContentsView::Create(this);
+ blimp_contents_view_ =
+ BlimpContentsView::Create(this, compositor_manager_.layer());
}
BlimpContentsImpl::~BlimpContentsImpl() {
@@ -73,6 +78,14 @@ gfx::NativeView BlimpContentsImpl::GetNativeView() {
return blimp_contents_view_->GetNativeView();
}
+void BlimpContentsImpl::Show() {
+ compositor_manager_.SetVisible(true);
+}
+
+void BlimpContentsImpl::Hide() {
+ compositor_manager_.SetVisible(false);
+}
+
bool BlimpContentsImpl::HasObserver(BlimpContentsObserver* observer) {
return observers_.HasObserver(observer);
}
« no previous file with comments | « blimp/client/core/contents/blimp_contents_impl.h ('k') | blimp/client/core/contents/blimp_contents_impl_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698