diff options
| author | tfarina@chromium.org <tfarina@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-07-21 01:07:18 +0000 |
|---|---|---|
| committer | tfarina@chromium.org <tfarina@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-07-21 01:07:18 +0000 |
| commit | 5b5c9b7f3c0772338c38ed5dac91b81a9e32765f (patch) | |
| tree | fb63882d3099713c7bebaeafcb0951da38907bd9 /sandbox/src | |
| parent | 0ec60890ea1cc069b04671f2df3000cdd48f6c3d (diff) | |
| download | chromium_src-5b5c9b7f3c0772338c38ed5dac91b81a9e32765f.zip chromium_src-5b5c9b7f3c0772338c38ed5dac91b81a9e32765f.tar.gz chromium_src-5b5c9b7f3c0772338c38ed5dac91b81a9e32765f.tar.bz2 | |
bookmarks: Don't expose an implementation detail function through BookmarkNode public API.
This function is only interesting for BookmarkModel and at this moment is just used there.
Instead of exposing it through the BookmarkNode class just do it internally in BookmarkModel.
This has the advantage of preventing further external clients of using it.
This is a responsibility of BookmarkModel, and just it knows when to call this function.
The consumers of BookmarkNode API don't know and shouldn't know where/when it's necessary to call
InvalidateIcon(), so lets the model do the work for them.
BUG=None
TEST=None
R=sky@chromium.org
Review URL: http://codereview.chromium.org/7401013
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@93306 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'sandbox/src')
0 files changed, 0 insertions, 0 deletions