Add support for mapping node tree on the client.

Add support for mapping node tree on the client.

R=sky@chromium.org
http://crbug.com/365012

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=268218

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=268252

[sky, 2014-05-02 23:05:08]

LGTM

https://codereview.chromium.org/260863008/diff/80001/mojo/services/public/cpp...
File mojo/services/public/cpp/view_manager/lib/view_manager_private.h (right):

https://codereview.chromium.org/260863008/diff/80001/mojo/services/public/cpp...
mojo/services/public/cpp/view_manager/lib/view_manager_private.h:29: void
set_tree(ViewTreeNode* root) { manager_->tree_.reset(root); }
scoped_ptr?

https://codereview.chromium.org/260863008/diff/80001/mojo/services/public/cpp...
mojo/services/public/cpp/view_manager/lib/view_manager_private.h:32: bool
connected() { return manager_->synchronizer_->connected(); }
const?

[Ben Goodger (Google), 2014-05-03 05:08:58]

The CQ bit was checked by ben@chromium.org

[commit-bot: I haz the power, 2014-05-03 05:09:06]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/ben@chromium.org/260863008/100001

[commit-bot: I haz the power, 2014-05-03 05:09:12]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-05-03 05:09:12]

Failed to apply patch for
mojo/services/public/cpp/view_manager/lib/view_manager.cc:
While running patch -p1 --forward --force --no-backup-if-mismatch;
  patching file mojo/services/public/cpp/view_manager/lib/view_manager.cc
  Hunk #1 FAILED at 1.
  1 out of 1 hunk FAILED -- saving rejects to file
mojo/services/public/cpp/view_manager/lib/view_manager.cc.rej

Patch:       mojo/services/public/cpp/view_manager/lib/view_manager.cc
Index: mojo/services/public/cpp/view_manager/lib/view_manager.cc
diff --git a/mojo/services/public/cpp/view_manager/lib/view_manager.cc
b/mojo/services/public/cpp/view_manager/lib/view_manager.cc
index
7fd82e7617a8bf39ae332593d50a167b0cb15912..3c45538281b797933354e65c26532d461a0fe9b2
100644
--- a/mojo/services/public/cpp/view_manager/lib/view_manager.cc
+++ b/mojo/services/public/cpp/view_manager/lib/view_manager.cc
@@ -1,20 +1,32 @@
 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
-
-#include "mojo/services/public/cpp/view_manager/view_manager.h"
-
-#include
"mojo/services/public/cpp/view_manager/lib/view_manager_synchronizer.h"
-
-namespace mojo {
-namespace services {
-namespace view_manager {
-
-ViewManager::ViewManager(Shell* shell)
-    : shell_(shell),
-      synchronizer_(new ViewManagerSynchronizer(this)) {}
-ViewManager::~ViewManager() {}
-
-}  // namespace view_manager
-}  // namespace services
-}  // namespace mojo
+
+#include "mojo/services/public/cpp/view_manager/view_manager.h"
+
+#include
"mojo/services/public/cpp/view_manager/lib/view_manager_synchronizer.h"
+
+namespace mojo {
+namespace services {
+namespace view_manager {
+
+ViewManager::ViewManager(Shell* shell)
+    : shell_(shell),
+      synchronizer_(new ViewManagerSynchronizer(this)) {}
+ViewManager::~ViewManager() {}
+
+void ViewManager::BuildNodeTree(const mojo::Callback<void()>& callback) {
+  synchronizer_->BuildNodeTree(callback);
+}
+
+void ViewManager::AddObserver(ViewManagerObserver* observer) {
+  observers_.AddObserver(observer);
+}
+
+void ViewManager::RemoveObserver(ViewManagerObserver* observer) {
+  observers_.RemoveObserver(observer);
+}
+
+}  // namespace view_manager
+}  // namespace services
+}  // namespace mojo

[Ben Goodger (Google), 2014-05-05 18:10:01]

Committed patchset #6 manually as r268218 (presubmit successful).

[Ben Goodger (Google), 2014-05-05 19:44:14]

Committed patchset #7 manually as r268252 (presubmit successful).