[go: nahoru, domu]

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};
 };