[sync] Fix crash due to rare corrupt bookmark data

This guards that GUID-inferring logic against incoming data that is
considered corrupt: the case where an incoming bookmark has a client tag
hash (which means it was created with a GUID) and yet the update
includes no GUID in specifics. This means the GUID was cleared out by
some old client, which is problematic and something that the server
should likely refuse to do.

For these cases, the GUID cannot be actually inferred and hence the
update should be ignored.

On top of that, to be cautious about users that already ran into this
state, the patch introduces a quirk to best-effort infer the GUID by
other means, if the bookmark was already tracked locally. This logic
is introduced merely to avoid a regression, because although some users
do run into crashes, the previous implementation may sometimes just fix
the data corruption by issuing a commit (due to implementation

Change-Id: I80b072aa41f17111ab2b4c5c95d09a9d0285059f
Bug: 1231450
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3045398
Commit-Queue: Mikel Astiz <mastiz@chromium.org>
Reviewed-by: Rushan Suleymanov <rushans@google.com>
Cr-Commit-Position: refs/heads/master@{#904341}
8 files changed
tree: 97d5caf9e3259c4f8a1a58699e2f61b321ffea70
  1. .clang-format
  2. .clang-tidy
  3. .eslintrc.js
  4. .git-blame-ignore-revs
  5. .gitattributes
  6. .gitignore
  7. .gn
  8. .mailmap
  9. .vpython
  10. .vpython3
  11. .yapfignore
  13. BUILD.gn
  15. DEPS
  19. LICENSE.chromium_os
  20. OWNERS
  21. PRESUBMIT.py
  22. PRESUBMIT_test.py
  23. PRESUBMIT_test_mocks.py
  24. README.md
  26. android_webview/
  27. apps/
  28. ash/
  29. base/
  30. build/
  31. build_overrides/
  32. buildtools/
  33. cc/
  34. chrome/
  35. chromecast/
  36. chromeos/
  37. cloud_print/
  38. codelabs/
  39. codereview.settings
  40. components/
  41. content/
  42. courgette/
  43. crypto/
  44. dbus/
  45. device/
  46. docs/
  47. extensions/
  48. fuchsia/
  49. gin/
  50. google_apis/
  51. google_update/
  52. gpu/
  53. headless/
  54. infra/
  55. ios/
  56. ipc/
  57. jingle/
  58. media/
  59. mojo/
  60. native_client_sdk/
  61. net/
  62. pdf/
  63. ppapi/
  64. printing/
  65. remoting/
  66. rlz/
  67. sandbox/
  68. services/
  69. skia/
  70. sql/
  71. storage/
  72. styleguide/
  73. testing/
  74. third_party/
  75. tools/
  76. ui/
  77. url/
  78. weblayer/

Logo Chromium

Chromium is an open-source browser project that aims to build a safer, faster, and more stable way for all users to experience the web.

The project's web site is https://www.chromium.org.

To check out the source code locally, don't use git clone! Instead, follow the instructions on how to get the code.

Documentation in the source is rooted in docs/README.md.

Learn how to Get Around the Chromium Source Code Directory Structure .

For historical reasons, there are some small top level directories. Now the guidance is that new top level directories are for product (e.g. Chrome, Android WebView, Ash). Even if these products have multiple executables, the code should be in subdirectories of the product.

If you found a bug, please file it at https://crbug.com/new.