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

Unified Diff: examples/sample_app/sample_app.cc

Issue 940293003: Add a Display and ContextProvider concept to mojom, use to recreate (Closed) Base URL: git@github.com:domokit/mojo.git@master
Patch Set: Created 5 years, 10 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: examples/sample_app/sample_app.cc
diff --git a/examples/sample_app/sample_app.cc b/examples/sample_app/sample_app.cc
index 38ba3c00a1f564cf82216a69a3ef313968289ec0..b0aceaa41245cdfb1e50d806a51f6183a54a65db 100644
--- a/examples/sample_app/sample_app.cc
+++ b/examples/sample_app/sample_app.cc
@@ -38,22 +38,22 @@ class SampleApp : public mojo::ApplicationDelegate,
SetEventDispatcher();
- // TODO(jamesr): Should be mojo:gpu_service
- app->ConnectToService("mojo:native_viewport_service", &gpu_service_);
-
mojo::SizePtr size(mojo::Size::New());
size->width = 800;
size->height = 600;
- viewport_->Create(size.Pass(),
- base::Bind(&SampleApp::OnCreatedNativeViewport,
- base::Unretained(this)));
+ viewport_->Create(size.Clone(), base::Bind(&SampleApp::OnMetricsChanged,
+ base::Unretained(this)));
viewport_->Show();
+ mojo::ContextProviderPtr onscreen_context_provider;
+ viewport_->GetContextProvider(GetProxy(&onscreen_context_provider));
+
+ gles2_client_.reset(new GLES2ClientImpl(onscreen_context_provider.Pass()));
+ gles2_client_->SetSize(*size);
}
void OnMetricsChanged(mojo::ViewportMetricsPtr metrics) {
assert(metrics);
- if (gles2_client_)
- gles2_client_->SetSize(*metrics->size);
+ gles2_client_->SetSize(*metrics->size);
viewport_->RequestMetrics(
base::Bind(&SampleApp::OnMetricsChanged, base::Unretained(this)));
}
@@ -73,26 +73,12 @@ class SampleApp : public mojo::ApplicationDelegate,
viewport_->SetEventDispatcher(ptr.Pass());
}
- void OnCreatedNativeViewport(uint64_t native_viewport_id,
- mojo::ViewportMetricsPtr metrics) {
- mojo::ViewportParameterListenerPtr listener;
- mojo::CommandBufferPtr command_buffer;
- // TODO(jamesr): Output to a surface instead.
- gpu_service_->CreateOnscreenGLES2Context(
- native_viewport_id, metrics->size.Clone(), GetProxy(&command_buffer),
- listener.Pass());
- gles2_client_.reset(new GLES2ClientImpl(command_buffer.Pass()));
- gles2_client_->SetSize(*metrics->size);
- viewport_->RequestMetrics(
- base::Bind(&SampleApp::OnMetricsChanged, base::Unretained(this)));
- }
-
// ErrorHandler implementation.
void OnConnectionError() override { mojo::RunLoop::current()->Quit(); }
scoped_ptr<GLES2ClientImpl> gles2_client_;
mojo::NativeViewportPtr viewport_;
- mojo::GpuPtr gpu_service_;
+ mojo::ContextProviderPtr onscreen_context_provider_;
mojo::Binding<NativeViewportEventDispatcher> dispatcher_binding_;
DISALLOW_COPY_AND_ASSIGN(SampleApp);

Powered by Google App Engine
This is Rietveld 408576698