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

Side by Side Diff: chrome/browser/media_galleries/media_galleries_preferences.cc

Issue 481433005: Extensions: Move id_util functions to crx_file component. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: revert last patchset. function returns Extension* and can't use an assert. Created 6 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 unified diff | Download patch | Annotate | Revision Log
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 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/media_galleries/media_galleries_preferences.h" 5 #include "chrome/browser/media_galleries/media_galleries_preferences.h"
6 6
7 #include "base/base_paths_posix.h" 7 #include "base/base_paths_posix.h"
8 #include "base/callback.h" 8 #include "base/callback.h"
9 #include "base/i18n/time_formatting.h" 9 #include "base/i18n/time_formatting.h"
10 #include "base/path_service.h" 10 #include "base/path_service.h"
11 #include "base/prefs/pref_service.h" 11 #include "base/prefs/pref_service.h"
12 #include "base/prefs/scoped_user_pref_update.h" 12 #include "base/prefs/scoped_user_pref_update.h"
13 #include "base/stl_util.h" 13 #include "base/stl_util.h"
14 #include "base/strings/string16.h" 14 #include "base/strings/string16.h"
15 #include "base/strings/string_number_conversions.h" 15 #include "base/strings/string_number_conversions.h"
16 #include "base/strings/utf_string_conversions.h" 16 #include "base/strings/utf_string_conversions.h"
17 #include "base/values.h" 17 #include "base/values.h"
18 #include "chrome/browser/browser_process.h" 18 #include "chrome/browser/browser_process.h"
19 #include "chrome/browser/extensions/api/media_galleries_private/media_galleries_ private_api.h" 19 #include "chrome/browser/extensions/api/media_galleries_private/media_galleries_ private_api.h"
20 #include "chrome/browser/extensions/extension_service.h" 20 #include "chrome/browser/extensions/extension_service.h"
21 #include "chrome/browser/media_galleries/fileapi/iapps_finder.h" 21 #include "chrome/browser/media_galleries/fileapi/iapps_finder.h"
22 #include "chrome/browser/media_galleries/fileapi/picasa_finder.h" 22 #include "chrome/browser/media_galleries/fileapi/picasa_finder.h"
23 #include "chrome/browser/media_galleries/imported_media_gallery_registry.h" 23 #include "chrome/browser/media_galleries/imported_media_gallery_registry.h"
24 #include "chrome/browser/media_galleries/media_file_system_registry.h" 24 #include "chrome/browser/media_galleries/media_file_system_registry.h"
25 #include "chrome/browser/media_galleries/media_galleries_histograms.h" 25 #include "chrome/browser/media_galleries/media_galleries_histograms.h"
26 #include "chrome/browser/profiles/profile.h" 26 #include "chrome/browser/profiles/profile.h"
27 #include "chrome/common/chrome_paths.h" 27 #include "chrome/common/chrome_paths.h"
28 #include "chrome/common/pref_names.h" 28 #include "chrome/common/pref_names.h"
29 #include "components/crx_file/id_util.h"
29 #include "components/pref_registry/pref_registry_syncable.h" 30 #include "components/pref_registry/pref_registry_syncable.h"
30 #include "components/storage_monitor/media_storage_util.h" 31 #include "components/storage_monitor/media_storage_util.h"
31 #include "components/storage_monitor/storage_monitor.h" 32 #include "components/storage_monitor/storage_monitor.h"
32 #include "content/public/browser/browser_thread.h" 33 #include "content/public/browser/browser_thread.h"
33 #include "extensions/browser/extension_prefs.h" 34 #include "extensions/browser/extension_prefs.h"
34 #include "extensions/browser/extension_system.h" 35 #include "extensions/browser/extension_system.h"
35 #include "extensions/browser/pref_names.h" 36 #include "extensions/browser/pref_names.h"
36 #include "extensions/common/extension.h"
37 #include "extensions/common/extension_set.h" 37 #include "extensions/common/extension_set.h"
38 #include "extensions/common/permissions/api_permission.h" 38 #include "extensions/common/permissions/api_permission.h"
39 #include "extensions/common/permissions/media_galleries_permission.h" 39 #include "extensions/common/permissions/media_galleries_permission.h"
40 #include "extensions/common/permissions/permissions_data.h" 40 #include "extensions/common/permissions/permissions_data.h"
41 #include "grit/generated_resources.h" 41 #include "grit/generated_resources.h"
42 #include "ui/base/l10n/l10n_util.h" 42 #include "ui/base/l10n/l10n_util.h"
43 43
44 using base::DictionaryValue; 44 using base::DictionaryValue;
45 using base::ListValue; 45 using base::ListValue;
46 using extensions::ExtensionPrefs; 46 using extensions::ExtensionPrefs;
(...skipping 1144 matching lines...) Expand 10 before | Expand all | Expand 10 after
1191 known_galleries_.find(id)->second.type != 1191 known_galleries_.find(id)->second.type !=
1192 MediaGalleryPrefInfo::kAutoDetected); 1192 MediaGalleryPrefInfo::kAutoDetected);
1193 ExtensionPrefs* prefs = GetExtensionPrefs(); 1193 ExtensionPrefs* prefs = GetExtensionPrefs();
1194 const base::DictionaryValue* extensions = 1194 const base::DictionaryValue* extensions =
1195 prefs->pref_service()->GetDictionary(extensions::pref_names::kExtensions); 1195 prefs->pref_service()->GetDictionary(extensions::pref_names::kExtensions);
1196 if (!extensions) 1196 if (!extensions)
1197 return true; 1197 return true;
1198 1198
1199 for (base::DictionaryValue::Iterator iter(*extensions); !iter.IsAtEnd(); 1199 for (base::DictionaryValue::Iterator iter(*extensions); !iter.IsAtEnd();
1200 iter.Advance()) { 1200 iter.Advance()) {
1201 if (!extensions::Extension::IdIsValid(iter.key())) { 1201 if (!crx_file::id_util::IdIsValid(iter.key())) {
1202 NOTREACHED(); 1202 NOTREACHED();
1203 continue; 1203 continue;
1204 } 1204 }
1205 std::vector<MediaGalleryPermission> permissions = 1205 std::vector<MediaGalleryPermission> permissions =
1206 GetGalleryPermissionsFromPrefs(iter.key()); 1206 GetGalleryPermissionsFromPrefs(iter.key());
1207 for (std::vector<MediaGalleryPermission>::const_iterator it = 1207 for (std::vector<MediaGalleryPermission>::const_iterator it =
1208 permissions.begin(); it != permissions.end(); ++it) { 1208 permissions.begin(); it != permissions.end(); ++it) {
1209 if (it->pref_id == id) { 1209 if (it->pref_id == id) {
1210 if (it->has_permission) 1210 if (it->has_permission)
1211 return true; 1211 return true;
(...skipping 220 matching lines...) Expand 10 before | Expand all | Expand 10 after
1432 MediaGalleryPrefId gallery_id) { 1432 MediaGalleryPrefId gallery_id) {
1433 DCHECK(IsInitialized()); 1433 DCHECK(IsInitialized());
1434 ExtensionPrefs* prefs = GetExtensionPrefs(); 1434 ExtensionPrefs* prefs = GetExtensionPrefs();
1435 const base::DictionaryValue* extensions = 1435 const base::DictionaryValue* extensions =
1436 prefs->pref_service()->GetDictionary(extensions::pref_names::kExtensions); 1436 prefs->pref_service()->GetDictionary(extensions::pref_names::kExtensions);
1437 if (!extensions) 1437 if (!extensions)
1438 return; 1438 return;
1439 1439
1440 for (base::DictionaryValue::Iterator iter(*extensions); !iter.IsAtEnd(); 1440 for (base::DictionaryValue::Iterator iter(*extensions); !iter.IsAtEnd();
1441 iter.Advance()) { 1441 iter.Advance()) {
1442 if (!extensions::Extension::IdIsValid(iter.key())) { 1442 if (!crx_file::id_util::IdIsValid(iter.key())) {
1443 NOTREACHED(); 1443 NOTREACHED();
1444 continue; 1444 continue;
1445 } 1445 }
1446 UnsetGalleryPermissionInPrefs(iter.key(), gallery_id); 1446 UnsetGalleryPermissionInPrefs(iter.key(), gallery_id);
1447 } 1447 }
1448 } 1448 }
1449 1449
1450 ExtensionPrefs* MediaGalleriesPreferences::GetExtensionPrefs() const { 1450 ExtensionPrefs* MediaGalleriesPreferences::GetExtensionPrefs() const {
1451 DCHECK(IsInitialized()); 1451 DCHECK(IsInitialized());
1452 if (extension_prefs_for_testing_) 1452 if (extension_prefs_for_testing_)
1453 return extension_prefs_for_testing_; 1453 return extension_prefs_for_testing_;
1454 return extensions::ExtensionPrefs::Get(profile_); 1454 return extensions::ExtensionPrefs::Get(profile_);
1455 } 1455 }
1456 1456
1457 void MediaGalleriesPreferences::SetExtensionPrefsForTesting( 1457 void MediaGalleriesPreferences::SetExtensionPrefsForTesting(
1458 extensions::ExtensionPrefs* extension_prefs) { 1458 extensions::ExtensionPrefs* extension_prefs) {
1459 DCHECK(IsInitialized()); 1459 DCHECK(IsInitialized());
1460 extension_prefs_for_testing_ = extension_prefs; 1460 extension_prefs_for_testing_ = extension_prefs;
1461 } 1461 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698