| Auto-Submit | +1 |
| 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. |
+alexmos@ for content/browser/child_process_security_policy_impl.cc
+alexmos@ for content/browser/child_process_security_policy_impl.cc
| 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. |
Migrate base::Contains() to std::ranges::contains() in content
base::Contains() is an alias to std::ranges::contains() since
crrev.com/c/7415207, so this CL has no behavior change.
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
I can't build content_unittests locally after this CL landed, getting the following error. Curious why this isn't turning the bots red. Do I need to change something in my environment to fix?
```
stdout:
../../content/browser/webui/web_ui_data_source_unittest.cc:490:15: error: no matching function for call to object of type 'const __contains'
490 | EXPECT_TRUE(std::ranges::contains(result, "hello plain!"));
| ^~~~~~~~~~~~~~~~~~~~~
../../third_party/googletest/src/googletest/include/gtest/gtest.h:1826:50: note: expanded from macro 'EXPECT_TRUE'
1826 | #define EXPECT_TRUE(condition) GTEST_EXPECT_TRUE(condition)
| ^~~~~~~~~
../../third_party/googletest/src/googletest/include/gtest/gtest.h:1811:23: note: expanded from macro 'GTEST_EXPECT_TRUE'
1811 | GTEST_TEST_BOOLEAN_(condition, #condition, false, true, \
| ^~~~~~~~~
../../third_party/googletest/src/googletest/include/gtest/internal/gtest-internal.h:1450:38: note: expanded from macro 'GTEST_TEST_BOOLEAN_'
1450 | ::testing::AssertionResult(expression)) \
| ^~~~~~~~~~
gen/third_party/libc++/src/include/__algorithm/ranges_contains.h:47:3: note: candidate template ignored: constraints not satisfied [with _Range = std::string &, _Type = char[13], _Proj = identity]
47 | operator()(_Range&& __range, const _Type& __value, _Proj __proj = {}) {
| ^
gen/third_party/libc++/src/include/__algorithm/ranges_contains.h:45:14: note: because 'indirect_binary_predicate<ranges::equal_to, projected<iterator_t<std::string &>, identity>, const char *[13]>' evaluated to false
45 | requires indirect_binary_predicate<ranges::equal_to, projected<iterator_t<_Range>, _Proj>, const _Type*>
| ^
gen/third_party/libc++/src/include/__iterator/concepts.h:264:5: note: because 'predicate<std::ranges::equal_to &, __indirect_value_t<std::__projected_impl<std::__wrap_iter<char *>, std::identity>::__type>, __indirect_value_t<const char (*)[13]>>' evaluated to false
264 | predicate<_Fp&, __indirect_value_t<_It1>, __indirect_value_t<_It2>> &&
| ^
gen/third_party/libc++/src/include/__concepts/predicate.h:28:21: note: because 'regular_invocable<std::ranges::equal_to &, char &, char (&)[13]>' evaluated to false
28 | concept predicate = regular_invocable<_Fn, _Args...> && __boolean_testable<invoke_result_t<_Fn, _Args...>>;
| ^
gen/third_party/libc++/src/include/__concepts/invocable.h:34:29: note: because 'invocable<std::ranges::equal_to &, char &, char (&)[13]>' evaluated to false
34 | concept regular_invocable = invocable<_Fn, _Args...>;
| ^
gen/third_party/libc++/src/include/__concepts/invocable.h:28:3: note: because 'std::invoke(std::forward<_Fn>(__fn), std::forward<_Args>(__args)...)' would be invalid: no matching function for call to 'invoke'
28 | std::invoke(std::forward<_Fn>(__fn), std::forward<_Args>(__args)...); // not required to be equality preserving
| ^
gen/third_party/libc++/src/include/__algorithm/ranges_contains.h:40:3: note: candidate function template not viable: requires at least 3 arguments, but 2 were provided
40 | operator()(_Iter __first, _Sent __last, const _Type& __value, _Proj __proj = {}) {
| ^ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
1 error generated.
build failed
```
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
Closing the loop here, fixing this at https://chromium-review.googlesource.com/c/chromium/src/+/7499012.