• Rushan Suleymanov's avatar
    [Sync] Optimize bookmarks initial merge · 12db6b66
    Rushan Suleymanov authored
    This CL slightly optimizes the code by changing the order of some
    operations during the initial merge. When trying to find the local
    matching node by semantics, the new order is: check matching GUID, check
    URL and then check for the same title (before current CL the order was
    reversed). In the real scenarios it should be more efficient because
    most of bookmarks already have GUID.
    
    This CL doesn't change behaviour.
    
    Bug: 1163257
    Change-Id: If9a2371c7c3f28962c8565513487175482ce4104
    Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2640398
    Commit-Queue: Rushan Suleymanov <rushans@google.com>
    Reviewed-by: default avatarMikel Astiz <mastiz@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#846021}
    12db6b66
bookmark_model_merger.cc 38.6 KB