diff --git a/frameworks/core/components_ng/layout/layout_property.cpp b/frameworks/core/components_ng/layout/layout_property.cpp index 088a9b74ba1f23b72cdd065cd7f81c2929acb27b..1bc71301bd5540863e79160de9f8d6df7feea844 100644 --- a/frameworks/core/components_ng/layout/layout_property.cpp +++ b/frameworks/core/components_ng/layout/layout_property.cpp @@ -941,7 +941,6 @@ PaddingPropertyF LayoutProperty::CreateSafeAreaPadding(bool adjustingRound) auto host = GetHost(); auto pipeline = host ? host->GetContext() : nullptr; ScaleProperty scaleProperty = ScaleProperty::CreateScaleProperty(pipeline); - auto rootWidth = pipeline ? pipeline->GetRootWidth() : PipelineContext::GetCurrentRootWidth(); if (layoutConstraint_.has_value()) { std::optional contentWithSafeArea = layoutConstraint_.value(); PaddingPropertyF roundOffFraction; @@ -962,6 +961,7 @@ PaddingPropertyF LayoutProperty::CreateSafeAreaPadding(bool adjustingRound) isRtl ? truncatedSafeAreaPadding.left : truncatedSafeAreaPadding.right); return truncatedSafeAreaPadding; } + auto rootWidth = pipeline ? pipeline->GetRootWidth() : PipelineContext::GetCurrentRootWidth(); return ConvertToPaddingPropertyF( safeAreaPadding_, scaleProperty, rootWidth, true, true); } @@ -974,7 +974,6 @@ PaddingPropertyF LayoutProperty::CreatePaddingAndBorder(bool includeSafeAreaPadd } auto host = GetHost(); auto pipeline = host ? host->GetContext() : nullptr; - auto rootWidth = pipeline ? pipeline->GetRootWidth() : PipelineContext::GetCurrentRootWidth(); ScaleProperty scaleProperty = ScaleProperty::CreateScaleProperty(pipeline); if (layoutConstraint_.has_value()) { auto padding = ConvertToPaddingPropertyF(padding_, scaleProperty, layoutConstraint_->percentReference.Width()); @@ -988,6 +987,7 @@ PaddingPropertyF LayoutProperty::CreatePaddingAndBorder(bool includeSafeAreaPadd } return CombinePaddingsAndBorder(safeAreaPadding, padding, borderWidth, {}); } + auto rootWidth = pipeline ? pipeline->GetRootWidth() : PipelineContext::GetCurrentRootWidth(); auto padding = ConvertToPaddingPropertyF(padding_, scaleProperty, rootWidth); auto borderWidth = ConvertToBorderWidthPropertyF(borderWidth_, scaleProperty, rootWidth); @@ -1005,13 +1005,13 @@ PaddingPropertyF LayoutProperty::CreatePaddingAndBorderWithDefault(float padding auto host = GetHost(); auto pipeline = host ? host->GetContext() : nullptr; ScaleProperty scaleProperty = ScaleProperty::CreateScaleProperty(pipeline); - auto rootWidth = pipeline ? pipeline->GetRootWidth() : PipelineContext::GetCurrentRootWidth(); if (layoutConstraint_.has_value()) { auto padding = ConvertToPaddingPropertyF(padding_, scaleProperty, layoutConstraint_->percentReference.Width()); auto borderWidth = ConvertToBorderWidthPropertyF(borderWidth_, scaleProperty, layoutConstraint_->percentReference.Width()); return CombinePaddingsAndBorder(safeAreaPadding, padding, borderWidth, defaultParem); } + auto rootWidth = pipeline ? pipeline->GetRootWidth() : PipelineContext::GetCurrentRootWidth(); auto padding = ConvertToPaddingPropertyF(padding_, scaleProperty, rootWidth); auto borderWidth = ConvertToBorderWidthPropertyF(borderWidth_, scaleProperty, rootWidth); diff --git a/interfaces/ets/ani/mediaquery/src/mediaquery.cpp b/interfaces/ets/ani/mediaquery/src/mediaquery.cpp index 5b77933584d02dfbdae687958af11818d3c9f93d..dc1b17b2d632fae9709e782a6d449f9d09d7c42f 100644 --- a/interfaces/ets/ani/mediaquery/src/mediaquery.cpp +++ b/interfaces/ets/ani/mediaquery/src/mediaquery.cpp @@ -149,7 +149,7 @@ public: ani_ref ref; ani_boolean wasReleased; listener->env_->WeakReference_GetReference(cbWref, &wasReleased, &ref); - ani_object result; + ani_object result = {}; listener->MediaQueryResult::AniSerializer(listener->env_, result); ani_ref resultRef = static_cast(result); listener->env_->FunctionalObject_Call(static_cast(ref), 1, &resultRef, nullptr); @@ -341,7 +341,7 @@ private: return TWO_ARGS; } - ani_env *env_; + ani_env *env_ = nullptr; std::list cbList_; static std::set>* delayDeleteListenerSets_; static std::set* delayDeleteCallbacks_; diff --git a/test/unittest/core/manager/safe_area_manager_test_ng.cpp b/test/unittest/core/manager/safe_area_manager_test_ng.cpp index 4a8b36503a54d83c9d55dd7aa8d3f849d5ec2886..0797af2c873862003fb462a29a15f870405616a9 100644 --- a/test/unittest/core/manager/safe_area_manager_test_ng.cpp +++ b/test/unittest/core/manager/safe_area_manager_test_ng.cpp @@ -1385,6 +1385,7 @@ HWTEST_F(SafeAreaManagerTest, AddNodeToExpandListIfNeededTest, TestSize.Level1) EXPECT_EQ(safeAreaManager_->AddNodeToExpandListIfNeeded(frameNode2), false); EXPECT_EQ(safeAreaManager_->AddNodeToExpandListIfNeeded(frameNode3), false); EXPECT_EQ(safeAreaManager_->AddNodeToExpandListIfNeeded(frameNode4), false); + EXPECT_EQ(safeAreaManager_->GetExpandNodeSet().size(), 5); safeAreaManager_->ClearNeedExpandNode(); EXPECT_EQ(safeAreaManager_->GetExpandNodeSet().size(), 0);