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

Side by Side Diff: chrome/browser/profiles/profile_impl.cc

Issue 6040005: More cleanup of DefaultApps code (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: remove unnecessary change, add comments Created 9 years, 11 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 unified diff | Download patch | Annotate | Revision Log
OLDNEW
1 // Copyright (c) 2010 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2010 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "chrome/browser/profiles/profile_impl.h" 5 #include "chrome/browser/profiles/profile_impl.h"
6 6
7 #include "app/resource_bundle.h" 7 #include "app/resource_bundle.h"
8 #include "base/command_line.h" 8 #include "base/command_line.h"
9 #include "base/environment.h" 9 #include "base/environment.h"
10 #include "base/file_path.h" 10 #include "base/file_path.h"
(...skipping 391 matching lines...) Expand 10 before | Expand all | Expand 10 after
402 402
403 std::string manifest = 403 std::string manifest =
404 ResourceBundle::GetSharedInstance().GetRawDataResource( 404 ResourceBundle::GetSharedInstance().GetRawDataResource(
405 iter->second).as_string(); 405 iter->second).as_string();
406 extensions_service_->register_component_extension( 406 extensions_service_->register_component_extension(
407 ExtensionService::ComponentExtensionInfo(manifest, path)); 407 ExtensionService::ComponentExtensionInfo(manifest, path));
408 } 408 }
409 } 409 }
410 410
411 void ProfileImpl::InstallDefaultApps() { 411 void ProfileImpl::InstallDefaultApps() {
412 // The web store only supports en-US at the moment, so we don't install 412 ExtensionService* extension_service = GetExtensionService();
413 // default apps in other locales. 413 DefaultApps* default_apps = extension_service->default_apps();
414 if (g_browser_process->GetApplicationLocale() != "en-US") 414
415 if (!default_apps->ShouldInstallDefaultApps(extension_service->GetAppIds()))
415 return; 416 return;
416 417
417 ExtensionService* extensions_service = GetExtensionService(); 418 const ExtensionIdSet& app_ids = default_apps->default_apps();
418 const ExtensionIdSet* app_ids = 419 for (ExtensionIdSet::const_iterator iter = app_ids.begin();
419 extensions_service->default_apps()->GetAppsToInstall(); 420 iter != app_ids.end(); ++iter) {
420 if (!app_ids) 421 extension_service->AddPendingExtensionFromDefaultAppList(*iter);
421 return;
422
423 for (ExtensionIdSet::const_iterator iter = app_ids->begin();
424 iter != app_ids->end(); ++iter) {
425 extensions_service->AddPendingExtensionFromDefaultAppList(*iter);
426 } 422 }
427 } 423 }
428 424
429 void ProfileImpl::InitWebResources() { 425 void ProfileImpl::InitWebResources() {
430 if (web_resource_service_) 426 if (web_resource_service_)
431 return; 427 return;
432 428
433 web_resource_service_ = new WebResourceService(this); 429 web_resource_service_ = new WebResourceService(this);
434 web_resource_service_->StartAfterDelay(); 430 web_resource_service_->StartAfterDelay();
435 } 431 }
(...skipping 927 matching lines...) Expand 10 before | Expand all | Expand 10 after
1363 } 1359 }
1364 1360
1365 PrerenderManager* ProfileImpl::GetPrerenderManager() { 1361 PrerenderManager* ProfileImpl::GetPrerenderManager() {
1366 CommandLine* cl = CommandLine::ForCurrentProcess(); 1362 CommandLine* cl = CommandLine::ForCurrentProcess();
1367 if (!cl->HasSwitch(switches::kEnablePagePrerender)) 1363 if (!cl->HasSwitch(switches::kEnablePagePrerender))
1368 return NULL; 1364 return NULL;
1369 if (!prerender_manager_.get()) 1365 if (!prerender_manager_.get())
1370 prerender_manager_.reset(new PrerenderManager(this)); 1366 prerender_manager_.reset(new PrerenderManager(this));
1371 return prerender_manager_.get(); 1367 return prerender_manager_.get();
1372 } 1368 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698