[Sync] Make BookmarkChangeProcessor resilient against updates before add

[Sync] Make BookmarkChangeProcessor resilient against updates before add

Other bookmarks observer might make their own changes on add, triggering updates
before the add call arrives. Be resilient against this.

BUG=369728
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=269645

[Nicolas Zea, 2014-05-09 20:08:41]

PTAL

[rlarocque, 2014-05-09 20:18:18]

On 2014/05/09 20:08:41, Nicolas Zea wrote:
> PTAL

Should there be a CreateOrUpdateSyncNode() function?  It looks like there's a
bit of duplication between the two calls to UpdateSyncNode() in
bookmark_change_processor.cc.

[Nicolas Zea, 2014-05-09 20:28:25]

On 2014/05/09 20:18:18, rlarocque wrote:
> On 2014/05/09 20:08:41, Nicolas Zea wrote:
> > PTAL
> 
> Should there be a CreateOrUpdateSyncNode() function?  It looks like there's a
> bit of duplication between the two calls to UpdateSyncNode() in
> bookmark_change_processor.cc.

Good point, done.

[rlarocque, 2014-05-09 20:32:21]

lgtm

[Nicolas Zea, 2014-05-09 20:33:36]

The CQ bit was checked by zea@chromium.org

[commit-bot: I haz the power, 2014-05-09 20:37:27]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/zea@chromium.org/277033002/60001

[commit-bot: I haz the power, 2014-05-11 00:14:37]

Change committed as 269645