ChromeOS: Per-user time zone: refactor tests first.

chrome/browser/extensions/test_extension_system.h

 // 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.
 
 #ifndef CHROME_BROWSER_EXTENSIONS_TEST_EXTENSION_SYSTEM_H_
 #define CHROME_BROWSER_EXTENSIONS_TEST_EXTENSION_SYSTEM_H_
 
 #include <memory>
 
 #include "extensions/browser/extension_system.h"
 #include "extensions/common/one_shot_event.h"
 
+#if defined(OS_CHROMEOS)
+#include "chrome/browser/chromeos/login/users/scoped_test_user_manager.h"
+#endif
+
 class Profile;
 class TestingValueStore;
 
 namespace base {
 class CommandLine;
 class FilePath;
 }
 
 namespace content {
 class BrowserContext;
@@ skipping 61 matching lines @@
  private:
   std::unique_ptr<StateStore> state_store_;
   scoped_refptr<TestValueStoreFactory> store_factory_;
   std::unique_ptr<ManagementPolicy> management_policy_;
   std::unique_ptr<RuntimeData> runtime_data_;
   std::unique_ptr<ExtensionService> extension_service_;
   scoped_refptr<InfoMap> info_map_;
   std::unique_ptr<QuotaService> quota_service_;
   std::unique_ptr<AppSorting> app_sorting_;
   OneShotEvent ready_;
+
+#if defined(OS_CHROMEOS)
+  std::unique_ptr<chromeos::ScopedTestUserManager> test_user_manager_;
+#endif

[stevenjb, 2017/05/25 21:20:24]

I think instead the extensions team may prefer a derived TestExtensionSystemCrOS
that includes this.

[Alexander Alekseev, 2017/05/26 02:18:01]

This sounds a bit too complicate. We will need to modify behavior for the all
tests that use this object, won't we?

 };
 
 }  // namespace extensions
 
 #endif  // CHROME_BROWSER_EXTENSIONS_TEST_EXTENSION_SYSTEM_H_