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

Unified Diff: mojo/examples/window_manager/window_manager.cc

Issue 327523004: Introduce very beginning of navigation (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: add url dependency to two smaple apps Created 6 years, 6 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
« no previous file with comments | « mojo/examples/nesting_app/nesting_app.cc ('k') | mojo/mojo_examples.gypi » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: mojo/examples/window_manager/window_manager.cc
diff --git a/mojo/examples/window_manager/window_manager.cc b/mojo/examples/window_manager/window_manager.cc
index d998f0ca4a1860e08433d24606967ad1f6497cb7..e9a561e8cad658ee2a3910829beb53e3b4d71a01 100644
--- a/mojo/examples/window_manager/window_manager.cc
+++ b/mojo/examples/window_manager/window_manager.cc
@@ -4,8 +4,10 @@
#include "base/basictypes.h"
#include "base/bind.h"
+#include "base/strings/stringprintf.h"
#include "mojo/examples/window_manager/window_manager.mojom.h"
#include "mojo/public/cpp/application/application.h"
+#include "mojo/services/navigation/navigation.mojom.h"
#include "mojo/services/public/cpp/view_manager/view.h"
#include "mojo/services/public/cpp/view_manager/view_manager.h"
#include "mojo/services/public/cpp/view_manager/view_manager_delegate.h"
@@ -30,6 +32,19 @@ namespace examples {
class WindowManager;
+namespace {
+
+const SkColor kColors[] = { SK_ColorYELLOW,
+ SK_ColorRED,
+ SK_ColorGREEN,
+ SK_ColorMAGENTA };
+
+const char kEmbeddedAppURL[] = "mojo:mojo_embedded_app";
+const char kNestingAppURL[] = "mojo:mojo_nesting_app";
+const char kMojoBrowserURL[] = "mojo:mojo_browser";
+
+} // namespace
+
class WindowManagerConnection : public InterfaceImpl<IWindowManager> {
public:
explicit WindowManagerConnection(WindowManager* window_manager)
@@ -69,11 +84,11 @@ class WindowManager : public Application,
virtual void OnViewInputEvent(View* view, const EventPtr& event) OVERRIDE {
if (event->action == ui::ET_MOUSE_RELEASED) {
if (event->flags & ui::EF_LEFT_MOUSE_BUTTON)
- CreateWindow("mojo:mojo_embedded_app");
+ CreateWindow(kEmbeddedAppURL);
else if (event->flags & ui::EF_RIGHT_MOUSE_BUTTON)
- CreateWindow("mojo:mojo_nesting_app");
+ CreateWindow(kNestingAppURL);
else if (event->flags & ui::EF_MIDDLE_MOUSE_BUTTON)
- CreateWindow("mojo:mojo_browser");
+ CreateWindow(kMojoBrowserURL);
}
}
@@ -94,7 +109,7 @@ class WindowManager : public Application,
view->AddObserver(this);
}
- void CreateWindow(const String& url) {
+ void CreateWindow(const std::string& url) {
ViewTreeNode* node = view_manager_->GetNodeById(parent_node_id_);
gfx::Rect bounds(50, 50, 200, 200);
@@ -108,6 +123,23 @@ class WindowManager : public Application,
node->AddChild(embedded);
embedded->SetBounds(bounds);
embedded->Embed(url);
+
+ // TODO(aa): Is there a way to ask for an interface and test whether it
+ // succeeded? That would be nicer than hard-coding the URLs that are known
+ // to support navigation.
+ if (url == kEmbeddedAppURL || url == kNestingAppURL) {
+ // TODO(aa): This means that there can only ever be one instance of every
+ // app, which seems wrong. Instead, perhaps embedder should get back a
+ // service provider that allows it to talk to embeddee.
+ navigation::NavigatorPtr navigator;
+ ConnectTo(url, &navigator);
+ navigation::NavigationDetailsPtr details(
+ navigation::NavigationDetails::New());
+ size_t index = node->children().size() - 1;
+ details->url = base::StringPrintf(
+ "%s/%x", kEmbeddedAppURL, kColors[index % arraysize(kColors)]);
+ navigator->Navigate(embedded->id(), details.Pass());
+ }
}
ViewManager* view_manager_;
« no previous file with comments | « mojo/examples/nesting_app/nesting_app.cc ('k') | mojo/mojo_examples.gypi » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698