Changeset 224320 in webkit

Timestamp:

Nov 1, 2017 10:43:02 PM (6 years ago)

Author:

jiewen_tan@apple.com

Message:

Let is<T>() accept RefPtrs
​https://bugs.webkit.org/show_bug.cgi?id=178612
<rdar://problem/35102004>

Reviewed by Ryosuke Niwa.

Source/WebCore:

Get rid of .get() for all is<T>() calls.

No changes in behaviors.

• bindings/js/JSElementCustom.cpp:

(WebCore::createNewElementWrapper):

• bindings/js/JSMediaStreamTrackCustom.cpp:

(WebCore::toJSNewlyCreated):

• bindings/js/JSNodeCustom.cpp:

(WebCore::createWrapperInline):

• bindings/js/JSTextCustom.cpp:

(WebCore::toJSNewlyCreated):

• css/CSSFilterImageValue.cpp:

(WebCore::CSSFilterImageValue::loadSubimages):

• css/CSSFontFaceSet.cpp:

(WebCore::CSSFontFaceSet::matchingFaces):

• css/CSSFontSelector.cpp:

(WebCore::CSSFontSelector::addFontFaceRule):

• css/CSSToStyleMap.cpp:

(WebCore::CSSToStyleMap::mapNinePieceImage):

• css/FontFace.cpp:

(WebCore::FontFace::create):

• css/StyleBuilderConverter.h:

(WebCore::StyleBuilderConverter::createGridTrackList):

• css/StyleBuilderCustom.h:

(WebCore::StyleBuilderCustom::applyValueCursor):
(WebCore::StyleBuilderCustom::applyValueContent):
(WebCore::StyleBuilderCustom::applyValueWillChange):

• css/StyleProperties.cpp:

(WebCore::StyleProperties::propertyAsColor const):
(WebCore::StyleProperties::propertyAsValueID const):

• css/StyleResolver.cpp:

(WebCore::StyleResolver::createFilterOperations):

• css/TransformFunctions.cpp:

(WebCore::transformsForValue):

• css/ViewportStyleResolver.cpp:

(WebCore::ViewportStyleResolver::getViewportArgumentValue const):

• dom/Document.cpp:

(WebCore::Document::setBodyOrFrameset):
(WebCore::Document::setFocusedElement):
(WebCore::Document::updateHoverActiveState):

• dom/Element.cpp:

(WebCore::Element::setOuterHTML):

• dom/Range.cpp:

(WebCore::Range::insertNode):

• editing/ApplyStyleCommand.cpp:

(WebCore::toIdentifier):
(WebCore::ApplyStyleCommand::applyBlockStyle):
(WebCore::ApplyStyleCommand::surroundNodeRangeWithElement):

• editing/EditingStyle.cpp:

(WebCore::identifierForStyleProperty):
(WebCore::HTMLElementEquivalent::valueIsPresentInStyle const):
(WebCore::EditingStyle::extractFontSizeDelta):
(WebCore::EditingStyle::textDirection const):
(WebCore::textDecorationValueList):
(WebCore::EditingStyle::prepareToApplyAt):
(WebCore::EditingStyle::legacyFontSize const):
(WebCore::EditingStyle::textDirectionForSelection):
(WebCore::StyleChange::StyleChange):
(WebCore::StyleChange::extractTextStyles):
(WebCore::diffTextDecorations):

• html/BaseChooserOnlyDateAndTimeInputType.cpp:

(WebCore::BaseChooserOnlyDateAndTimeInputType::updateAppearance):

• html/FormAssociatedElement.cpp:

(WebCore::FormAssociatedElement::findAssociatedForm):

• html/HTMLAreaElement.cpp:

(WebCore::HTMLAreaElement::imageElement const):

• html/HTMLElement.cpp:

(WebCore::HTMLElement::setOuterText):
(WebCore::HTMLElement::dirAttributeChanged):

• html/HTMLEmbedElement.cpp:

(WebCore::HTMLEmbedElement::rendererIsNeeded):

• html/HTMLFormControlElement.cpp:

(WebCore::HTMLFormControlElement::computeIsDisabledByFieldsetAncestor const):

• html/HTMLInputElement.cpp:

(WebCore::HTMLInputElement::dataList const):

• html/HTMLOptionElement.cpp:

(WebCore::HTMLOptionElement::setText):
(WebCore::HTMLOptionElement::textIndentedToRespectGroupLabel const):

• html/HTMLPlugInElement.cpp:

(WebCore::HTMLPlugInElement::isKeyboardFocusable const):
(WebCore::HTMLPlugInElement::isUserObservable const):

• html/HTMLSelectElement.cpp:

(WebCore::HTMLSelectElement::selectOption):

• html/HTMLSummaryElement.cpp:

(WebCore::isClickableControl):

• html/HTMLTextFormControlElement.cpp:

(WebCore::HTMLTextFormControlElement::adjustInnerTextStyle const):

• html/RadioInputType.cpp:

(WebCore::RadioInputType::isKeyboardFocusable const):

• html/parser/HTMLConstructionSite.cpp:

(WebCore::HTMLConstructionSite::attachLater):
(WebCore::HTMLConstructionSite::insertTextNode):

• html/shadow/MediaControlElements.cpp:

(WebCore::MediaControlClosedCaptionsTrackListElement::defaultEventHandler):

• html/track/TextTrackList.cpp:

(TextTrackList::append):

• inspector/InspectorDOMAgent.cpp:

(WebCore::InspectorDOMAgent::setNodeName):

• inspector/PageScriptDebugServer.cpp:

(WebCore::PageScriptDebugServer::setJavaScriptPaused):

• page/FocusController.cpp:

(WebCore::shouldClearSelectionWhenChangingFocusedElement):

• page/FrameView.cpp:

(WebCore::FrameView::hasCustomScrollbars const):

• page/Page.cpp:

(WebCore::Page::pluginViews):

• page/mac/EventHandlerMac.mm:

(WebCore::EventHandler::platformPrepareForWheelEvents):

• rendering/RenderBoxModelObject.cpp:

(WebCore::RenderBoxModelObject::paintFillLayerExtended):

• testing/Internals.cpp:

(WebCore::deferredStyleRulesCountForList):
(WebCore::deferredGroupRulesCountForList):
(WebCore::deferredKeyframesRulesCountForList):

Source/WTF:

Add support for is<T>() for RefPtrs.

• wtf/RefPtr.h:

(WTF::is):

Location:

trunk/Source

Files:

