[webkit-changes] [WebKit/WebKit] a465a8: Web Inspector: Regression(260175 at main) Grid overla...

Patrick Angle noreply at github.com
Wed Apr 12 08:28:58 PDT 2023


  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: a465a8a38ab52cfd57d2c2e710d72ea6f53d9c81
      https://github.com/WebKit/WebKit/commit/a465a8a38ab52cfd57d2c2e710d72ea6f53d9c81
  Author: Patrick Angle <pangle at apple.com>
  Date:   2023-04-12 (Wed, 12 Apr 2023)

  Changed paths:
    M LayoutTests/inspector/dom/showGridOverlay.html
    M Source/WebCore/inspector/InspectorOverlay.cpp

  Log Message:
  -----------
  Web Inspector: Regression(260175 at main) Grid overlay for grids using repeat syntax triggers assert performing downcast
https://bugs.webkit.org/show_bug.cgi?id=255311
rdar://107764654

Reviewed by Devin Rousso.

After 260175 at main, CSSGridIntegerRepeatValue and CSSGridAutoRepeatValue no longer inherit from CSSValueList, and instead
inherit from CSSValueContainingVector. The inspector overlay code was written in such a way that we were checking if the
class was one of the repeat types, but then downcasting to CSSValueList making that relation less apparent. Converting
the downcasts to dynamicDowncasts where we previously had the `is` checks fixes the issue and guards against this
mistake in the future.

* LayoutTests/inspector/dom/showGridOverlay.html:
* Source/WebCore/inspector/InspectorOverlay.cpp:
(WebCore::authoredGridTrackSizes):

Canonical link: https://commits.webkit.org/262869@main




More information about the webkit-changes mailing list