Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1522)

Unified Diff: chrome/browser/profiles/profile_manager_unittest.cc

Issue 10823217: Create/Delete windows profile shortcuts (Closed) Base URL: https://src.chromium.org/chrome/trunk/src/
Patch Set: Created 8 years, 4 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/profiles/profile_manager_unittest.cc
===================================================================
--- chrome/browser/profiles/profile_manager_unittest.cc (revision 150342)
+++ chrome/browser/profiles/profile_manager_unittest.cc (working copy)
@@ -23,12 +23,15 @@
#include "chrome/browser/profiles/profile.h"
#include "chrome/browser/profiles/profile_info_cache.h"
#include "chrome/browser/profiles/profile_manager.h"
+#include "chrome/browser/profiles/profile_shortcut_manager.h"
#include "chrome/browser/ui/browser.h"
#include "chrome/common/chrome_constants.h"
#include "chrome/common/chrome_notification_types.h"
#include "chrome/common/chrome_paths.h"
#include "chrome/common/chrome_switches.h"
#include "chrome/common/pref_names.h"
+#include "chrome/installer/util/browser_distribution.h"
+#include "chrome/installer/util/shell_util.h"
#include "chrome/test/base/test_browser_window.h"
#include "chrome/test/base/testing_browser_process.h"
#include "chrome/test/base/testing_pref_service.h"
@@ -37,6 +40,7 @@
#include "content/public/test/test_browser_thread.h"
#include "testing/gmock/include/gmock/gmock.h"
#include "testing/gtest/include/gtest/gtest.h"
+#include "ui/base/resource/resource_bundle.h"
#if defined(OS_CHROMEOS)
#include "chrome/browser/chromeos/cros/cros_library.h"
@@ -77,13 +81,6 @@
return new TestingProfile(path, this);
}
-
-#if defined(OS_WIN)
- virtual ProfileShortcutManagerWin* CreateShortcutManager() OVERRIDE {
- // We should avoid creating shortcuts in these tests.
- return NULL;
- }
-#endif
};
} // namespace testing
@@ -117,8 +114,9 @@
ASSERT_TRUE(temp_dir_.CreateUniqueTempDir());
static_cast<TestingBrowserProcess*>(g_browser_process)->SetProfileManager(
new testing::ProfileManager(temp_dir_.path()));
+
#if defined(OS_CHROMEOS)
- CommandLine *cl = CommandLine::ForCurrentProcess();
+ CommandLine* cl = CommandLine::ForCurrentProcess();
cl->AppendSwitch(switches::kTestType);
#endif
}
@@ -287,13 +285,16 @@
profile_manager->CreateProfileAsync(dest_path,
base::Bind(&MockObserver::OnProfileCreated,
- base::Unretained(&mock_observer1)), string16(), string16());
+ base::Unretained(&mock_observer1)),
+ string16(), string16());
profile_manager->CreateProfileAsync(dest_path,
base::Bind(&MockObserver::OnProfileCreated,
- base::Unretained(&mock_observer2)), string16(), string16());
+ base::Unretained(&mock_observer2)),
+ string16(), string16());
profile_manager->CreateProfileAsync(dest_path,
base::Bind(&MockObserver::OnProfileCreated,
- base::Unretained(&mock_observer3)), string16(), string16());
+ base::Unretained(&mock_observer3)),
+ string16(), string16());
message_loop_.RunAllPending();
}

Powered by Google App Engine
This is Rietveld 408576698