From 5ce2c0144b70270762cc991593f45bb89a731efa Mon Sep 17 00:00:00 2001 From: Valentin Shergin Date: Fri, 28 Sep 2018 14:46:29 -0700 Subject: [PATCH] Fabric: Fixed a bug in LayoutMetrics::operator== Summary: Trivial. We missed `pointScaleFactor`. Reviewed By: mdvacca Differential Revision: D10112051 fbshipit-source-id: 980b8c310fbb3701008765509dce5b6e61852c0e --- ReactCommon/fabric/core/layout/LayoutMetrics.h | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/ReactCommon/fabric/core/layout/LayoutMetrics.h b/ReactCommon/fabric/core/layout/LayoutMetrics.h index 6d4b9891942991..9e519221c2335f 100644 --- a/ReactCommon/fabric/core/layout/LayoutMetrics.h +++ b/ReactCommon/fabric/core/layout/LayoutMetrics.h @@ -31,13 +31,13 @@ struct LayoutMetrics { }; } - bool operator ==(const LayoutMetrics& rhs) const { + bool operator==(const LayoutMetrics &rhs) const { return - std::tie(this->frame, this->contentInsets, this->borderWidth, this->displayType, this->layoutDirection) == - std::tie(rhs.frame, rhs.contentInsets, rhs.borderWidth, rhs.displayType, rhs.layoutDirection); + std::tie(this->frame, this->contentInsets, this->borderWidth, this->displayType, this->layoutDirection, this->pointScaleFactor) == + std::tie(rhs.frame, rhs.contentInsets, rhs.borderWidth, rhs.displayType, rhs.layoutDirection, this->pointScaleFactor); } - bool operator !=(const LayoutMetrics& rhs) const { + bool operator!=(const LayoutMetrics &rhs) const { return !(*this == rhs); } };