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

Unified Diff: mojo/services/view_manager/public/cpp/lib/view_manager_client_impl.cc

Issue 830593003: Update mojo sdk to rev 9fbbc4f0fef1187312316c0ed992342474e139f1 (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: cherry-pick mojo 9d3b8dd17f12d20035a14737fdc38dd926890ff8 Created 5 years, 11 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: mojo/services/view_manager/public/cpp/lib/view_manager_client_impl.cc
diff --git a/mojo/services/view_manager/public/cpp/lib/view_manager_client_impl.cc b/mojo/services/view_manager/public/cpp/lib/view_manager_client_impl.cc
index e210c2bc42882e784369d82cf1c8c9beaf0c037d..ef043e0147e7aa51560f35fb2f531f048d04efe1 100644
--- a/mojo/services/view_manager/public/cpp/lib/view_manager_client_impl.cc
+++ b/mojo/services/view_manager/public/cpp/lib/view_manager_client_impl.cc
@@ -2,7 +2,7 @@
// Use of this source code is governed by a BSD-style license that can be
// found in the LICENSE file.
-#include "mojo/services/view_manager/public/cpp/lib/view_manager_client_impl.h"
+#include "view_manager/public/cpp/lib/view_manager_client_impl.h"
#include "base/bind.h"
#include "base/message_loop/message_loop.h"
@@ -12,10 +12,10 @@
#include "mojo/public/cpp/application/service_provider_impl.h"
#include "mojo/public/interfaces/application/service_provider.mojom.h"
#include "mojo/public/interfaces/application/shell.mojom.h"
-#include "mojo/services/view_manager/public/cpp/lib/view_private.h"
-#include "mojo/services/view_manager/public/cpp/util.h"
-#include "mojo/services/view_manager/public/cpp/view_manager_delegate.h"
-#include "mojo/services/view_manager/public/cpp/view_observer.h"
+#include "view_manager/public/cpp/lib/view_private.h"
+#include "view_manager/public/cpp/util.h"
+#include "view_manager/public/cpp/view_manager_delegate.h"
+#include "view_manager/public/cpp/view_observer.h"
namespace mojo {
@@ -36,6 +36,7 @@ View* AddViewToViewManager(ViewManagerClientImpl* client,
private_view.set_id(view_data->view_id);
private_view.set_visible(view_data->visible);
private_view.set_drawn(view_data->drawn);
+ private_view.set_viewport_metrics(view_data->viewport_metrics.Pass());
private_view.set_properties(
view_data->properties.To<std::map<std::string, std::vector<uint8_t>>>());
client->AddView(view);
@@ -274,12 +275,11 @@ void ViewManagerClientImpl::OnEmbed(
ServiceProviderImpl* exported_services = nullptr;
scoped_ptr<ServiceProvider> remote;
- if (parent_services.is_pending()) {
- // BindToRequest() binds the lifetime of |exported_services| to the pipe.
- exported_services = new ServiceProviderImpl;
- BindToRequest(exported_services, &parent_services);
- remote.reset(exported_services->CreateRemoteServiceProvider());
- }
+ // BindToRequest() binds the lifetime of |exported_services| to the pipe.
+ exported_services = new ServiceProviderImpl;
+ BindToRequest(exported_services, &parent_services);
+ remote.reset(exported_services->CreateRemoteServiceProvider());
+
window_manager_.Bind(window_manager_pipe.Pass());
window_manager_.set_client(this);
// base::Unretained() is safe here as |window_manager_| is bound to our

Powered by Google App Engine
This is Rietveld 408576698