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

Unified Diff: components/resource_provider/file_utils.cc

Issue 1743473002: Change Mojo URLs to structured names (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@18collapse
Patch Set: . Created 4 years, 10 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 | « components/resource_provider/file_utils.h ('k') | components/resource_provider/file_utils_unittest.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: components/resource_provider/file_utils.cc
diff --git a/components/resource_provider/file_utils.cc b/components/resource_provider/file_utils.cc
index 1b740369441556126a897a9d4e5fe73345ad5f2f..9c3948eccef1b48cf23845948f82c0ef271cfcf1 100644
--- a/components/resource_provider/file_utils.cc
+++ b/components/resource_provider/file_utils.cc
@@ -29,11 +29,8 @@ bool IsPathNameValid(const std::string& name) {
} // namespace
-base::FilePath GetPathForApplicationUrl(const std::string& application_url) {
- // We don't want to use GURL because it can behave differently depending on
- // whether mojo:// has been registered as a standard scheme or not. Also, we
- // can get mojo:foo or mojo://foo urls here.
- std::string path = application_url;
+base::FilePath GetPathForApplicationName(const std::string& application_name) {
+ std::string path = application_name;
const bool is_mojo =
base::StartsWith(path, "mojo:", base::CompareCase::INSENSITIVE_ASCII);
const bool is_exe =
« no previous file with comments | « components/resource_provider/file_utils.h ('k') | components/resource_provider/file_utils_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698