Enable LayoutNG for checkboxes and radio buttons

blink::BaseCheckableInputType is the super class of
blink::CheckboxInputType and blink::RadioInputType, and it has no
other subclasses.

This CL should have no behavior changes.

Bug: 1040826
Change-Id: I706672c9ff34544adbe068f7bd16d143c98c7d05
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2000397
Commit-Queue: Kent Tamura <tkent@chromium.org>
Reviewed-by: Yoshifumi Inoue <yosin@chromium.org>
Reviewed-by: Koji Ishii <kojii@chromium.org>
Cr-Commit-Position: refs/heads/master@{#731851}

third_party/blink/renderer/core/html/forms/base_checkable_input_type.cc

@@ -121,4 +121,8 @@ bool BaseCheckableInputType::IsCheckable() {
   return true;
 }
 
+bool BaseCheckableInputType::TypeShouldForceLegacyLayout() const {
+  return false;
+}
+
 }  // namespace blink

third_party/blink/renderer/core/html/forms/base_checkable_input_type.h

@@ -70,6 +70,7 @@ class BaseCheckableInputType : public InputType, public InputTypeView {
                 TextControlSetValueSelection) final;
   void ReadingChecked() const final;
   bool IsCheckable() final;
+  bool TypeShouldForceLegacyLayout() const final;
 };
 
 }  // namespace blink

third_party/blink/renderer/core/html/forms/html_form_control_element_with_state.h

@@ -61,7 +61,7 @@ class CORE_EXPORT HTMLFormControlElementWithState
   bool IsFormControlElementWithState() const final;
 
  private:
-  bool TypeShouldForceLegacyLayout() const final { return true; }
+  bool TypeShouldForceLegacyLayout() const override { return true; }
   int DefaultTabIndex() const override;
 
   // https://html.spec.whatwg.org/C/#autofill-anchor-mantle

third_party/blink/renderer/core/html/forms/html_input_element.cc

@@ -882,6 +882,11 @@ bool HTMLInputElement::LayoutObjectIsNeeded(const ComputedStyle& style) const {
          TextControlElement::LayoutObjectIsNeeded(style);
 }
 
+// TODO(crbug.com/1040826): Remove this override.
+bool HTMLInputElement::TypeShouldForceLegacyLayout() const {
+  return input_type_view_->TypeShouldForceLegacyLayout();
+}
+
 LayoutObject* HTMLInputElement::CreateLayoutObject(const ComputedStyle& style,
                                                    LegacyLayout legacy) {
   return input_type_view_->CreateLayoutObject(style, legacy);

third_party/blink/renderer/core/html/forms/html_input_element.h

@@ -365,6 +365,7 @@ class CORE_EXPORT HTMLInputElement
 
   void CloneNonAttributePropertiesFrom(const Element&, CloneChildrenFlag) final;
 
+  bool TypeShouldForceLegacyLayout() const final;
   void AttachLayoutTree(AttachContext&) final;
 
   void AppendToFormData(FormData&) final;

third_party/blink/renderer/core/html/forms/input_type_view.cc

@@ -87,6 +87,10 @@ HTMLFormElement* InputTypeView::FormForSubmission() const {
   return GetElement().Form();
 }
 
+bool InputTypeView::TypeShouldForceLegacyLayout() const {
+  return true;
+}
+
 LayoutObject* InputTypeView::CreateLayoutObject(const ComputedStyle& style,
                                                 LegacyLayout legacy) const {
   return LayoutObject::CreateObject(&GetElement(), style, legacy);

third_party/blink/renderer/core/html/forms/input_type_view.h

@@ -99,6 +99,7 @@ class CORE_EXPORT InputTypeView : public GarbageCollectedMixin {
   void DispatchSimulatedClickIfActive(KeyboardEvent&) const;
 
   virtual void SubtreeHasChanged();
+  virtual bool TypeShouldForceLegacyLayout() const;
   virtual LayoutObject* CreateLayoutObject(const ComputedStyle&,
                                            LegacyLayout) const;
   virtual scoped_refptr<ComputedStyle> CustomStyleForLayoutObject(