Use "if" check to avoid crash

Use "if" instead of "CHECK" to avoid crash at MayHaveHTMLLabel

Change-Id: Ia30df38b994ccf418967fc31f0ceb0dbadfda65d
Signed-off-by: fr.fang <fr.fang@samsung.com>

diff --git a/third_party/blink/renderer/modules/accessibility/ax_object_cache_impl.cc b/third_party/blink/renderer/modules/accessibility/ax_object_cache_impl.cc
index ced89edd3e567f660bc7ce2219bd212ed61836ae..3798d92b6bf9e7ce0cd98e7323c5ebbddfff5af2 100644 (file)
--- a/third_party/blink/renderer/modules/accessibility/ax_object_cache_impl.cc
+++ b/third_party/blink/renderer/modules/accessibility/ax_object_cache_impl.cc
@@ -3352,10 +3352,15 @@ void AXObjectCacheImpl::ValidatedAriaOwnedChildren(
 }
 
 bool AXObjectCacheImpl::MayHaveHTMLLabel(const HTMLElement& elem) {
-  CHECK(elem.GetDocument().Lifecycle().GetState() >=
-        DocumentLifecycle::kLayoutClean)
-      << "Unclean document at lifecycle " << elem.GetDocument().ToString();
-  CHECK(relation_cache_);
+  if (!(elem.GetDocument().Lifecycle().GetState() >=
+        DocumentLifecycle::kLayoutClean)) {
+    LOG(ERROR) << "Unclean document at lifecycle "
+               << elem.GetDocument().ToString();
+    return false;
+  }
+
+  if (!relation_cache_)
+    return false;
 
   // Return false if this type of element will not accept a <label for> label.
   if (!elem.IsLabelable())