Reland "[css-pseudo] Set default ::marker styles in a real UA sheet"

This is a reland of a53560e3c83ab4d7b379247d16995ca774e772ba

The problem in the reverted original patch was that I forgot to clear
marker_style_sheet_ in CSSDefaultStyleSheets::PrepareForLeakDetection.
Then, when running tests with --enable-leak-detection, it could happen
that EnsureDefaultStyleSheetsForPseudoElement would think that the UA
rules for ::marker were already in default_pseudo_element_style_. But
it would actually be empty, so the ::marker could get no rule, making
HasMatchedProperties() be false, so PseudoStyleForElementInternal would
return false, and PseudoStyleForElement would return a nullptr style.
So there would be no ::marker.

Original change's description:
> [css-pseudo] Set default ::marker styles in a real UA sheet
>
> Before this patch, default styles for ::marker were assigned manually
> when resolving the style. The problem was that, since the rules didn't
> actually exist in any stylesheet, they were not exposed in devtools.
> Also, devtools assume that if a pseudo-element exists, it must have some
> rule. This could not hold for ::marker, and then trying to add styles
> to it would fail.
>
> This patch fixes that by making the styles actual UA rules. However,
> adding a '::marker' selector directly to the UA sheet for elements would
> be bad: it would match every single element (even non-list-items) due to
> the universal selector, and then it would set an internal bit for the
> marker. So HasAnyPseudoElementStyles() would return true, which means
> extra work when handling style changes.
>
> Therefore, this patch adds a new UA sheet just for pseudo-elements. This
> sheet is not used for elements, ensuring that HasAnyPseudoElementStyles
> is still false if there are no author styles for a pseudo-element.
>
> The selector parser was treating ::marker as invalid if the runtime flag
> CSSMarkerPseudoElement was disabled. But now it's accepted in that case
> for UA styles.
>
> There already was a kHasMarkerPseudoElement counter to track how widely
> used is ::marker on the web. Now this counter will only be incremented
> for author rules.
>
> Bug: 457718, 1082377
>
> TEST=blink_unittests --gtest_filter=StyleResolverTest.CSSMarkerPseudoElement
> TEST=blink_unittests --gtest_filter=UseCounterHelperTest.CSSMarkerPseudoElementUA
> TEST=blink_unittests --gtest_filter=UseCounterHelperTest.CSSMarkerPseudoElementAuthor
> TEST=http/tests/devtools/elements/styles-2/pseudo-elements.js
>
> Change-Id: Ib5583dc1952bfec2cce8f33fe8332cc39016c34a
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2239579
> Reviewed-by: Mike West <mkwst@chromium.org>
> Reviewed-by: Rune Lillesveen <futhark@chromium.org>
> Commit-Queue: Oriol Brufau <obrufau@igalia.com>
> Cr-Commit-Position: refs/heads/master@{#778303}

TBR=mkwst@chromium.org

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

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.