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

Unified Diff: mojo/services/catalog/catalog.cc

Issue 1850623004: Create .mojo applications in a common subdir under out/Debug (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: . Created 4 years, 9 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/public/mojo_constants.gni ('k') | mojo/shell/runner/host/child_process_host.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: mojo/services/catalog/catalog.cc
diff --git a/mojo/services/catalog/catalog.cc b/mojo/services/catalog/catalog.cc
index 35af6b0428d0ee2b44e583bc2f4c938910f01eb2..6864f8e4cc3984a25743efd00f46cb912c6b2bd6 100644
--- a/mojo/services/catalog/catalog.cc
+++ b/mojo/services/catalog/catalog.cc
@@ -23,8 +23,10 @@ base::FilePath GetManifestPath(const base::FilePath& package_dir,
// TODO(beng): think more about how this should be done for exe targets.
std::string type = mojo::GetNameType(name);
std::string path = mojo::GetNamePath(name);
- if (type == mojo::kNameType_Mojo)
- return package_dir.AppendASCII(path + "/manifest.json");
+ if (type == mojo::kNameType_Mojo) {
+ return package_dir.AppendASCII("Mojo Applications").AppendASCII(
+ path + "/manifest.json");
+ }
if (type == mojo::kNameType_Exe)
return package_dir.AppendASCII(path + "_manifest.json");
return base::FilePath();
@@ -36,7 +38,8 @@ base::FilePath GetPackagePath(const base::FilePath& package_dir,
if (type == mojo::kNameType_Mojo) {
// It's still a mojo: URL, use the default mapping scheme.
const std::string host = mojo::GetNamePath(name);
- return package_dir.AppendASCII(host + "/" + host + ".mojo");
+ return package_dir.AppendASCII("Mojo Applications").AppendASCII(
+ host + "/" + host + ".mojo");
}
if (type == mojo::kNameType_Exe) {
#if defined OS_WIN
« no previous file with comments | « mojo/public/mojo_constants.gni ('k') | mojo/shell/runner/host/child_process_host.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698