MiraclePtr: Rewrite chromeos-ash [5/N]
This was automated.
It rewrites chromeos-ash and annotate the dangling pointers.
This part was split out of the root CL:
https://chromium-review.googlesource.com/c/chromium/src/+/4459392
number file selected
------ ----------- --------
1700 chrome/
1407 ash/ x
626 chromeos/
147 components/
53 ui/
16 extensions/
14 media/
9 remoting/
8 services/
5 content/
3 cc/
1 gpu/
1 crypto/
Steps:
1. Execute the rewriter:
./tools/clang/rewrite_raw_ptr_fields/rewrite-multiple-platforms.sh chromeos-ash
2. Execute all the tests. Print the dangling pointers using
https://godbolt.org/z/o8adYnc7a idea. Then apply the edits using:
https://github.com/ArthurSonzogni/chrome-dangling-ptr-apply-edit
3. Apply `git cl format`
4. Add `ExperimentalAsh`. This happened using a vim macro on the root
patch. Overall there are exactly 7461 new pointers, and 183 dangling
pointers.
5. Apply `git cl format`
Cq-Include-Trybots:luci.chromium.try:android-pie-arm64-dbg,android-x64-Cq-Include-Trybots: luci.chromium.try:android-pie-arm64-dbg,android-x64-cast,android_compile_dbg,android_compile_x64_dbg,android_compile_x86_dbg,android_cronet,chromeos-amd64-generic-dbg,chromeos-arm-generic-rel,dawn-android-arm-deps-rel,dawn-mac-x64-deps-rel,fuchsia-arm64-rel,fuchsia-x64-cast-receiver-rel,ios-simulator,lacros-amd64-generic-rel,lacros-arm-generic-rel,linux-chromeos-compile-dbg,linux-libfuzzer-asan-rel,linux_chromium_asan_rel_ng,linux_chromium_compile_dbg_ng,linux_chromium_tsan_rel_ng,linux_vr,mac_chromium_compile_dbg_ng,win-libfuzzer-asan-rel,win_chromium_compile_dbg_ng,android-arm64-rel,android-nougat-x86-rel,android_optional_gpu_tests_rel,chromeos-amd64-generic-rel,dawn-linux-x64-deps-rel,dawn-win10-x64-deps-rel,dawn-win10-x86-deps-rel,linux-chromeos-rel,linux-lacros-rel,linux-rel,linux-wayland-rel,linux-x64-castos,linux_optional_gpu_tests_rel,mac_optional_gpu_tests_rel,win_optional_gpu_tests_rel
Include-Ci-Only-Tests: true
Bug:1430482
Change-Id: Ie1e24f40a0688c1148b27455d05aae73803c10da
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/4456581
Commit-Queue: Arthur Sonzogni <arthursonzogni@chromium.org>
Reviewed-by: Bartek Nowierski <bartekn@chromium.org>
Owners-Override: Bartek Nowierski <bartekn@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1134206}
diff --git a/ash/system/phonehub/app_stream_launcher_view.h b/ash/system/phonehub/app_stream_launcher_view.h
index ffa5de98..ed6e7ed1 100644
--- a/ash/system/phonehub/app_stream_launcher_view.h
+++ b/ash/system/phonehub/app_stream_launcher_view.h
@@ -10,6 +10,7 @@
#include "ash/ash_export.h"
#include "ash/system/phonehub/phone_hub_content_view.h"
+#include "base/memory/raw_ptr.h"
#include "chromeos/ash/components/phonehub/app_stream_launcher_data_model.h"
#include "chromeos/ash/components/phonehub/notification.h"
#include "chromeos/ash/components/phonehub/recent_app_click_observer.h"
@@ -80,11 +81,11 @@
void CreateGridView(
const std::vector<phonehub::Notification::AppMetadata>* apps_list);
- views::Button* arrow_back_button_ = nullptr;
- phonehub::PhoneHubManager* phone_hub_manager_;
+ raw_ptr<views::Button, ExperimentalAsh> arrow_back_button_ = nullptr;
+ raw_ptr<phonehub::PhoneHubManager, ExperimentalAsh> phone_hub_manager_;
// Contains all the apps
- views::View* items_container_;
+ raw_ptr<views::View, ExperimentalAsh> items_container_;
base::WeakPtrFactory<AppStreamLauncherView> weak_factory_{this};
};