Add scoped path overrides for system directories for ExtensionBrowserTests.

chrome/browser/extensions/extension_browsertest.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_EXTENSION_BROWSERTEST_H_
 #define CHROME_BROWSER_EXTENSIONS_EXTENSION_BROWSERTEST_H_
 
 #include <string>
 
 #include "base/command_line.h"
 
 #include "base/files/file_path.h"
 #include "base/files/scoped_temp_dir.h"
 #include "chrome/browser/extensions/extension_test_notification_observer.h"
 #include "chrome/browser/profiles/profile.h"
 #include "chrome/browser/ui/browser.h"
 #include "chrome/common/extensions/features/feature_channel.h"
 #include "chrome/test/base/in_process_browser_test.h"
 #include "content/public/browser/web_contents.h"
 #include "extensions/browser/extension_host.h"
 #include "extensions/browser/extension_system.h"
 #include "extensions/common/extension.h"
 #include "extensions/common/feature_switch.h"
 #include "extensions/common/manifest.h"
 
 class ExtensionService;
 class Profile;
 
+namespace base {
+class ScopedPathOverride;
+}
+
 namespace extensions {
 class ExtensionCacheFake;
 class ExtensionSet;
 class ProcessManager;
 }
 
 // Base class for extension browser tests. Provides utilities for loading,
 // unloading, and installing extensions.
 class ExtensionBrowserTest : virtual public InProcessBrowserTest {
  protected:
@@ skipping 329 matching lines @@
       bool wait_for_idle,
       bool is_ephemeral);
 
   // Make the current channel "dev" for the duration of the test.
   extensions::ScopedCurrentChannel current_channel_;
 
   // Disable external install UI.
   extensions::FeatureSwitch::ScopedOverride
       override_prompt_for_external_extensions_;
 
+  // Use mock shortcut directories to ensure app shortcuts are cleaned up.

[Yoyo Zhou, 2014/09/15 18:54:55]

This should probably be in a #if defined(OS_WIN) block (see compile failures on
non-windows).

[calamity, 2014/09/16 00:56:21]

Done.

+  scoped_ptr<base::ScopedPathOverride> user_desktop_override_;

[gab, 2014/09/16 00:01:10]

No need for scoped_ptr, a direct member is sufficient.

[calamity, 2014/09/16 00:56:21]

Done.

+  scoped_ptr<base::ScopedPathOverride> common_desktop_override_;
+  scoped_ptr<base::ScopedPathOverride> user_quick_launch_override_;
+  scoped_ptr<base::ScopedPathOverride> start_menu_override_;
+  scoped_ptr<base::ScopedPathOverride> common_start_menu_override_;
+
   // The default profile to be used.
   Profile* profile_;
 
   // Cache cache implementation.
   scoped_ptr<extensions::ExtensionCacheFake> test_extension_cache_;
 };
 
 #endif  // CHROME_BROWSER_EXTENSIONS_EXTENSION_BROWSERTEST_H_