diff options
| author | munjal@chromium.org <munjal@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-10-29 06:48:15 +0000 |
|---|---|---|
| committer | munjal@chromium.org <munjal@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-10-29 06:48:15 +0000 |
| commit | 6536c99fc129175331b21855f20c089c0cda44d6 (patch) | |
| tree | f3698842574a52a51256329a7fff2376c8064a70 /sdch | |
| parent | 327344ef227af4c66b42cb7e948d4c4096eab83f (diff) | |
| download | chromium_src-6536c99fc129175331b21855f20c089c0cda44d6.zip chromium_src-6536c99fc129175331b21855f20c089c0cda44d6.tar.gz chromium_src-6536c99fc129175331b21855f20c089c0cda44d6.tar.bz2 | |
Make some improvements to sync model associator:
- Fail the load assocications if there the total number of
nodes in the bookmark model and the sync model don't match
(issue: 25542)
- Add a bookmark id index and use that during load assocications
instead of calling BookmarkModel::GetNodeByID since the latter
traverses the entier tree in DFS order until it finds the node
with the given ID.
- Add a unit test that would exercise the new model associator
load associations behavior.
BUG=25542
TEST=Unit test added.
Review URL: http://codereview.chromium.org/337037
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@30441 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'sdch')
0 files changed, 0 insertions, 0 deletions