Google Git
Sign in
chromium / chromium / src / efb6fcfbc3c74932c8030feaafaf90b725604a68
commitefb6fcfbc3c74932c8030feaafaf90b725604a68[log] [tgz]
authorLily Chen <[email protected]>Fri Apr 19 04:17:54 2019
committerCommit Bot <[email protected]>Fri Apr 19 04:17:54 2019
tree89894d547a3ef5e8036d0b85fb67456de47080a4
parent52a28e416a9c4fe3a939f1fbdee309fef52b3c91 [diff]
Reporting: Refactor ReportingCache's storage of clients and endpoints

See https://docs.google.com/document/d/19vDLCRVP-WvWxA_YNZfhkkW6fS5A2Ea9ulJ8i8AcPAM/edit?usp=sharing
for details.

This CL refactors the client side of the ReportingCacheImpl. Clients,
endpoint groups, and endpoints are now represented by separate structs.
The cache stores each in separate maps. Endpoints are evicted if an
origin exceeds the per-origin endpoint limit, or if the cache exceeds
the global endpoint limit.

This fixes an incorrect assumption made by the previous implementation
that a given endpoint URL cannot be in multiple groups for the same
origin.

Bug: 912951
Change-Id: I0c548c3cf20ac57a9f17482300dd9a57a14e53b9
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1453109
Commit-Queue: Lily Chen <[email protected]>
Reviewed-by: Joshua Bell <[email protected]>
Reviewed-by: Eric Orth <[email protected]>
Cr-Commit-Position: refs/heads/master@{#652467}
29 files changed
tree: 89894d547a3ef5e8036d0b85fb67456de47080a4
  1. android_webview/
  2. apps/
  3. ash/
  4. base/
  5. build/
  6. build_overrides/
  7. buildtools/
  8. cc/
  9. chrome/
  10. chrome_elf/
  11. chromecast/
  12. chromeos/
  13. cloud_print/
  14. components/
  15. content/
  16. courgette/
  17. crypto/
  18. dbus/
  19. device/
  20. docs/
  21. extensions/
  22. fuchsia/
  23. gin/
  24. google_apis/
  25. google_update/
  26. gpu/
  27. headless/
  28. infra/
  29. ios/
  30. ipc/
  31. jingle/
  32. media/
  33. mojo/
  34. native_client_sdk/
  35. net/
  36. pdf/
  37. ppapi/
  38. printing/
  39. remoting/
  40. rlz/
  41. sandbox/
  42. services/
  43. skia/
  44. sql/
  45. storage/
  46. styleguide/
  47. testing/
  48. third_party/
  49. tools/
  50. ui/
  51. url/
  52. .clang-format
  53. .eslintrc.js
  54. .git-blame-ignore-revs
  55. .gitattributes
  56. .gitignore
  57. .gn
  58. .vpython
  59. AUTHORS
  60. BUILD.gn
  61. CODE_OF_CONDUCT.md
  62. codereview.settings
  63. DEPS
  64. ENG_REVIEW_OWNERS
  65. LICENSE
  66. LICENSE.chromium_os
  67. OWNERS
  68. PRESUBMIT.py
  69. PRESUBMIT_test.py
  70. PRESUBMIT_test_mocks.py
  71. README.md
  72. WATCHLISTS
README.md

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.

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

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