chromeos: Update note-taking apps when ARC-enabled profile added.

chrome/browser/chromeos/note_taking_helper_unittest.cc

 // Copyright 2016 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/chromeos/note_taking_helper.h"
 
 #include <utility>
 
 #include "ash/common/ash_switches.h"
 #include "base/bind.h"
 #include "base/command_line.h"
 #include "base/files/file_path.h"
 #include "base/macros.h"
+#include "base/memory/ptr_util.h"
 #include "base/run_loop.h"
 #include "base/strings/stringprintf.h"
+#include "base/strings/utf_string_conversions.h"
 #include "base/test/histogram_tester.h"
 #include "chrome/browser/chrome_notification_types.h"
 #include "chrome/browser/chromeos/file_manager/path_util.h"
 #include "chrome/browser/extensions/extension_service.h"
 #include "chrome/browser/extensions/test_extension_system.h"
+#include "chrome/browser/prefs/browser_prefs.h"
 #include "chrome/browser/ui/app_list/arc/arc_app_test.h"
 #include "chrome/common/pref_names.h"
 #include "chrome/test/base/browser_with_test_window_test.h"
 #include "chrome/test/base/testing_browser_process.h"
 #include "chrome/test/base/testing_profile_manager.h"
 #include "chromeos/dbus/dbus_thread_manager.h"
 #include "chromeos/dbus/fake_session_manager_client.h"
 #include "components/arc/arc_bridge_service.h"
 #include "components/arc/arc_service_manager.h"
 #include "components/arc/common/intent_helper.mojom.h"
 #include "components/arc/test/fake_intent_helper_instance.h"
 #include "components/crx_file/id_util.h"
+#include "components/sync_preferences/testing_pref_service_syncable.h"
 #include "content/public/browser/notification_service.h"
 #include "content/public/browser/notification_source.h"
 #include "extensions/common/api/app_runtime.h"
 #include "extensions/common/extension_builder.h"
 #include "extensions/common/extension_id.h"
 #include "extensions/common/value_builder.h"
 #include "url/gurl.h"
 
 namespace app_runtime = extensions::api::app_runtime;
 
@@ skipping 435 matching lines @@
   EXPECT_FALSE(helper()->android_apps_received());
 
   // After the callback to receive intent handlers has run, the "apps received"
   // member should be updated (even if there aren't any apps).
   helper()->OnIntentFiltersUpdated();
   base::RunLoop().RunUntilIdle();
   EXPECT_TRUE(helper()->android_enabled());
   EXPECT_TRUE(helper()->android_apps_received());
 }
 
-TEST_F(NoteTakingHelperTest, ListAndroidApps) {
+TEST_F(NoteTakingHelperTest, AddProfileWithPlayStoreEnabled) {
+  Init(ENABLE_PALETTE);
+  EXPECT_FALSE(helper()->play_store_enabled());
+  EXPECT_FALSE(helper()->android_apps_received());
+
+  TestObserver observer;
+  ASSERT_EQ(0, observer.num_updates());
+
+  // Add a second profile with the ARC-enabled pref already set. The Play Store
+  // should be immediately regarded as being enabled and the observer should be
+  // notified, since OnArcPlayStoreEnabledChanged() apparently isn't called in
+  // this case: http://crbug.com/700554
+  const char kSecondProfileName[] = "second-profile";
+  auto prefs = base::MakeUnique<sync_preferences::TestingPrefServiceSyncable>();
+  chrome::RegisterUserProfilePrefs(prefs->registry());
+  prefs->SetBoolean(prefs::kArcEnabled, true);
+  profile_manager_->CreateTestingProfile(
+      kSecondProfileName, std::move(prefs), base::ASCIIToUTF16("Second User"),
+      1 /* avatar_id */, std::string() /* supervised_user_id */,
+      TestingProfile::TestingFactories());
+  EXPECT_TRUE(helper()->play_store_enabled());
+  EXPECT_FALSE(helper()->android_apps_received());
+  EXPECT_EQ(1, observer.num_updates());
+
+  // Notification of updated intent filters should result in the apps being
+  // refreshed.
+  helper()->OnIntentFiltersUpdated();
+  base::RunLoop().RunUntilIdle();
+  EXPECT_TRUE(helper()->play_store_enabled());
+  EXPECT_TRUE(helper()->android_apps_received());
+  EXPECT_EQ(2, observer.num_updates());
+
+  profile_manager_->DeleteTestingProfile(kSecondProfileName);
+}
+
+TEST_P(NoteTakingHelperTest, ListAndroidApps) {
   // Add two Android apps.
   std::vector<IntentHandlerInfoPtr> handlers;
   const std::string kName1 = "App 1";
   const std::string kPackage1 = "org.chromium.package1";
   handlers.emplace_back(CreateIntentHandlerInfo(kName1, kPackage1));
   const std::string kName2 = "App 2";
   const std::string kPackage2 = "org.chromium.package2";
   handlers.emplace_back(CreateIntentHandlerInfo(kName2, kPackage2));
   intent_helper_.SetIntentHandlers(NoteTakingHelper::kIntentAction,
                                    std::move(handlers));
@@ skipping 189 matching lines @@
   InitExtensionService(second_profile);
   EXPECT_EQ(0, observer.num_updates());
   InstallExtension(keep_extension.get(), second_profile);
   EXPECT_EQ(1, observer.num_updates());
   UninstallExtension(keep_extension.get(), second_profile);
   EXPECT_EQ(2, observer.num_updates());
   profile_manager_->DeleteTestingProfile(kSecondProfileName);
 }
 
 }  // namespace chromeos