Commit 717f6a62 authored by Yoshifumi Inoue's avatar Yoshifumi Inoue Committed by Commit Bot

[NGFragmentItem] Adapt LayoutInlineTest.RegionHitTest to NGFragmentItem

This patch changes |LayoutInlineTest.RegionHitTest| to check
having inline boxes instead of |HitTestCulledInline()| becuase
|NGFragmentItem| doesn't use |HitTestCulledInline()|.

Bug: 982194
Change-Id: Ib62395142113900a36d4451a7896e2191916ec11
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2108210Reviewed-by: default avatarKent Tamura <tkent@chromium.org>
Commit-Queue: Kent Tamura <tkent@chromium.org>
Commit-Queue: Yoshifumi Inoue <yosin@chromium.org>
Auto-Submit: Yoshifumi Inoue <yosin@chromium.org>
Cr-Commit-Position: refs/heads/master@{#751254}
parent 5c69afb2
...@@ -8,6 +8,7 @@ ...@@ -8,6 +8,7 @@
#include "testing/gmock/include/gmock/gmock.h" #include "testing/gmock/include/gmock/gmock.h"
#include "testing/gtest/include/gtest/gtest.h" #include "testing/gtest/include/gtest/gtest.h"
#include "third_party/blink/renderer/core/layout/layout_block_flow.h" #include "third_party/blink/renderer/core/layout/layout_block_flow.h"
#include "third_party/blink/renderer/core/layout/ng/inline/ng_inline_cursor.h"
#include "third_party/blink/renderer/core/testing/core_unit_test_helper.h" #include "third_party/blink/renderer/core/testing/core_unit_test_helper.h"
#include "third_party/blink/renderer/platform/runtime_enabled_features.h" #include "third_party/blink/renderer/platform/runtime_enabled_features.h"
#include "third_party/blink/renderer/platform/testing/runtime_enabled_features_test_helpers.h" #include "third_party/blink/renderer/platform/testing/runtime_enabled_features_test_helpers.h"
...@@ -110,6 +111,14 @@ TEST_F(LayoutInlineTest, RegionHitTest) { ...@@ -110,6 +111,14 @@ TEST_F(LayoutInlineTest, RegionHitTest) {
ToLayoutInline(GetLayoutObjectByElementId("lotsOfBoxes")); ToLayoutInline(GetLayoutObjectByElementId("lotsOfBoxes"));
ASSERT_TRUE(lots_of_boxes); ASSERT_TRUE(lots_of_boxes);
if (RuntimeEnabledFeatures::LayoutNGFragmentItemEnabled()) {
NGInlineCursor cursor;
cursor.MoveTo(*lots_of_boxes);
ASSERT_TRUE(cursor);
EXPECT_EQ(lots_of_boxes, cursor.Current().GetLayoutObject());
return;
}
HitTestRequest hit_request(HitTestRequest::kTouchEvent | HitTestRequest hit_request(HitTestRequest::kTouchEvent |
HitTestRequest::kListBased); HitTestRequest::kListBased);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment