Prevent appearing INVALID position ordinal in sync.

chrome/browser/ui/app_list/extension_app_model_builder_unittest.cc

Index: chrome/browser/ui/app_list/extension_app_model_builder_unittest.cc
diff --git a/chrome/browser/ui/app_list/extension_app_model_builder_unittest.cc b/chrome/browser/ui/app_list/extension_app_model_builder_unittest.cc
index 71711bbafeee5eee2a99d4c5719b14e12520a292..d21cfebd59e4c6a11548f5abc96413d50c389943 100644
--- a/chrome/browser/ui/app_list/extension_app_model_builder_unittest.cc
+++ b/chrome/browser/ui/app_list/extension_app_model_builder_unittest.cc
@@ -72,7 +72,7 @@ scoped_refptr<extensions::Extension> MakeApp(const std::string& name,
   return app;
 }
 
-const char kDefaultApps[] = "Hosted App,Packaged App 1,Packaged App 2";
+const char kDefaultApps[] = "Packaged App 1,Packaged App 2,Hosted App";

[khmel, 2017/01/04 23:28:12]

chrome/test/data/extensions/app_list/Preferences contains 't' for page_ordinal
for "Hosted App" and 'n' for Packaged App *. So new order looks correct.
Alternative fix to change 't' in file above to something < 'n'

 const size_t kDefaultAppCount = 3u;
 
 }  // namespace
@@ -183,7 +183,7 @@ TEST_F(ExtensionAppModelBuilderTest, Uninstall) {
                                extensions::UNINSTALL_REASON_FOR_TESTING,
                                base::Bind(&base::DoNothing),
                                NULL);
-  EXPECT_EQ(std::string("Hosted App,Packaged App 1"),
+  EXPECT_EQ(std::string("Packaged App 1,Hosted App"),
             GetModelContent(model_.get()));
 
   base::RunLoop().RunUntilIdle();
@@ -201,7 +201,7 @@ TEST_F(ExtensionAppModelBuilderTest, UninstallTerminatedApp) {
                                extensions::UNINSTALL_REASON_FOR_TESTING,
                                base::Bind(&base::DoNothing),
                                NULL);
-  EXPECT_EQ(std::string("Hosted App,Packaged App 1"),
+  EXPECT_EQ(std::string("Packaged App 1,Hosted App"),
             GetModelContent(model_.get()));
 
   base::RunLoop().RunUntilIdle();