Google Git
Sign in
chromium / chromium / src / 44ebd87f31b9f87ad75466e2ed29dc3d8cc51a6f
commit44ebd87f31b9f87ad75466e2ed29dc3d8cc51a6f[log] [tgz]
authorLei Zhang <thestig@chromium.org>Wed Jun 25 17:21:18 2025
committerChromium LUCI CQ <chromium-scoped@luci-project-accounts.iam.gserviceaccount.com>Wed Jun 25 17:21:18 2025
tree101f149a54aba12e489ba89046718860dabc21c6
parent21d0c927c7b988123be129f1599842d3ced18e95 [diff]
Printing: Remove kFastEnumeratePrinters feature and associated code

This feature shipped with M136 in an enabled-by-default state. Since
nobody complained about it, delete the feature and any code that can
only be used when the feature is disabled. As part of this, also remove:

- The chrome://flags entry.
- PrintBackendWinTest.MANUAL_ValidateFastEnumeratePrinters.
- GetBasicPrinterInfoMixedMethodForTesting() used by the test above.
- Printing.EnumeratePrinters.BasicInfo.Registry histogram.

The histogram does show some failures fetching the printer's location.
Though there are no known user complains about this. If users do
complain about it in the future, the plan is to work with the users to
diagnose and fix their issue.

After removing GetBasicPrinterInfoMixedMethodForTesting(),
GetBasicPrinterInfoMixedMethod() can be folded into its caller.

OBSOLETE_HISTOGRAM[Printing.EnumeratePrinters.BasicInfo.Registry]=No longer needed.

Bug: 427495363
Change-Id: Ib0c792dbae27e17d35025c0541e1a1eca5d7a286
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/6669736
Reviewed-by: Gavin Williams <gavinwill@chromium.org>
Commit-Queue: Lei Zhang <thestig@chromium.org>
Reviewed-by: Andy Phan <andyphan@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1478636}
8 files changed
tree: 101f149a54aba12e489ba89046718860dabc21c6
  1. .github/
  2. android_webview/
  3. apps/
  4. ash/
  5. base/
  6. build/
  7. build_overrides/
  8. buildtools/
  9. cc/
  10. chrome/
  11. chromecast/
  12. chromeos/
  13. codelabs/
  14. components/
  15. content/
  16. crypto/
  17. dbus/
  18. device/
  19. docs/
  20. extensions/
  21. fuchsia_web/
  22. gin/
  23. google_apis/
  24. gpu/
  25. headless/
  26. infra/
  27. ios/
  28. ipc/
  29. media/
  30. mojo/
  31. native_client_sdk/
  32. net/
  33. pdf/
  34. ppapi/
  35. printing/
  36. remoting/
  37. rlz/
  38. sandbox/
  39. services/
  40. skia/
  41. sql/
  42. storage/
  43. styleguide/
  44. testing/
  45. third_party/
  46. tools/
  47. ui/
  48. url/
  49. webkit/
  50. clank
  51. internal
  52. ios_internal
  53. signing_keys
  54. v8
  55. .clang-format
  56. .clang-tidy
  57. .clangd
  58. .cursorignore
  59. .git-blame-ignore-revs
  60. .gitallowed
  61. .gitattributes
  62. .gitignore
  63. .gitmodules
  64. .gn
  65. .mailmap
  66. .rustfmt.toml
  67. .vpython3
  68. .yapfignore
  69. ATL_OWNERS
  70. AUTHORS
  71. BUILD.gn
  72. CODE_OF_CONDUCT.md
  73. codereview.settings
  74. CPPLINT.cfg
  75. CRYPTO_OWNERS
  76. DEPS
  77. DIR_METADATA
  78. LICENSE
  79. LICENSE.chromium_os
  80. OWNERS
  81. PRESUBMIT.py
  82. PRESUBMIT_test.py
  83. PRESUBMIT_test_mocks.py
  84. README.md
  85. SECURITY_OWNERS
  86. 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.

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.