diff options
| author | chron@chromium.org <chron@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-04-05 23:56:16 +0000 |
|---|---|---|
| committer | chron@chromium.org <chron@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-04-05 23:56:16 +0000 |
| commit | dd8fa1a9964a4b68eea27675be52a8aceda3af0f (patch) | |
| tree | d81363972e96ef6a36118846eed1d2ec0d556e35 /chrome/chrome_browser.gypi | |
| parent | 585926926a4dc5d5f1b99b7ec8ad00368b289aa6 (diff) | |
| download | chromium_src-dd8fa1a9964a4b68eea27675be52a8aceda3af0f.zip chromium_src-dd8fa1a9964a4b68eea27675be52a8aceda3af0f.tar.gz chromium_src-dd8fa1a9964a4b68eea27675be52a8aceda3af0f.tar.bz2 | |
Add a scoped notifier for pref dictionaries / lists.
Other options included subclassing the pref value types, making the firenotifications public, or just friend classing.
Since values can contain other values, such as dictionary within dictionary, it didn't seem like a good idea to subclass. We might want to consider combining the scoped lookup with the GetMutableList call at some point, but some callers keep a long reference so that might be a bad idea.
BUG=38557
TEST=manual
Review URL: http://codereview.chromium.org/1549020
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@43678 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/chrome_browser.gypi')
| -rw-r--r--[-rwxr-xr-x] | chrome/chrome_browser.gypi | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/chrome/chrome_browser.gypi b/chrome/chrome_browser.gypi index 96ba447..08bc4ae 100755..100644 --- a/chrome/chrome_browser.gypi +++ b/chrome/chrome_browser.gypi @@ -1860,6 +1860,8 @@ 'browser/safe_browsing/safe_browsing_store_sqlite.h', 'browser/safe_browsing/safe_browsing_util.cc', 'browser/safe_browsing/safe_browsing_util.h', + 'browser/scoped_pref_update.cc', + 'browser/scoped_pref_update.h', 'browser/search_engines/edit_search_engine_controller.cc', 'browser/search_engines/edit_search_engine_controller.h', 'browser/search_engines/keyword_editor_controller.cc', |