Use InitialStyleForElement for invisible elements

We previously used ComputedStyle::Create when creating ComputedStyle for
invisible elements, but the ComputedStyle made with that lacks some
setup needed for it to actually be usable correctly. Example case is
the style's font fallback list is not set, leading to some crashes.

Bug: 904791
Change-Id: I512082af6dcf7a5076eb033a2f9a78b83020e47e
Reviewed-on: https://chromium-review.googlesource.com/c/1345952
Commit-Queue: Rakina Zata Amni <rakina@chromium.org>
Reviewed-by: Rune Lillesveen <futhark@chromium.org>
Cr-Commit-Position: refs/heads/master@{#610322}

third_party/blink/renderer/core/css/resolver/style_resolver.h

@@ -75,6 +75,8 @@ class CORE_EXPORT StyleResolver final
       const ComputedStyle* layout_parent_style = nullptr,
       RuleMatchingBehavior = kMatchAllRules);
 
+  static scoped_refptr<ComputedStyle> InitialStyleForElement(Document&);
+
   static AnimatableValue* CreateAnimatableValueSnapshot(
       Element&,
       const ComputedStyle& base_style,
@@ -138,8 +140,6 @@ class CORE_EXPORT StyleResolver final
  private:
   explicit StyleResolver(Document&);
 
-  static scoped_refptr<ComputedStyle> InitialStyleForElement(Document&);
-
   // FIXME: This should probably go away, folded into FontBuilder.
   void UpdateFont(StyleResolverState&);
 

third_party/blink/renderer/core/dom/element.cc

@@ -2145,7 +2145,8 @@ scoped_refptr<ComputedStyle> Element::StyleForLayoutObject() {
 
   if (RuntimeEnabledFeatures::InvisibleDOMEnabled() &&
       hasAttribute(html_names::kInvisibleAttr)) {
-    auto style = ComputedStyle::Create();
+    auto style =
+        GetDocument().GetStyleResolver()->InitialStyleForElement(GetDocument());
     style->SetDisplay(EDisplay::kNone);
     return style;
   }