• WTF/ChangeLog (modified) (1 diff)
• WTF/wtf/RefPtr.h (modified) (1 diff)
• WebCore/ChangeLog (modified) (1 diff)
• WebCore/bindings/js/JSElementCustom.cpp (modified) (1 diff)
• WebCore/bindings/js/JSMediaStreamTrackCustom.cpp (modified) (1 diff)
• WebCore/bindings/js/JSNodeCustom.cpp (modified) (1 diff)
• WebCore/bindings/js/JSTextCustom.cpp (modified) (1 diff)
• WebCore/css/CSSFilterImageValue.cpp (modified) (1 diff)
• WebCore/css/CSSFontFaceSet.cpp (modified) (1 diff)
• WebCore/css/CSSFontSelector.cpp (modified) (1 diff)
• WebCore/css/CSSToStyleMap.cpp (modified) (2 diffs)
• WebCore/css/FontFace.cpp (modified) (1 diff)
• WebCore/css/StyleBuilderConverter.h (modified) (2 diffs)
• WebCore/css/StyleBuilderCustom.h (modified) (4 diffs)
• WebCore/css/StyleProperties.cpp (modified) (2 diffs)
• WebCore/css/StyleResolver.cpp (modified) (2 diffs)
• WebCore/css/TransformFunctions.cpp (modified) (1 diff)
• WebCore/css/ViewportStyleResolver.cpp (modified) (1 diff)
• WebCore/dom/Document.cpp (modified) (3 diffs)
• WebCore/dom/Element.cpp (modified) (1 diff)
• WebCore/dom/Range.cpp (modified) (1 diff)
• WebCore/editing/ApplyStyleCommand.cpp (modified) (3 diffs)
• WebCore/editing/EditingStyle.cpp (modified) (15 diffs)
• WebCore/html/BaseChooserOnlyDateAndTimeInputType.cpp (modified) (1 diff)
• WebCore/html/FormAssociatedElement.cpp (modified) (1 diff)
• WebCore/html/HTMLAreaElement.cpp (modified) (1 diff)
• WebCore/html/HTMLElement.cpp (modified) (2 diffs)
• WebCore/html/HTMLEmbedElement.cpp (modified) (1 diff)
• WebCore/html/HTMLFormControlElement.cpp (modified) (1 diff)
• WebCore/html/HTMLInputElement.cpp (modified) (1 diff)
• WebCore/html/HTMLOptionElement.cpp (modified) (2 diffs)
• WebCore/html/HTMLPlugInElement.cpp (modified) (2 diffs)
• WebCore/html/HTMLSelectElement.cpp (modified) (1 diff)
• WebCore/html/HTMLSummaryElement.cpp (modified) (1 diff)
• WebCore/html/HTMLTextFormControlElement.cpp (modified) (1 diff)
• WebCore/html/RadioInputType.cpp (modified) (1 diff)
• WebCore/html/parser/HTMLConstructionSite.cpp (modified) (2 diffs)
• WebCore/html/shadow/MediaControlElements.cpp (modified) (1 diff)
• WebCore/html/track/TextTrackList.cpp (modified) (1 diff)
• WebCore/inspector/InspectorDOMAgent.cpp (modified) (1 diff)
• WebCore/inspector/PageScriptDebugServer.cpp (modified) (1 diff)
• WebCore/page/FocusController.cpp (modified) (1 diff)
• WebCore/page/FrameView.cpp (modified) (1 diff)
• WebCore/page/Page.cpp (modified) (1 diff)
• WebCore/page/mac/EventHandlerMac.mm (modified) (1 diff)
• WebCore/rendering/RenderBoxModelObject.cpp (modified) (1 diff)
• WebCore/testing/Internals.cpp (modified) (5 diffs)

trunk/Source/WTF/ChangeLog

@@ +1 @@
+2017-11-01  Jiewen Tan  <jiewen_tan@apple.com>
+
+        Let is<T>() accept RefPtrs
+        https://bugs.webkit.org/show_bug.cgi?id=178612
+        <rdar://problem/35102004>
+
+        Reviewed by Ryosuke Niwa.
+
+        Add support for is<T>() for RefPtrs.
+
+        * wtf/RefPtr.h:
+        (WTF::is):
+
 2017-11-01  Simon Fraser  <simon.fraser@apple.com>
 

trunk/Source/WTF/wtf/RefPtr.h

@@ -239 +239 @@
 }
 
+template<typename ExpectedType, typename ArgType> inline bool is(RefPtr<ArgType>& source)
+{
+    return is<ExpectedType>(source.get());
+}
+
+template<typename ExpectedType, typename ArgType> inline bool is(const RefPtr<ArgType>& source)
+{
+    return is<ExpectedType>(source.get());
+}
+
 } // namespace WTF
 

trunk/Source/WebCore/ChangeLog

@@ +1 @@
+2017-11-01  Jiewen Tan  <jiewen_tan@apple.com>
+
+        Let is<T>() accept RefPtrs
+        https://bugs.webkit.org/show_bug.cgi?id=178612
+        <rdar://problem/35102004>
+
+        Reviewed by Ryosuke Niwa.
+
+        Get rid of .get() for all is<T>() calls.
+
+        No changes in behaviors.
+
+        * bindings/js/JSElementCustom.cpp:
+        (WebCore::createNewElementWrapper):
+        * bindings/js/JSMediaStreamTrackCustom.cpp:
+        (WebCore::toJSNewlyCreated):
+        * bindings/js/JSNodeCustom.cpp:
+        (WebCore::createWrapperInline):
+        * bindings/js/JSTextCustom.cpp:
+        (WebCore::toJSNewlyCreated):
+        * css/CSSFilterImageValue.cpp:
+        (WebCore::CSSFilterImageValue::loadSubimages):
+        * css/CSSFontFaceSet.cpp:
+        (WebCore::CSSFontFaceSet::matchingFaces):
+        * css/CSSFontSelector.cpp:
+        (WebCore::CSSFontSelector::addFontFaceRule):
+        * css/CSSToStyleMap.cpp:
+        (WebCore::CSSToStyleMap::mapNinePieceImage):
+        * css/FontFace.cpp:
+        (WebCore::FontFace::create):
+        * css/StyleBuilderConverter.h:
+        (WebCore::StyleBuilderConverter::createGridTrackList):
+        * css/StyleBuilderCustom.h:
+        (WebCore::StyleBuilderCustom::applyValueCursor):
+        (WebCore::StyleBuilderCustom::applyValueContent):
+        (WebCore::StyleBuilderCustom::applyValueWillChange):
+        * css/StyleProperties.cpp:
+        (WebCore::StyleProperties::propertyAsColor const):
+        (WebCore::StyleProperties::propertyAsValueID const):
+        * css/StyleResolver.cpp:
+        (WebCore::StyleResolver::createFilterOperations):
+        * css/TransformFunctions.cpp:
+        (WebCore::transformsForValue):
+        * css/ViewportStyleResolver.cpp:
+        (WebCore::ViewportStyleResolver::getViewportArgumentValue const):
+        * dom/Document.cpp:
+        (WebCore::Document::setBodyOrFrameset):
+        (WebCore::Document::setFocusedElement):
+        (WebCore::Document::updateHoverActiveState):
+        * dom/Element.cpp:
+        (WebCore::Element::setOuterHTML):
+        * dom/Range.cpp:
+        (WebCore::Range::insertNode):
+        * editing/ApplyStyleCommand.cpp:
+        (WebCore::toIdentifier):
+        (WebCore::ApplyStyleCommand::applyBlockStyle):
+        (WebCore::ApplyStyleCommand::surroundNodeRangeWithElement):
+        * editing/EditingStyle.cpp:
+        (WebCore::identifierForStyleProperty):
+        (WebCore::HTMLElementEquivalent::valueIsPresentInStyle const):
+        (WebCore::EditingStyle::extractFontSizeDelta):
+        (WebCore::EditingStyle::textDirection const):
+        (WebCore::textDecorationValueList):
+        (WebCore::EditingStyle::prepareToApplyAt):
+        (WebCore::EditingStyle::legacyFontSize const):
+        (WebCore::EditingStyle::textDirectionForSelection):
+        (WebCore::StyleChange::StyleChange):
+        (WebCore::StyleChange::extractTextStyles):
+        (WebCore::diffTextDecorations):
+        * html/BaseChooserOnlyDateAndTimeInputType.cpp:
+        (WebCore::BaseChooserOnlyDateAndTimeInputType::updateAppearance):
+        * html/FormAssociatedElement.cpp:
+        (WebCore::FormAssociatedElement::findAssociatedForm):
+        * html/HTMLAreaElement.cpp:
+        (WebCore::HTMLAreaElement::imageElement const):
+        * html/HTMLElement.cpp:
+        (WebCore::HTMLElement::setOuterText):
+        (WebCore::HTMLElement::dirAttributeChanged):
+        * html/HTMLEmbedElement.cpp:
+        (WebCore::HTMLEmbedElement::rendererIsNeeded):
+        * html/HTMLFormControlElement.cpp:
+        (WebCore::HTMLFormControlElement::computeIsDisabledByFieldsetAncestor const):
+        * html/HTMLInputElement.cpp:
+        (WebCore::HTMLInputElement::dataList const):
+        * html/HTMLOptionElement.cpp:
+        (WebCore::HTMLOptionElement::setText):
+        (WebCore::HTMLOptionElement::textIndentedToRespectGroupLabel const):
+        * html/HTMLPlugInElement.cpp:
+        (WebCore::HTMLPlugInElement::isKeyboardFocusable const):
+        (WebCore::HTMLPlugInElement::isUserObservable const):
+        * html/HTMLSelectElement.cpp:
+        (WebCore::HTMLSelectElement::selectOption):
+        * html/HTMLSummaryElement.cpp:
+        (WebCore::isClickableControl):
+        * html/HTMLTextFormControlElement.cpp:
+        (WebCore::HTMLTextFormControlElement::adjustInnerTextStyle const):
+        * html/RadioInputType.cpp:
+        (WebCore::RadioInputType::isKeyboardFocusable const):
+        * html/parser/HTMLConstructionSite.cpp:
+        (WebCore::HTMLConstructionSite::attachLater):
+        (WebCore::HTMLConstructionSite::insertTextNode):
+        * html/shadow/MediaControlElements.cpp:
+        (WebCore::MediaControlClosedCaptionsTrackListElement::defaultEventHandler):
+        * html/track/TextTrackList.cpp:
+        (TextTrackList::append):
+        * inspector/InspectorDOMAgent.cpp:
+        (WebCore::InspectorDOMAgent::setNodeName):
+        * inspector/PageScriptDebugServer.cpp:
+        (WebCore::PageScriptDebugServer::setJavaScriptPaused):
+        * page/FocusController.cpp:
+        (WebCore::shouldClearSelectionWhenChangingFocusedElement):
+        * page/FrameView.cpp:
+        (WebCore::FrameView::hasCustomScrollbars const):
+        * page/Page.cpp:
+        (WebCore::Page::pluginViews):
+        * page/mac/EventHandlerMac.mm:
+        (WebCore::EventHandler::platformPrepareForWheelEvents):
+        * rendering/RenderBoxModelObject.cpp:
+        (WebCore::RenderBoxModelObject::paintFillLayerExtended):
+        * testing/Internals.cpp:
+        (WebCore::deferredStyleRulesCountForList):
+        (WebCore::deferredGroupRulesCountForList):
+        (WebCore::deferredKeyframesRulesCountForList):
+
 2017-11-01  Fujii Hironori  <Hironori.Fujii@sony.com>
 

trunk/Source/WebCore/bindings/js/JSElementCustom.cpp

@@ -50 +50 @@
 static JSValue createNewElementWrapper(JSDOMGlobalObject* globalObject, Ref<Element>&& element)
 {
-    if (is<HTMLElement>(element.get()))
+    if (is<HTMLElement>(element))
         return createJSHTMLWrapper(globalObject, static_reference_cast<HTMLElement>(WTFMove(element)));
-    if (is<SVGElement>(element.get()))
+    if (is<SVGElement>(element))
         return createJSSVGWrapper(globalObject, static_reference_cast<SVGElement>(WTFMove(element)));
     return createWrapper<Element>(globalObject, WTFMove(element));

trunk/Source/WebCore/bindings/js/JSMediaStreamTrackCustom.cpp

@@ -36 +36 @@
 JSC::JSValue toJSNewlyCreated(JSC::ExecState*, JSDOMGlobalObject* globalObject, Ref<MediaStreamTrack>&& object)
 {
-    if (is<CanvasCaptureMediaStreamTrack>(object.get()))
+    if (is<CanvasCaptureMediaStreamTrack>(object))
         return createWrapper<CanvasCaptureMediaStreamTrack>(globalObject, WTFMove(object));
     return createWrapper<MediaStreamTrack>(globalObject, WTFMove(object));

trunk/Source/WebCore/bindings/js/JSNodeCustom.cpp

@@ -129 +129 @@
     switch (node->nodeType()) {
         case Node::ELEMENT_NODE:
-            if (is<HTMLElement>(node.get()))
+            if (is<HTMLElement>(node))
                 wrapper = createJSHTMLWrapper(globalObject, static_reference_cast<HTMLElement>(WTFMove(node)));
-            else if (is<SVGElement>(node.get()))
+            else if (is<SVGElement>(node))
                 wrapper = createJSSVGWrapper(globalObject, static_reference_cast<SVGElement>(WTFMove(node)));
             else

trunk/Source/WebCore/bindings/js/JSTextCustom.cpp

@@ -35 +35 @@
 JSValue toJSNewlyCreated(ExecState*, JSDOMGlobalObject* globalObject, Ref<Text>&& text)
 {
-    if (is<CDATASection>(text.get()))
+    if (is<CDATASection>(text))
         return createWrapper<CDATASection>(globalObject, WTFMove(text));
     return createWrapper<Text>(globalObject, WTFMove(text));

trunk/Source/WebCore/css/CSSFilterImageValue.cpp

@@ -96 +96 @@
 
     for (auto& filterOperation : m_filterOperations.operations()) {
-        if (!is<ReferenceFilterOperation>(filterOperation.get()))
+        if (!is<ReferenceFilterOperation>(filterOperation))
             continue;
         auto& referenceFilterOperation = downcast<ReferenceFilterOperation>(*filterOperation);

trunk/Source/WebCore/css/CSSFontFaceSet.cpp

@@ -339 +339 @@
 
     auto family = style->getPropertyCSSValue(CSSPropertyFontFamily);
-    if (!is<CSSValueList>(family.get()))
+    if (!is<CSSValueList>(family))
         return Exception { SyntaxError };
     CSSValueList& familyList = downcast<CSSValueList>(*family);

trunk/Source/WebCore/css/CSSFontSelector.cpp

@@ -159 +159 @@
     RefPtr<CSSValue> variantEastAsian = style.getPropertyCSSValue(CSSPropertyFontVariantEastAsian);
     RefPtr<CSSValue> loadingBehavior = style.getPropertyCSSValue(CSSPropertyFontDisplay);
-    if (!is<CSSValueList>(fontFamily.get()) || !is<CSSValueList>(src.get()) || (unicodeRange && !is<CSSValueList>(*unicodeRange)))
+    if (!is<CSSValueList>(fontFamily) || !is<CSSValueList>(src) || (unicodeRange && !is<CSSValueList>(*unicodeRange)))
         return;
 

trunk/Source/WebCore/css/CSSToStyleMap.cpp

@@ -550 +550 @@
 
     for (auto& current : borderImage) {
-        if (is<CSSImageValue>(current.get()) || is<CSSImageGeneratorValue>(current.get()) || is<CSSImageSetValue>(current.get()))
+        if (is<CSSImageValue>(current) || is<CSSImageGeneratorValue>(current) || is<CSSImageSetValue>(current))
             image.setImage(styleImage(current.get()));
-        else if (is<CSSBorderImageSliceValue>(current.get()))
+        else if (is<CSSBorderImageSliceValue>(current))
             mapNinePieceImageSlice(current, image);
-        else if (is<CSSValueList>(current.get())) {
+        else if (is<CSSValueList>(current)) {
             CSSValueList& slashList = downcast<CSSValueList>(current.get());
             // Map in the image slices.
@@ -567 +567 @@
             if (slashList.item(2))
                 image.setOutset(mapNinePieceImageQuad(*slashList.item(2)));
-        } else if (is<CSSPrimitiveValue>(current.get())) {
+        } else if (is<CSSPrimitiveValue>(current)) {
             // Set the appropriate rules for stretch/round/repeat of the slices.
             mapNinePieceImageRepeat(current, image);

trunk/Source/WebCore/css/FontFace.cpp

@@ -67 +67 @@
         [&] (String& string) -> ExceptionOr<void> {
             auto value = FontFace::parseString(string, CSSPropertySrc);
-            if (!is<CSSValueList>(value.get()))
+            if (!is<CSSValueList>(value))
                 return Exception { SyntaxError };
             CSSFontFace::appendSources(result->backing(), downcast<CSSValueList>(*value), &document, false);

trunk/Source/WebCore/css/StyleBuilderConverter.h

@@ -910 +910 @@
     unsigned currentNamedGridLine = 0;
     for (auto& currentValue : downcast<CSSValueList>(value)) {
-        if (is<CSSGridLineNamesValue>(currentValue.get())) {
+        if (is<CSSGridLineNamesValue>(currentValue)) {
             createGridLineNamesList(currentValue.get(), currentNamedGridLine, tracksData.m_namedGridLines, tracksData.m_orderedNamedGridLines);
             continue;
@@ -922 +922 @@
             tracksData.m_autoRepeatType = autoRepeatID == CSSValueAutoFill ? AutoFill : AutoFit;
             for (auto& autoRepeatValue : downcast<CSSValueList>(currentValue.get())) {
-                if (is<CSSGridLineNamesValue>(autoRepeatValue.get())) {
+                if (is<CSSGridLineNamesValue>(autoRepeatValue)) {
                     createGridLineNamesList(autoRepeatValue.get(), autoRepeatIndex, tracksData.m_autoRepeatNamedGridLines, tracksData.m_autoRepeatOrderedNamedGridLines);
                     continue;

trunk/Source/WebCore/css/StyleBuilderCustom.h

@@ -1230 +1230 @@
     auto& list = downcast<CSSValueList>(value);
     for (auto& item : list) {
-        if (is<CSSCursorImageValue>(item.get())) {
+        if (is<CSSCursorImageValue>(item)) {
             auto& image = downcast<CSSCursorImageValue>(item.get());
             styleResolver.style()->addCursor(styleResolver.styleImage(image), image.hotSpot());
@@ -1411 +1411 @@
     bool didSet = false;
     for (auto& item : downcast<CSSValueList>(value)) {
-        if (is<CSSImageGeneratorValue>(item.get())) {
-            if (is<CSSGradientValue>(item.get()))
+        if (is<CSSImageGeneratorValue>(item)) {
+            if (is<CSSGradientValue>(item))
                 styleResolver.style()->setContent(StyleGeneratedImage::create(downcast<CSSGradientValue>(item.get()).gradientWithStylesResolved(styleResolver)), didSet);
             else
                 styleResolver.style()->setContent(StyleGeneratedImage::create(downcast<CSSImageGeneratorValue>(item.get())), didSet);
             didSet = true;
-        } else if (is<CSSImageSetValue>(item.get())) {
+        } else if (is<CSSImageSetValue>(item)) {
             styleResolver.style()->setContent(StyleCachedImage::create(item), didSet);
             didSet = true;
         }
 
-        if (is<CSSImageValue>(item.get())) {
+        if (is<CSSImageValue>(item)) {
             styleResolver.style()->setContent(StyleCachedImage::create(item), didSet);
             didSet = true;
@@ -1428 +1428 @@
         }
 
-        if (!is<CSSPrimitiveValue>(item.get()))
+        if (!is<CSSPrimitiveValue>(item))
             continue;
 
@@ -1860 +1860 @@
     auto willChange = WillChangeData::create();
     for (auto& item : downcast<CSSValueList>(value)) {
-        if (!is<CSSPrimitiveValue>(item.get()))
+        if (!is<CSSPrimitiveValue>(item))
             continue;
         auto& primitiveValue = downcast<CSSPrimitiveValue>(item.get());

trunk/Source/WebCore/css/StyleProperties.cpp

@@ -242 +242 @@
 {
     auto colorValue = getPropertyCSSValue(property);
-    if (!is<CSSPrimitiveValue>(colorValue.get()))
+    if (!is<CSSPrimitiveValue>(colorValue))
         return std::nullopt;
 
@@ -252 +252 @@
 {
     auto cssValue = getPropertyCSSValue(property);
-    return is<CSSPrimitiveValue>(cssValue.get()) ? downcast<CSSPrimitiveValue>(*cssValue).valueID() : CSSValueInvalid;
+    return is<CSSPrimitiveValue>(cssValue) ? downcast<CSSPrimitiveValue>(*cssValue).valueID() : CSSValueInvalid;
 }
 

trunk/Source/WebCore/css/StyleResolver.cpp

@@ -1918 +1918 @@
     for (auto& currentValue : downcast<CSSValueList>(inValue)) {
 
-        if (is<CSSPrimitiveValue>(currentValue.get())) {
+        if (is<CSSPrimitiveValue>(currentValue)) {
             auto& primitiveValue = downcast<CSSPrimitiveValue>(currentValue.get());
             if (!primitiveValue.isURI())
@@ -1931 +1931 @@
         }
 
-        if (!is<CSSFunctionValue>(currentValue.get()))
+        if (!is<CSSFunctionValue>(currentValue))
             continue;
 

trunk/Source/WebCore/css/TransformFunctions.cpp

@@ -111 +111 @@
     TransformOperations operations;
     for (auto& currentValue : downcast<CSSValueList>(value)) {
-        if (!is<CSSFunctionValue>(currentValue.get()))
+        if (!is<CSSFunctionValue>(currentValue))
             continue;
 

trunk/Source/WebCore/css/ViewportStyleResolver.cpp

@@ -105 +105 @@
 
     RefPtr<CSSValue> value = m_propertySet->getPropertyCSSValue(id);
-    if (!is<CSSPrimitiveValue>(value.get()))
+    if (!is<CSSPrimitiveValue>(value))
         return defaultValue;
 

trunk/Source/WebCore/dom/Document.cpp

@@ -2646 +2646 @@
 ExceptionOr<void> Document::setBodyOrFrameset(RefPtr<HTMLElement>&& newBody)
 {
-    if (!is<HTMLBodyElement>(newBody.get()) && !is<HTMLFrameSetElement>(newBody.get()))
+    if (!is<HTMLBodyElement>(newBody) && !is<HTMLFrameSetElement>(newBody))
         return Exception { HierarchyRequestError };
 
@@ -3845 +3845 @@
         }
 
-        if (is<HTMLInputElement>(oldFocusedElement.get())) {
+        if (is<HTMLInputElement>(oldFocusedElement)) {
             // HTMLInputElement::didBlur just scrolls text fields back to the beginning.
             // FIXME: This could be done asynchronusly.
@@ -6848 +6848 @@
         }
         // Unset hovered nodes in sub frame documents if the old hovered node was a frame owner.
-        if (is<HTMLFrameOwnerElement>(oldHoveredElement.get())) {
+        if (is<HTMLFrameOwnerElement>(oldHoveredElement)) {
             if (Document* contentDocument = downcast<HTMLFrameOwnerElement>(*oldHoveredElement).contentDocument())
                 contentDocument->updateHoverActiveState(request, nullptr);

trunk/Source/WebCore/dom/Element.cpp

@@ -2570 +2570 @@
 
     RefPtr<Node> node = next ? next->previousSibling() : nullptr;
-    if (is<Text>(node.get())) {
+    if (is<Text>(node)) {
         auto result = mergeWithNextTextNode(downcast<Text>(*node));
         if (result.hasException())
             return result.releaseException();
     }
-    if (is<Text>(prev.get())) {
+    if (is<Text>(prev)) {
         auto result = mergeWithNextTextNode(downcast<Text>(*prev));
         if (result.hasException())

trunk/Source/WebCore/dom/Range.cpp

@@ -919 +919 @@
 
     unsigned newOffset = referenceNode ? referenceNode->computeNodeIndex() : parent->countChildNodes();
-    if (is<DocumentFragment>(node.get()))
+    if (is<DocumentFragment>(node))
         newOffset += downcast<DocumentFragment>(node.get()).countChildNodes();
     else

trunk/Source/WebCore/editing/ApplyStyleCommand.cpp

@@ -60 +60 @@
 static int toIdentifier(RefPtr<CSSValue>&& value)
 {
-    return is<CSSPrimitiveValue>(value.get()) ? downcast<CSSPrimitiveValue>(*value).valueID() : 0;
+    return is<CSSPrimitiveValue>(value) ? downcast<CSSPrimitiveValue>(*value).valueID() : 0;
 }
 
@@ -271 +271 @@
             }
             ASSERT(!block || is<HTMLElement>(*block));
-            if (is<HTMLElement>(block.get())) {
+            if (is<HTMLElement>(block)) {
                 removeCSSStyle(style, downcast<HTMLElement>(*block));
                 if (!m_removeOnly)
@@ -1357 +1357 @@
         mergeIdenticalElements(element, downcast<Element>(*nextSibling));
 
-    if (is<Element>(previousSibling.get()) && previousSibling->hasEditableStyle()) {
+    if (is<Element>(previousSibling) && previousSibling->hasEditableStyle()) {
         auto* mergedElement = previousSibling->nextSibling();
         ASSERT(mergedElement);

trunk/Source/WebCore/editing/EditingStyle.cpp

@@ -144 +144 @@
 {
     RefPtr<CSSValue> value = extractPropertyValue(style, propertyID);
-    if (propertyID == CSSPropertyFontStyle && is<CSSFontStyleValue>(value.get()) && downcast<CSSFontStyleValue>(value.get())->isItalicOrOblique())
+    if (propertyID == CSSPropertyFontStyle && is<CSSFontStyleValue>(value) && downcast<CSSFontStyleValue>(value.get())->isItalicOrOblique())
         return CSSValueItalic;
-    if (!is<CSSPrimitiveValue>(value.get()))
+    if (!is<CSSPrimitiveValue>(value))
         return 0;
     return downcast<CSSPrimitiveValue>(*value).valueID();
@@ -199 +199 @@
 {
     RefPtr<CSSValue> value = style.m_mutableStyle->getPropertyCSSValue(m_propertyID);
-    return matches(element) && is<CSSPrimitiveValue>(value.get()) && downcast<CSSPrimitiveValue>(*value).valueID() == m_primitiveValue->valueID();
+    return matches(element) && is<CSSPrimitiveValue>(value) && downcast<CSSPrimitiveValue>(*value).valueID() == m_primitiveValue->valueID();
 }
 
@@ -238 +238 @@
         if (!styleValue)
             styleValue = style.m_mutableStyle->getPropertyCSSValue(CSSPropertyTextDecoration);
-        return is<CSSValueList>(styleValue.get()) && downcast<CSSValueList>(*styleValue).hasValue(m_primitiveValue.get());
+        return is<CSSValueList>(styleValue) && downcast<CSSValueList>(*styleValue).hasValue(m_primitiveValue.get());
     }
 
@@ -503 +503 @@
     // Get the adjustment amount out of the style.
     RefPtr<CSSValue> value = m_mutableStyle->getPropertyCSSValue(CSSPropertyWebkitFontSizeDelta);
-    if (!is<CSSPrimitiveValue>(value.get()))
+    if (!is<CSSPrimitiveValue>(value))
         return;
 
@@ -551 +551 @@
 
     RefPtr<CSSValue> unicodeBidi = m_mutableStyle->getPropertyCSSValue(CSSPropertyUnicodeBidi);
-    if (!is<CSSPrimitiveValue>(unicodeBidi.get()))
+    if (!is<CSSPrimitiveValue>(unicodeBidi))
         return false;
 
@@ -557 +557 @@
     if (unicodeBidiValue == CSSValueEmbed) {
         RefPtr<CSSValue> direction = m_mutableStyle->getPropertyCSSValue(CSSPropertyDirection);
-        if (!is<CSSPrimitiveValue>(direction.get()))
+        if (!is<CSSPrimitiveValue>(direction))
             return false;
 
@@ -800 +800 @@
 {
     RefPtr<CSSValue> value = properties.getPropertyCSSValue(CSSPropertyTextDecoration);
-    if (!is<CSSValueList>(value.get()))
+    if (!is<CSSValueList>(value))
         return nullptr;
     return downcast<CSSValueList>(value.get());
@@ -1090 +1090 @@
         m_mutableStyle->removeProperty(CSSPropertyBackgroundColor);
 
-    if (is<CSSPrimitiveValue>(unicodeBidi.get())) {
+    if (is<CSSPrimitiveValue>(unicodeBidi)) {
         m_mutableStyle->setProperty(CSSPropertyUnicodeBidi, static_cast<CSSValueID>(downcast<CSSPrimitiveValue>(*unicodeBidi).valueID()));
-        if (is<CSSPrimitiveValue>(direction.get()))
+        if (is<CSSPrimitiveValue>(direction))
             m_mutableStyle->setProperty(CSSPropertyDirection, static_cast<CSSValueID>(downcast<CSSPrimitiveValue>(*direction).valueID()));
     }
@@ -1407 +1407 @@
 {
     RefPtr<CSSValue> cssValue = m_mutableStyle->getPropertyCSSValue(CSSPropertyFontSize);
-    if (!is<CSSPrimitiveValue>(cssValue.get()))
+    if (!is<CSSPrimitiveValue>(cssValue))
         return 0;
     return legacyFontSizeFromCSSValue(document, downcast<CSSPrimitiveValue>(cssValue.get()),
@@ -1476 +1476 @@
 
             RefPtr<CSSValue> unicodeBidi = ComputedStyleExtractor(n).propertyValue(CSSPropertyUnicodeBidi);
-            if (!is<CSSPrimitiveValue>(unicodeBidi.get()))
+            if (!is<CSSPrimitiveValue>(unicodeBidi))
                 continue;
 
@@ -1505 +1505 @@
         ComputedStyleExtractor computedStyle(node);
         RefPtr<CSSValue> unicodeBidi = computedStyle.propertyValue(CSSPropertyUnicodeBidi);
-        if (!is<CSSPrimitiveValue>(unicodeBidi.get()))
+        if (!is<CSSPrimitiveValue>(unicodeBidi))
             continue;
 
@@ -1517 +1517 @@
         ASSERT(unicodeBidiValue == CSSValueEmbed);
         RefPtr<CSSValue> direction = computedStyle.propertyValue(CSSPropertyDirection);
-        if (!is<CSSPrimitiveValue>(direction.get()))
+        if (!is<CSSPrimitiveValue>(direction))
             continue;
 
@@ -1585 +1585 @@
     if (shouldAddUnderline || shouldAddStrikeThrough) {
         RefPtr<CSSValue> value = computedStyle.propertyValue(CSSPropertyWebkitTextDecorationsInEffect);
-        if (!is<CSSValueList>(value.get()))
+        if (!is<CSSValueList>(value))
             value = computedStyle.propertyValue(CSSPropertyTextDecoration);
 
         RefPtr<CSSValueList> valueList;
-        if (is<CSSValueList>(value.get()))
+        if (is<CSSValueList>(value))
             valueList = downcast<CSSValueList>(value.get());
 
@@ -1668 +1668 @@
     // Furthermore, text-decoration: none has been trimmed so that text-decoration property is always a CSSValueList.
     RefPtr<CSSValue> textDecoration = style.getPropertyCSSValue(CSSPropertyTextDecoration);
-    if (is<CSSValueList>(textDecoration.get())) {
+    if (is<CSSValueList>(textDecoration)) {
         auto& cssValuePool = CSSValuePool::singleton();
         RefPtr<CSSPrimitiveValue> underline = cssValuePool.createIdentifierValue(CSSValueUnderline);
@@ -1719 +1719 @@
 {
     RefPtr<CSSValue> textDecoration = style.getPropertyCSSValue(propertID);
-    if (!is<CSSValueList>(textDecoration.get()) || !is<CSSValueList>(refTextDecoration))
+    if (!is<CSSValueList>(textDecoration) || !is<CSSValueList>(refTextDecoration))
         return;
 

trunk/Source/WebCore/html/BaseChooserOnlyDateAndTimeInputType.cpp

@@ -71 +71 @@
 {
     RefPtr<Node> node = element().userAgentShadowRoot()->firstChild();
-    if (!is<HTMLElement>(node.get()))
+    if (!is<HTMLElement>(node))
         return;
     String displayValue = visibleValue();

trunk/Source/WebCore/html/FormAssociatedElement.cpp

@@ -109 +109 @@
         // treeScope().getElementById() over the given element.
         RefPtr<Element> newFormCandidate = element->treeScope().getElementById(formId);
-        if (is<HTMLFormElement>(newFormCandidate.get()))
+        if (is<HTMLFormElement>(newFormCandidate))
             return downcast<HTMLFormElement>(newFormCandidate.get());
         return nullptr;

trunk/Source/WebCore/html/HTMLAreaElement.cpp

@@ -189 +189 @@
 {
     RefPtr<Node> mapElement = parentNode();
-    if (!is<HTMLMapElement>(mapElement.get()))
+    if (!is<HTMLMapElement>(mapElement))
         return nullptr;
     

trunk/Source/WebCore/html/HTMLElement.cpp

@@ -562 +562 @@
 
     RefPtr<Node> node = next ? next->previousSibling() : nullptr;
-    if (is<Text>(node.get())) {
+    if (is<Text>(node)) {
         auto result = mergeWithNextTextNode(downcast<Text>(*node));
         if (result.hasException())
             return result.releaseException();
     }
-    if (is<Text>(prev.get())) {
+    if (is<Text>(prev)) {
         auto result = mergeWithNextTextNode(downcast<Text>(*prev));
         if (result.hasException())
@@ -841 +841 @@
     RefPtr<Element> parent = parentElement();
 
-    if (is<HTMLElement>(parent.get()) && parent->selfOrAncestorHasDirAutoAttribute())
+    if (is<HTMLElement>(parent) && parent->selfOrAncestorHasDirAutoAttribute())
         downcast<HTMLElement>(*parent).adjustDirectionalityIfNeededAfterChildAttributeChanged(this);
 

trunk/Source/WebCore/html/HTMLEmbedElement.cpp

@@ -201 +201 @@
     // should be ignored and not get a renderer.
     RefPtr<ContainerNode> parent = parentNode();
-    if (is<HTMLObjectElement>(parent.get())) {
+    if (is<HTMLObjectElement>(parent)) {
         if (!parent->renderer())
             return false;

trunk/Source/WebCore/html/HTMLFormControlElement.cpp

@@ -125 +125 @@
         if (is<HTMLFieldSetElement>(*ancestor) && ancestor->hasAttributeWithoutSynchronization(disabledAttr)) {
             HTMLFieldSetElement& fieldSetAncestor = downcast<HTMLFieldSetElement>(*ancestor);
-            bool isInFirstLegend = is<HTMLLegendElement>(previousAncestor.get()) && previousAncestor == fieldSetAncestor.legend();
+            bool isInFirstLegend = is<HTMLLegendElement>(previousAncestor) && previousAncestor == fieldSetAncestor.legend();
             return !isInFirstLegend;
         }

trunk/Source/WebCore/html/HTMLInputElement.cpp

@@ -1598 +1598 @@
 
     RefPtr<Element> element = treeScope().getElementById(attributeWithoutSynchronization(listAttr));
-    if (!is<HTMLDataListElement>(element.get())
+    if (!is<HTMLDataListElement>(element))
         return nullptr;
 

trunk/Source/WebCore/html/HTMLOptionElement.cpp

@@ -121 +121 @@
     // Handle the common special case where there's exactly 1 child node, and it's a text node.
     RefPtr<Node> child = firstChild();
-    if (is<Text>(child.get()) && !child->nextSibling())
+    if (is<Text>(child) && !child->nextSibling())
         downcast<Text>(*child).setData(text);
     else {
@@ -302 +302 @@
 {
     RefPtr<ContainerNode> parent = parentNode();
-    if (is<HTMLOptGroupElement>(parent.get()))
+    if (is<HTMLOptGroupElement>(parent))
         return "    " + displayLabel();
     return displayLabel();

trunk/Source/WebCore/html/HTMLPlugInElement.cpp

@@ -228 +228 @@
 
     RefPtr<Widget> widget = pluginWidget();
-    if (!is<PluginViewBase>(widget.get()))
+    if (!is<PluginViewBase>(widget))
         return false;
 
@@ -243 +243 @@
     // No widget - can't be anything to see or hear here.
     RefPtr<Widget> widget = pluginWidget(PluginLoadingPolicy::DoNotLoad);
-    if (!is<PluginViewBase>(widget.get()))
+    if (!is<PluginViewBase>(widget))
         return false;
 

trunk/Source/WebCore/html/HTMLSelectElement.cpp

@@ -866 +866 @@
         deselectItemsWithoutValidation(element.get());
 
-    if (is<HTMLOptionElement>(element.get())) {
+    if (is<HTMLOptionElement>(element)) {
         if (m_activeSelectionAnchorIndex < 0 || shouldDeselect)
             setActiveSelectionAnchorIndex(listIndex);

trunk/Source/WebCore/html/HTMLSummaryElement.cpp

@@ -101 +101 @@
         return true;
     RefPtr<Element> host = element.shadowHost();
-    return host && is<HTMLFormControlElement>(host.get());
+    return host && is<HTMLFormControlElement>(host);
 }
 

trunk/Source/WebCore/html/HTMLTextFormControlElement.cpp

@@ -799 +799 @@
         if (const StyleProperties* properties = innerText->presentationAttributeStyle()) {
             RefPtr<CSSValue> value = properties->getPropertyCSSValue(CSSPropertyWebkitUserModify);
-            if (is<CSSPrimitiveValue>(value.get()))
+            if (is<CSSPrimitiveValue>(value))
                 textBlockStyle.setUserModify(downcast<CSSPrimitiveValue>(*value));
         }

trunk/Source/WebCore/html/RadioInputType.cpp

@@ -122 +122 @@
     // skip any other elements in the group.
     RefPtr<Element> currentFocusedNode = element().document().focusedElement();
-    if (is<HTMLInputElement>(currentFocusedNode.get())) {
+    if (is<HTMLInputElement>(currentFocusedNode)) {
         HTMLInputElement& focusedInput = downcast<HTMLInputElement>(*currentFocusedNode);
         if (focusedInput.isRadioButton() && focusedInput.form() == element().form() && focusedInput.name() == element().name())

trunk/Source/WebCore/html/parser/HTMLConstructionSite.cpp

@@ -191 +191 @@
 void HTMLConstructionSite::attachLater(ContainerNode& parent, Ref<Node>&& child, bool selfClosing)
 {
-    ASSERT(scriptingContentIsAllowed(m_parserContentPolicy) || !is<Element>(child.get()) || !isScriptElement(downcast<Element>(child.get())));
+    ASSERT(scriptingContentIsAllowed(m_parserContentPolicy) || !is<Element>(child) || !isScriptElement(downcast<Element>(child.get())));
     ASSERT(pluginContentIsAllowed(m_parserContentPolicy) || !child->isPluginElement());
 
@@ -585 +585 @@
 
     RefPtr<Node> previousChild = task.nextChild ? task.nextChild->previousSibling() : task.parent->lastChild();
-    if (is<Text>(previousChild.get())) {
+    if (is<Text>(previousChild)) {
         // FIXME: We're only supposed to append to this text node if it
         // was the last text node inserted by the parser.

trunk/Source/WebCore/html/shadow/MediaControlElements.cpp

@@ -690 +690 @@
     if (event.type() == eventNames().clickEvent) {
         RefPtr<Node> target = event.target()->toNode();
-        if (!is<Element>(target.get()))
+        if (!is<Element>(target))
             return;
 

trunk/Source/WebCore/html/track/TextTrackList.cpp

@@ -188 +188 @@
     if (track->trackType() == TextTrack::AddTrack)
         m_addTrackTracks.append(track.ptr());
-    else if (is<LoadableTextTrack>(track.get())) {
+    else if (is<LoadableTextTrack>(track)) {
         // Insert tracks added for <track> element in tree order.
         size_t index = downcast<LoadableTextTrack>(track.get()).trackElementIndex();

trunk/Source/WebCore/inspector/InspectorDOMAgent.cpp

@@ -737 +737 @@
 
     RefPtr<Node> oldNode = nodeForId(nodeId);
-    if (!is<Element>(oldNode.get()))
+    if (!is<Element>(oldNode))
         return;
 

trunk/Source/WebCore/inspector/PageScriptDebugServer.cpp

@@ -172 +172 @@
     if (auto* view = frame.view()) {
         for (auto& child : view->children()) {
-            if (!is<PluginViewBase>(child.get()))
+            if (!is<PluginViewBase>(child))
                 continue;
             downcast<PluginViewBase>(child.get()).setJavaScriptPaused(paused);

trunk/Source/WebCore/page/FocusController.cpp

@@ -775 +775 @@
     // FIXME: These additional checks should not be necessary. We should consider generally keeping the selection whenever the
     // focused element is blurred, with no new element taking focus.
-    if (!oldFocusedElement->isRootEditableElement() && !is<HTMLInputElement>(oldFocusedElement.get()) && !is<HTMLTextAreaElement>(oldFocusedElement.get()))
+    if (!oldFocusedElement->isRootEditableElement() && !is<HTMLInputElement>(oldFocusedElement) && !is<HTMLTextAreaElement>(oldFocusedElement))
         return true;
 

trunk/Source/WebCore/page/FrameView.cpp

@@ -3769 +3769 @@
 {
     for (auto& widget : children()) {
-        if (is<FrameView>(widget.get())) {
+        if (is<FrameView>(widget)) {
             if (downcast<FrameView>(widget.get()).hasCustomScrollbars())
                 return true;
-        } else if (is<Scrollbar>(widget.get())) {
+        } else if (is<Scrollbar>(widget)) {
             if (downcast<Scrollbar>(widget.get()).isCustomScrollbar())
                 return true;

trunk/Source/WebCore/page/Page.cpp

@@ -1405 +1405 @@
             break;
         for (auto& widget : view->children()) {
-            if (is<PluginViewBase>(widget.get()))
+            if (is<PluginViewBase>(widget))
                 views.append(downcast<PluginViewBase>(widget.get()));
         }

trunk/Source/WebCore/page/mac/EventHandlerMac.mm

@@ -934 +934 @@
         } else {
             scrollableContainer = findEnclosingScrollableContainer(wheelEventTarget.get(), wheelEvent.deltaX(), wheelEvent.deltaY());
-            if (scrollableContainer && !is<HTMLIFrameElement>(wheelEventTarget.get()))
+            if (scrollableContainer && !is<HTMLIFrameElement>(wheelEventTarget))
                 scrollableArea = scrollableAreaForContainerNode(*scrollableContainer);
             else {

trunk/Source/WebCore/rendering/RenderBoxModelObject.cpp

@@ -912 +912 @@
             context.setDrawLuminanceMask(bgLayer.maskSourceType() == MaskLuminance);
 
-            if (is<BitmapImage>(image.get()))
+            if (is<BitmapImage>(image))
                 downcast<BitmapImage>(*image).updateFromSettings(settings());
 

trunk/Source/WebCore/testing/Internals.cpp

@@ -1034 +1034 @@
     unsigned count = 0;
     for (auto rule : childRules) {
-        if (is<StyleRule>(rule.get())) {
+        if (is<StyleRule>(rule)) {
             auto* cssRule = downcast<StyleRule>(rule.get());
             if (!cssRule->propertiesWithoutDeferredParsing())
@@ -1042 +1042 @@
 
         StyleRuleGroup* groupRule = nullptr;
-        if (is<StyleRuleMedia>(rule.get()))
+        if (is<StyleRuleMedia>(rule))
             groupRule = downcast<StyleRuleMedia>(rule.get());
-        else if (is<StyleRuleSupports>(rule.get()))
+        else if (is<StyleRuleSupports>(rule))
             groupRule = downcast<StyleRuleSupports>(rule.get());
         if (!groupRule)
@@ -1069 +1069 @@
     for (auto rule : childRules) {
         StyleRuleGroup* groupRule = nullptr;
-        if (is<StyleRuleMedia>(rule.get()))
+        if (is<StyleRuleMedia>(rule))
             groupRule = downcast<StyleRuleMedia>(rule.get());
-        else if (is<StyleRuleSupports>(rule.get()))
+        else if (is<StyleRuleSupports>(rule))
             groupRule = downcast<StyleRuleSupports>(rule.get());
         if (!groupRule)
@@ -1094 +1094 @@
     unsigned count = 0;
     for (auto rule : childRules) {
-        if (is<StyleRuleKeyframes>(rule.get())) {
+        if (is<StyleRuleKeyframes>(rule)) {
             auto* cssRule = downcast<StyleRuleKeyframes>(rule.get());
             if (!cssRule->keyframesWithoutDeferredParsing())
@@ -1102 +1102 @@
 
         StyleRuleGroup* groupRule = nullptr;
-        if (is<StyleRuleMedia>(rule.get()))
+        if (is<StyleRuleMedia>(rule))
             groupRule = downcast<StyleRuleMedia>(rule.get());
-        else if (is<StyleRuleSupports>(rule.get()))
+        else if (is<StyleRuleSupports>(rule))
             groupRule = downcast<StyleRuleSupports>(rule.get());
         if (!groupRule)