|
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
|