| SLSA-Policy-Verified | +1 |
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
| SLSA-Policy-Verified | +1 |
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
| SLSA-Policy-Verified | +1 |
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
| SLSA-Policy-Verified | +1 |
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
| SLSA-Policy-Verified | +1 |
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
| SLSA-Policy-Verified | +1 |
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
| SLSA-Policy-Verified | +1 |
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
| SLSA-Policy-Verified | +1 |
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
| SLSA-Policy-Verified | +1 |
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
| SLSA-Policy-Verified | +1 |
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
| SLSA-Policy-Verified | +1 |
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
| SLSA-Policy-Verified | +1 |
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
| SLSA-Policy-Verified | +1 |
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
| SLSA-Policy-Verified | +1 |
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
| SLSA-Policy-Verified | +1 |
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
| SLSA-Policy-Verified | +1 |
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
CHECK(interaction_stats_.username_value == stats->username_value);Let's try to revert these afterwards. This is surprising to me. components/password_manager/core/browser/password_store/interactions_stats.h misses an `#include <string>`.
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
| Code-Review | +1 |
I don't understand the #include <cstddef> and #include <cstdint> because I think that they give us std::size_t instead of size_t and std::int64_t instead of int64_t. We can look into this asynchronously.
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
CHECK(interaction_stats_.username_value == stats->username_value);Let's try to revert these afterwards. This is surprising to me. components/password_manager/core/browser/password_store/interactions_stats.h misses an `#include <string>`.
I can give it another try afterwards.
components/password_manager/core/browser/password_store/interactions_stats.h misses an #include <string
I had added that include manually in my attempts to make the CHECK work, and didn't remove it to avoid another bot run.
#include <cstddef>Christoph SchweringI wonder where this comes from.
Discussed offline: from `size_t`, but it should be `stddef.h`. I'd prefer fixing this in a followup to avoid another bot run.
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
CHECK(interaction_stats_.username_value == stats->username_value);Christoph SchweringLet's try to revert these afterwards. This is surprising to me. components/password_manager/core/browser/password_store/interactions_stats.h misses an `#include <string>`.
I can give it another try afterwards.
components/password_manager/core/browser/password_store/interactions_stats.h misses an #include <string
I had added that include manually in my attempts to make the CHECK work, and didn't remove it to avoid another bot run.
components/password_manager/core/browser/password_store/interactions_stats.h misses an #include <string
I had added that include manually in my attempts to make the CHECK work, and didn't remove it to avoid another bot run.
I responded too quickly. The include I was referring to is `#include <string>` in `chrome/browser/ui/passwords/bubble_controllers/save_update_bubble_controller.cc`, which isn't necessary.
`components/password_manager/core/browser/password_store/interactions_stats.h` actually contains and should contain an `#include <string>`.
CHECK(interaction_stats_.username_value == stats->username_value);Christoph SchweringLet's try to revert these afterwards. This is surprising to me. components/password_manager/core/browser/password_store/interactions_stats.h misses an `#include <string>`.
Christoph SchweringI can give it another try afterwards.
components/password_manager/core/browser/password_store/interactions_stats.h misses an #include <string
I had added that include manually in my attempts to make the CHECK work, and didn't remove it to avoid another bot run.
components/password_manager/core/browser/password_store/interactions_stats.h misses an #include <string
I had added that include manually in my attempts to make the CHECK work, and didn't remove it to avoid another bot run.
I responded too quickly. The include I was referring to is `#include <string>` in `chrome/browser/ui/passwords/bubble_controllers/save_update_bubble_controller.cc`, which isn't necessary.
`components/password_manager/core/browser/password_store/interactions_stats.h` actually contains and should contain an `#include <string>`.
| Code-Review | +1 |
| Commit-Queue | +2 |
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
[Autofill] IWYU for //components/autofill/core/browser
TLDR: This CL only affects #includes except two trivial non-#include
changes (see (3) and (4) below).
This CL applies clang-include-cleaner to the production-code C++ files
in //components/autofill/core/browser.
(This is a by-product of generating a dependency graph in preparation
of crbug.com/489636471.)
Afterwards, I applied the following steps manually:
(1) Since clang-include-cleaner used the wrong include paths for some
sources in third_party (abseil, icu, libaddress, libphonenumber,
protobuf, re2, tflite), I fixed those paths manually.
(2) Due to now-missing transitive includes, I added the necessary
includes to tests and some files outside of
//components/autofill/core/browser.
(3) In autofill_structured_address_utils.cc, clang-include-cleaner
added a lot of Blink files for some reason. I cleaned this up manually.
(4) In PWM's save_update_bubble_controller.cc, I replaced a
CHECK_EQ(x, y) of two std::u16strings with a CHECK(x == y) because
CHECK_EQ(x, y) doesn't compile; I couldn't figure out the reason.
(5) In test_credit_card_save_strike_database.h, I added a forward
declaration to make the compiler happy.
(6) I moved new includes from a file in //components/payments and six
files in //ios to headers in //components/autofill to avoid reviews
in other timezones. I've added TODOs and will fix this in a followup
CL.
To validate the CL, I:
- Ran clang-include-cleaner a second time to confirm the results.
- Spot-checked the added and removed includes in >10 files.
- Checked that only includes are affected (% (3) and (4) above):
$ git diff -U0 origin/main | \
grep -E '^[+-]' | \
grep -Ev '^(---|\+\+\+)' | \
grep -Ev '^.(#include|#if|#endif|$)'
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
CHECK(interaction_stats_.username_value == stats->username_value);Christoph SchweringLet's try to revert these afterwards. This is surprising to me. components/password_manager/core/browser/password_store/interactions_stats.h misses an `#include <string>`.
Christoph SchweringI can give it another try afterwards.
components/password_manager/core/browser/password_store/interactions_stats.h misses an #include <string
I had added that include manually in my attempts to make the CHECK work, and didn't remove it to avoid another bot run.
Dominic Battrécomponents/password_manager/core/browser/password_store/interactions_stats.h misses an #include <string
I had added that include manually in my attempts to make the CHECK work, and didn't remove it to avoid another bot run.
I responded too quickly. The include I was referring to is `#include <string>` in `chrome/browser/ui/passwords/bubble_controllers/save_update_bubble_controller.cc`, which isn't necessary.
`components/password_manager/core/browser/password_store/interactions_stats.h` actually contains and should contain an `#include <string>`.
https://source.chromium.org/chromium/chromium/src/+/main:components/password_manager/core/browser/password_store/interactions_stats.h;drc=a8f56567951a7413780be19fabb2bfc7a7426bf3 does not show me an `#include <string>`.
That's right, but this CL adds it: https://chromium-review.git.corp.google.com/c/chromium/src/+/7849414/17/components/password_manager/core/browser/password_store/interactions_stats.h 😊
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |