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

Unified Diff: chrome/browser/chromeos/extensions/default_app_order_unittest.cc

Issue 216763004: OEM folder name support for apps installed from rootfs (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: rebase Created 6 years, 8 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
Index: chrome/browser/chromeos/extensions/default_app_order_unittest.cc
diff --git a/chrome/browser/chromeos/extensions/default_app_order_unittest.cc b/chrome/browser/chromeos/extensions/default_app_order_unittest.cc
index 39b7aab736d564d6cb6f283cef3a7c4cd8aace78..721fa477a8dc91a2f8630b09812c8407d428ee77 100644
--- a/chrome/browser/chromeos/extensions/default_app_order_unittest.cc
+++ b/chrome/browser/chromeos/extensions/default_app_order_unittest.cc
@@ -79,7 +79,9 @@ TEST_F(DefaultAppOrderTest, BuiltInDefault) {
// Tests external order file overrides built-in default.
TEST_F(DefaultAppOrderTest, ExternalOrder) {
- const char kExternalOrder[] = "[\"app1\",\"app2\",\"app3\"]";
+ const char kExternalOrder[] = "[\"app1\",\"app2\",\"app3\","
+ "{ \"oem_apps_folder\": true,\"localized_content\": {"
+ " \"default\": {\"name\": \"OEM name\"}}}]";
CreateExternalOrderFile(std::string(kExternalOrder));
scoped_ptr<default_app_order::ExternalLoader> loader(
@@ -91,6 +93,7 @@ TEST_F(DefaultAppOrderTest, ExternalOrder) {
EXPECT_EQ(std::string("app1"), apps[0]);
EXPECT_EQ(std::string("app2"), apps[1]);
EXPECT_EQ(std::string("app3"), apps[2]);
+ EXPECT_EQ(std::string("OEM name"), default_app_order::GetOemAppsFolderName());
}
// Tests none-existent order file gives built-in default.

Powered by Google App Engine
This is Rietveld 408576698