Skip to content
Snippets Groups Projects
Commit 6d60e9b6 authored by tony@chromium.org's avatar tony@chromium.org
Browse files

Remove the intializaton of the Download Manager when bringing up

the options dialog.  This was done a long long time ago because
we used to register the download dir pref in the Download Manager
constructor.  Since that's no longer the case, we don't need to
intialize the Download Manager.

Review URL: http://codereview.chromium.org/208021


git-svn-id: svn://svn.chromium.org/chrome/trunk/src@26583 0039d316-1c4b-4281-b951-d872f2087c98
parent bfb7ac75
No related branches found
No related tags found
No related merge requests found
......@@ -81,9 +81,6 @@ OptionsWindowGtk::OptionsWindowGtk(Profile* profile)
general_page_(profile_),
content_page_(profile_),
advanced_page_(profile_) {
// The download manager needs to be initialized before the contents of the
// Options Window are created.
profile_->GetDownloadManager();
// We don't need to observe changes in this value.
last_selected_page_.Init(prefs::kOptionsWindowLastTabIndex,
g_browser_process->local_state(), NULL);
......
......@@ -89,9 +89,6 @@ OptionsWindowView::OptionsWindowView(Profile* profile)
// the record comes from the original profile, but we explicitly use
// the original profile to avoid potential problems.
: profile_(profile->GetOriginalProfile()) {
// The download manager needs to be initialized before the contents of the
// Options Window are created.
profile_->GetDownloadManager();
// We don't need to observe changes in this value.
last_selected_page_.Init(prefs::kOptionsWindowLastTabIndex,
g_browser_process->local_state(), NULL);
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment