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

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

Issue 331323007: Eliminate Launchable. Use Navigator instead. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: . 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 a96becea6e640b0c835265b296afa09e7c10eb60..52189e1bd2dfdbbfb11f39c45f200abebd766376 100644
--- a/mojo/examples/window_manager/window_manager.cc
+++ b/mojo/examples/window_manager/window_manager.cc
@@ -137,7 +137,13 @@ class WindowManager : public Application,
size_t index = node->children().size() - 1;
details->url = base::StringPrintf(
"%s/%x", kEmbeddedAppURL, kColors[index % arraysize(kColors)]);
- navigator->Navigate(embedded->id(), details.Pass());
+
+ // TODO(beng): remove once nullable parameters land.
+ navigation::ResponseDetailsPtr response_details(
+ navigation::ResponseDetails::New());
+ navigator->Navigate(embedded->id(),
+ details.Pass(),
+ response_details.Pass());
}
}
« 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