xref: /aosp_15_r20/external/cronet/base/no_destructor_unittest.cc (revision 6777b5387eb2ff775bb5750e3f5d96f37fb7352b)
1*6777b538SAndroid Build Coastguard Worker // Copyright 2018 The Chromium Authors
2*6777b538SAndroid Build Coastguard Worker // Use of this source code is governed by a BSD-style license that can be
3*6777b538SAndroid Build Coastguard Worker // found in the LICENSE file.
4*6777b538SAndroid Build Coastguard Worker 
5*6777b538SAndroid Build Coastguard Worker #include "base/no_destructor.h"
6*6777b538SAndroid Build Coastguard Worker 
7*6777b538SAndroid Build Coastguard Worker #include <memory>
8*6777b538SAndroid Build Coastguard Worker #include <string>
9*6777b538SAndroid Build Coastguard Worker #include <utility>
10*6777b538SAndroid Build Coastguard Worker #include <vector>
11*6777b538SAndroid Build Coastguard Worker 
12*6777b538SAndroid Build Coastguard Worker #include "base/atomicops.h"
13*6777b538SAndroid Build Coastguard Worker #include "base/barrier_closure.h"
14*6777b538SAndroid Build Coastguard Worker #include "base/check.h"
15*6777b538SAndroid Build Coastguard Worker #include "base/functional/bind.h"
16*6777b538SAndroid Build Coastguard Worker #include "base/system/sys_info.h"
17*6777b538SAndroid Build Coastguard Worker #include "base/threading/platform_thread.h"
18*6777b538SAndroid Build Coastguard Worker #include "base/threading/simple_thread.h"
19*6777b538SAndroid Build Coastguard Worker #include "build/build_config.h"
20*6777b538SAndroid Build Coastguard Worker #include "testing/gtest/include/gtest/gtest.h"
21*6777b538SAndroid Build Coastguard Worker 
22*6777b538SAndroid Build Coastguard Worker namespace base {
23*6777b538SAndroid Build Coastguard Worker 
24*6777b538SAndroid Build Coastguard Worker namespace {
25*6777b538SAndroid Build Coastguard Worker 
26*6777b538SAndroid Build Coastguard Worker static_assert(!std::is_trivially_destructible_v<std::string>);
27*6777b538SAndroid Build Coastguard Worker static_assert(
28*6777b538SAndroid Build Coastguard Worker     std::is_trivially_destructible_v<base::NoDestructor<std::string>>);
29*6777b538SAndroid Build Coastguard Worker 
30*6777b538SAndroid Build Coastguard Worker struct CheckOnDestroy {
~CheckOnDestroybase::__anonc85363690111::CheckOnDestroy31*6777b538SAndroid Build Coastguard Worker   ~CheckOnDestroy() { CHECK(false); }
32*6777b538SAndroid Build Coastguard Worker };
33*6777b538SAndroid Build Coastguard Worker 
TEST(NoDestructorTest,SkipsDestructors)34*6777b538SAndroid Build Coastguard Worker TEST(NoDestructorTest, SkipsDestructors) {
35*6777b538SAndroid Build Coastguard Worker   NoDestructor<CheckOnDestroy> destructor_should_not_run;
36*6777b538SAndroid Build Coastguard Worker }
37*6777b538SAndroid Build Coastguard Worker 
38*6777b538SAndroid Build Coastguard Worker struct UncopyableUnmovable {
39*6777b538SAndroid Build Coastguard Worker   UncopyableUnmovable() = default;
UncopyableUnmovablebase::__anonc85363690111::UncopyableUnmovable40*6777b538SAndroid Build Coastguard Worker   explicit UncopyableUnmovable(int value) : value(value) {}
41*6777b538SAndroid Build Coastguard Worker 
42*6777b538SAndroid Build Coastguard Worker   UncopyableUnmovable(const UncopyableUnmovable&) = delete;
43*6777b538SAndroid Build Coastguard Worker   UncopyableUnmovable& operator=(const UncopyableUnmovable&) = delete;
44*6777b538SAndroid Build Coastguard Worker 
45*6777b538SAndroid Build Coastguard Worker   int value = 1;
46*6777b538SAndroid Build Coastguard Worker   std::string something_with_a_nontrivial_destructor;
47*6777b538SAndroid Build Coastguard Worker };
48*6777b538SAndroid Build Coastguard Worker 
49*6777b538SAndroid Build Coastguard Worker struct CopyOnly {
50*6777b538SAndroid Build Coastguard Worker   CopyOnly() = default;
51*6777b538SAndroid Build Coastguard Worker 
52*6777b538SAndroid Build Coastguard Worker   CopyOnly(const CopyOnly&) = default;
53*6777b538SAndroid Build Coastguard Worker   CopyOnly& operator=(const CopyOnly&) = default;
54*6777b538SAndroid Build Coastguard Worker 
55*6777b538SAndroid Build Coastguard Worker   CopyOnly(CopyOnly&&) = delete;
56*6777b538SAndroid Build Coastguard Worker   CopyOnly& operator=(CopyOnly&&) = delete;
57*6777b538SAndroid Build Coastguard Worker };
58*6777b538SAndroid Build Coastguard Worker 
59*6777b538SAndroid Build Coastguard Worker struct MoveOnly {
60*6777b538SAndroid Build Coastguard Worker   MoveOnly() = default;
61*6777b538SAndroid Build Coastguard Worker 
62*6777b538SAndroid Build Coastguard Worker   MoveOnly(const MoveOnly&) = delete;
63*6777b538SAndroid Build Coastguard Worker   MoveOnly& operator=(const MoveOnly&) = delete;
64*6777b538SAndroid Build Coastguard Worker 
65*6777b538SAndroid Build Coastguard Worker   MoveOnly(MoveOnly&&) = default;
66*6777b538SAndroid Build Coastguard Worker   MoveOnly& operator=(MoveOnly&&) = default;
67*6777b538SAndroid Build Coastguard Worker };
68*6777b538SAndroid Build Coastguard Worker 
69*6777b538SAndroid Build Coastguard Worker struct ForwardingTestStruct {
ForwardingTestStructbase::__anonc85363690111::ForwardingTestStruct70*6777b538SAndroid Build Coastguard Worker   ForwardingTestStruct(const CopyOnly&, MoveOnly&&) {}
71*6777b538SAndroid Build Coastguard Worker 
72*6777b538SAndroid Build Coastguard Worker   std::string something_with_a_nontrivial_destructor;
73*6777b538SAndroid Build Coastguard Worker };
74*6777b538SAndroid Build Coastguard Worker 
TEST(NoDestructorTest,UncopyableUnmovable)75*6777b538SAndroid Build Coastguard Worker TEST(NoDestructorTest, UncopyableUnmovable) {
76*6777b538SAndroid Build Coastguard Worker   static NoDestructor<UncopyableUnmovable> default_constructed;
77*6777b538SAndroid Build Coastguard Worker   EXPECT_EQ(1, default_constructed->value);
78*6777b538SAndroid Build Coastguard Worker 
79*6777b538SAndroid Build Coastguard Worker   static NoDestructor<UncopyableUnmovable> constructed_with_arg(-1);
80*6777b538SAndroid Build Coastguard Worker   EXPECT_EQ(-1, constructed_with_arg->value);
81*6777b538SAndroid Build Coastguard Worker }
82*6777b538SAndroid Build Coastguard Worker 
TEST(NoDestructorTest,ForwardsArguments)83*6777b538SAndroid Build Coastguard Worker TEST(NoDestructorTest, ForwardsArguments) {
84*6777b538SAndroid Build Coastguard Worker   CopyOnly copy_only;
85*6777b538SAndroid Build Coastguard Worker   MoveOnly move_only;
86*6777b538SAndroid Build Coastguard Worker 
87*6777b538SAndroid Build Coastguard Worker   static NoDestructor<ForwardingTestStruct> test_forwarding(
88*6777b538SAndroid Build Coastguard Worker       copy_only, std::move(move_only));
89*6777b538SAndroid Build Coastguard Worker }
90*6777b538SAndroid Build Coastguard Worker 
TEST(NoDestructorTest,Accessors)91*6777b538SAndroid Build Coastguard Worker TEST(NoDestructorTest, Accessors) {
92*6777b538SAndroid Build Coastguard Worker   static NoDestructor<std::string> awesome("awesome");
93*6777b538SAndroid Build Coastguard Worker 
94*6777b538SAndroid Build Coastguard Worker   EXPECT_EQ("awesome", *awesome);
95*6777b538SAndroid Build Coastguard Worker   EXPECT_EQ(0, awesome->compare("awesome"));
96*6777b538SAndroid Build Coastguard Worker   EXPECT_EQ(0, awesome.get()->compare("awesome"));
97*6777b538SAndroid Build Coastguard Worker }
98*6777b538SAndroid Build Coastguard Worker 
99*6777b538SAndroid Build Coastguard Worker // Passing initializer list to a NoDestructor like in this test
100*6777b538SAndroid Build Coastguard Worker // is ambiguous in GCC.
101*6777b538SAndroid Build Coastguard Worker // https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84849
102*6777b538SAndroid Build Coastguard Worker #if !defined(COMPILER_GCC) && !defined(__clang__)
TEST(NoDestructorTest,InitializerList)103*6777b538SAndroid Build Coastguard Worker TEST(NoDestructorTest, InitializerList) {
104*6777b538SAndroid Build Coastguard Worker   static NoDestructor<std::vector<std::string>> vector({"a", "b", "c"});
105*6777b538SAndroid Build Coastguard Worker }
106*6777b538SAndroid Build Coastguard Worker #endif
107*6777b538SAndroid Build Coastguard Worker }  // namespace
108*6777b538SAndroid Build Coastguard Worker 
109*6777b538SAndroid Build Coastguard Worker namespace {
110*6777b538SAndroid Build Coastguard Worker 
111*6777b538SAndroid Build Coastguard Worker // A class whose constructor busy-loops until it is told to complete
112*6777b538SAndroid Build Coastguard Worker // construction.
113*6777b538SAndroid Build Coastguard Worker class BlockingConstructor {
114*6777b538SAndroid Build Coastguard Worker  public:
BlockingConstructor()115*6777b538SAndroid Build Coastguard Worker   BlockingConstructor() {
116*6777b538SAndroid Build Coastguard Worker     EXPECT_FALSE(WasConstructorCalled());
117*6777b538SAndroid Build Coastguard Worker     subtle::NoBarrier_Store(&constructor_called_, 1);
118*6777b538SAndroid Build Coastguard Worker     EXPECT_TRUE(WasConstructorCalled());
119*6777b538SAndroid Build Coastguard Worker     while (!subtle::NoBarrier_Load(&complete_construction_))
120*6777b538SAndroid Build Coastguard Worker       PlatformThread::YieldCurrentThread();
121*6777b538SAndroid Build Coastguard Worker     done_construction_ = true;
122*6777b538SAndroid Build Coastguard Worker   }
123*6777b538SAndroid Build Coastguard Worker   BlockingConstructor(const BlockingConstructor&) = delete;
124*6777b538SAndroid Build Coastguard Worker   BlockingConstructor& operator=(const BlockingConstructor&) = delete;
125*6777b538SAndroid Build Coastguard Worker   ~BlockingConstructor() = delete;
126*6777b538SAndroid Build Coastguard Worker 
127*6777b538SAndroid Build Coastguard Worker   // Returns true if BlockingConstructor() was entered.
WasConstructorCalled()128*6777b538SAndroid Build Coastguard Worker   static bool WasConstructorCalled() {
129*6777b538SAndroid Build Coastguard Worker     return subtle::NoBarrier_Load(&constructor_called_);
130*6777b538SAndroid Build Coastguard Worker   }
131*6777b538SAndroid Build Coastguard Worker 
132*6777b538SAndroid Build Coastguard Worker   // Instructs BlockingConstructor() that it may now unblock its construction.
CompleteConstructionNow()133*6777b538SAndroid Build Coastguard Worker   static void CompleteConstructionNow() {
134*6777b538SAndroid Build Coastguard Worker     subtle::NoBarrier_Store(&complete_construction_, 1);
135*6777b538SAndroid Build Coastguard Worker   }
136*6777b538SAndroid Build Coastguard Worker 
done_construction() const137*6777b538SAndroid Build Coastguard Worker   bool done_construction() const { return done_construction_; }
138*6777b538SAndroid Build Coastguard Worker 
139*6777b538SAndroid Build Coastguard Worker  private:
140*6777b538SAndroid Build Coastguard Worker   // Use Atomic32 instead of AtomicFlag for them to be trivially initialized.
141*6777b538SAndroid Build Coastguard Worker   static subtle::Atomic32 constructor_called_;
142*6777b538SAndroid Build Coastguard Worker   static subtle::Atomic32 complete_construction_;
143*6777b538SAndroid Build Coastguard Worker 
144*6777b538SAndroid Build Coastguard Worker   bool done_construction_ = false;
145*6777b538SAndroid Build Coastguard Worker };
146*6777b538SAndroid Build Coastguard Worker 
147*6777b538SAndroid Build Coastguard Worker // static
148*6777b538SAndroid Build Coastguard Worker subtle::Atomic32 BlockingConstructor::constructor_called_ = 0;
149*6777b538SAndroid Build Coastguard Worker // static
150*6777b538SAndroid Build Coastguard Worker subtle::Atomic32 BlockingConstructor::complete_construction_ = 0;
151*6777b538SAndroid Build Coastguard Worker 
152*6777b538SAndroid Build Coastguard Worker // A SimpleThread running at |thread_type| which invokes |before_get| (optional)
153*6777b538SAndroid Build Coastguard Worker // and then invokes thread-safe scoped-static-initializationconstruction on its
154*6777b538SAndroid Build Coastguard Worker // NoDestructor instance.
155*6777b538SAndroid Build Coastguard Worker class BlockingConstructorThread : public SimpleThread {
156*6777b538SAndroid Build Coastguard Worker  public:
BlockingConstructorThread(ThreadType thread_type,OnceClosure before_get)157*6777b538SAndroid Build Coastguard Worker   BlockingConstructorThread(ThreadType thread_type, OnceClosure before_get)
158*6777b538SAndroid Build Coastguard Worker       : SimpleThread("BlockingConstructorThread", Options(thread_type)),
159*6777b538SAndroid Build Coastguard Worker         before_get_(std::move(before_get)) {}
160*6777b538SAndroid Build Coastguard Worker   BlockingConstructorThread(const BlockingConstructorThread&) = delete;
161*6777b538SAndroid Build Coastguard Worker   BlockingConstructorThread& operator=(const BlockingConstructorThread&) =
162*6777b538SAndroid Build Coastguard Worker       delete;
163*6777b538SAndroid Build Coastguard Worker 
Run()164*6777b538SAndroid Build Coastguard Worker   void Run() override {
165*6777b538SAndroid Build Coastguard Worker     if (before_get_)
166*6777b538SAndroid Build Coastguard Worker       std::move(before_get_).Run();
167*6777b538SAndroid Build Coastguard Worker 
168*6777b538SAndroid Build Coastguard Worker     static NoDestructor<BlockingConstructor> instance;
169*6777b538SAndroid Build Coastguard Worker     EXPECT_TRUE(instance->done_construction());
170*6777b538SAndroid Build Coastguard Worker   }
171*6777b538SAndroid Build Coastguard Worker 
172*6777b538SAndroid Build Coastguard Worker  private:
173*6777b538SAndroid Build Coastguard Worker   OnceClosure before_get_;
174*6777b538SAndroid Build Coastguard Worker };
175*6777b538SAndroid Build Coastguard Worker 
176*6777b538SAndroid Build Coastguard Worker }  // namespace
177*6777b538SAndroid Build Coastguard Worker 
178*6777b538SAndroid Build Coastguard Worker // Tests that if the thread assigned to construct the local-static
179*6777b538SAndroid Build Coastguard Worker // initialization of the NoDestructor runs at background priority : the
180*6777b538SAndroid Build Coastguard Worker // foreground threads will yield to it enough for it to eventually complete
181*6777b538SAndroid Build Coastguard Worker // construction. While local-static thread-safe initialization isn't specific to
182*6777b538SAndroid Build Coastguard Worker // NoDestructor, it is tested here as NoDestructor is set to replace
183*6777b538SAndroid Build Coastguard Worker // LazyInstance and this is an important regression test for it
184*6777b538SAndroid Build Coastguard Worker // (https://crbug.com/797129).
TEST(NoDestructorTest,PriorityInversionAtStaticInitializationResolves)185*6777b538SAndroid Build Coastguard Worker TEST(NoDestructorTest, PriorityInversionAtStaticInitializationResolves) {
186*6777b538SAndroid Build Coastguard Worker   TimeTicks test_begin = TimeTicks::Now();
187*6777b538SAndroid Build Coastguard Worker 
188*6777b538SAndroid Build Coastguard Worker   // Construct BlockingConstructor from a thread that is lower priority than the
189*6777b538SAndroid Build Coastguard Worker   // other threads that will be constructed. This thread used to be BACKGROUND
190*6777b538SAndroid Build Coastguard Worker   // priority but that caused it to be starved by other simultaneously running
191*6777b538SAndroid Build Coastguard Worker   // test processes, leading to false-positive failures.
192*6777b538SAndroid Build Coastguard Worker   BlockingConstructorThread background_getter(ThreadType::kDefault,
193*6777b538SAndroid Build Coastguard Worker                                               OnceClosure());
194*6777b538SAndroid Build Coastguard Worker   background_getter.Start();
195*6777b538SAndroid Build Coastguard Worker 
196*6777b538SAndroid Build Coastguard Worker   while (!BlockingConstructor::WasConstructorCalled())
197*6777b538SAndroid Build Coastguard Worker     PlatformThread::Sleep(Milliseconds(1));
198*6777b538SAndroid Build Coastguard Worker 
199*6777b538SAndroid Build Coastguard Worker   // Spin 4 foreground thread per core contending to get the already under
200*6777b538SAndroid Build Coastguard Worker   // construction NoDestructor. When they are all running and poking at it :
201*6777b538SAndroid Build Coastguard Worker   // allow the background thread to complete its work.
202*6777b538SAndroid Build Coastguard Worker   const int kNumForegroundThreads = 4 * SysInfo::NumberOfProcessors();
203*6777b538SAndroid Build Coastguard Worker   std::vector<std::unique_ptr<SimpleThread>> foreground_threads;
204*6777b538SAndroid Build Coastguard Worker   RepeatingClosure foreground_thread_ready_callback =
205*6777b538SAndroid Build Coastguard Worker       BarrierClosure(kNumForegroundThreads,
206*6777b538SAndroid Build Coastguard Worker                      BindOnce(&BlockingConstructor::CompleteConstructionNow));
207*6777b538SAndroid Build Coastguard Worker   for (int i = 0; i < kNumForegroundThreads; ++i) {
208*6777b538SAndroid Build Coastguard Worker     // Create threads that are higher priority than background_getter. See above
209*6777b538SAndroid Build Coastguard Worker     // for why these particular priorities are chosen.
210*6777b538SAndroid Build Coastguard Worker     foreground_threads.push_back(std::make_unique<BlockingConstructorThread>(
211*6777b538SAndroid Build Coastguard Worker         ThreadType::kDisplayCritical, foreground_thread_ready_callback));
212*6777b538SAndroid Build Coastguard Worker     foreground_threads.back()->Start();
213*6777b538SAndroid Build Coastguard Worker   }
214*6777b538SAndroid Build Coastguard Worker 
215*6777b538SAndroid Build Coastguard Worker   // This test will hang if the foreground threads become stuck in
216*6777b538SAndroid Build Coastguard Worker   // NoDestructor's construction per the background thread never being scheduled
217*6777b538SAndroid Build Coastguard Worker   // to complete construction.
218*6777b538SAndroid Build Coastguard Worker   for (auto& foreground_thread : foreground_threads)
219*6777b538SAndroid Build Coastguard Worker     foreground_thread->Join();
220*6777b538SAndroid Build Coastguard Worker   background_getter.Join();
221*6777b538SAndroid Build Coastguard Worker 
222*6777b538SAndroid Build Coastguard Worker   // Fail if this test takes more than 5 seconds (it takes 5-10 seconds on a
223*6777b538SAndroid Build Coastguard Worker   // Z840 without https://crrev.com/527445 but is expected to be fast (~30ms)
224*6777b538SAndroid Build Coastguard Worker   // with the fix).
225*6777b538SAndroid Build Coastguard Worker   EXPECT_LT(TimeTicks::Now() - test_begin, Seconds(5));
226*6777b538SAndroid Build Coastguard Worker }
227*6777b538SAndroid Build Coastguard Worker 
228*6777b538SAndroid Build Coastguard Worker }  // namespace base
229