arc: Make Play Store item persistance in shelf.

chrome/browser/policy/policy_browsertest.cc

 // Copyright (c) 2012 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 <stddef.h>
 #include <stdint.h>
 
 #include <algorithm>
 #include <memory>
 #include <string>
@@ skipping 652 matching lines @@
  protected:
   PolicyTest() {}
   ~PolicyTest() override {}
 
   void SetUp() override {
     test_extension_cache_.reset(new extensions::ExtensionCacheFake());
     InProcessBrowserTest::SetUp();
   }
 
   void SetUpInProcessBrowserTestFixture() override {
-    base::CommandLine::ForCurrentProcess()->AppendSwitch("noerrdialogs");
+    base::CommandLine* const command_line =
+        base::CommandLine::ForCurrentProcess();
+    command_line->AppendSwitch("noerrdialogs");
+    command_line->AppendSwitch(chromeos::switches::kEnableArc);

[pastarmovj, 2016/05/17 22:13:05]

please document the behavior and the reason why these switch is needed for this
test suite.

[khmel, 2016/05/17 22:48:35]

Done, and moved it to Arc test, where it looks more logical

     EXPECT_CALL(provider_, IsInitializationComplete(_))
         .WillRepeatedly(Return(true));
     BrowserPolicyConnector::SetPolicyProviderForTesting(&provider_);
   }
 
   void SetUpOnMainThread() override {
     BrowserThread::PostTask(
         BrowserThread::IO, FROM_HERE,
         base::Bind(chrome_browser_net::SetUrlRequestMocksEnabled, true));
     if (extension_service()->updater()) {
@@ skipping 3343 matching lines @@
 
   SetEmptyPolicy();
   // Policy not set.
   CheckSystemTimezoneAutomaticDetectionPolicyUnset();
   EXPECT_TRUE(CheckResolveTimezoneByGeolocation(true, false));
   EXPECT_TRUE(manager->TimeZoneResolverShouldBeRunningForTests());
 }
 #endif  // defined(OS_CHROMEOS)
 
 }  // namespace policy