Bug 211405 - REGRESSION (r261109): [ Mac ] inspector/css/getAllStyleSheets.html is failing
Summary: REGRESSION (r261109): [ Mac ] inspector/css/getAllStyleSheets.html is failing
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Web Inspector (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Nobody
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2020-05-04 15:07 PDT by Truitt Savell
Modified: 2020-05-04 16:23 PDT (History)
4 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Truitt Savell 2020-05-04 15:07:20 PDT
inspector/css/getAllStyleSheets.html

This test it failing sense the changes in r261109

History:
https://results.webkit.org/?suite=layout-tests&test=inspector%2Fcss%2FgetAllStyleSheets.html

Diff:
--- /Volumes/Data/slave/catalina-release-tests-wk1/build/layout-test-results/inspector/css/getAllStyleSheets-expected.txt
+++ /Volumes/Data/slave/catalina-release-tests-wk1/build/layout-test-results/inspector/css/getAllStyleSheets-actual.txt
@@ -1,20 +1,35 @@
 Test CSS.getAllStyleSheets returns expected information for different style sheet sources.
 
-PASS: There should be four stylesheets.
+Uncaught exception in Inspector page: WI.cssManager.fetchStyleSheetsIfNeeded is not a function. (In 'WI.cssManager.fetchStyleSheetsIfNeeded()', 'WI.cssManager.fetchStyleSheetsIfNeeded' is undefined)
 
-URL: inspector/css/resources/external.css
-OFFSET: (0, 0)
-PASS: Stylesheet should have a frame.
+Stack Trace:
+#0: (anonymous) (test)
+#1: (anonymous) (runTestMethodInFrontend)
+#2: (anonymous) (eval code)
+#3: eval [native code]
+#4: (anonymous) (TestCombined.js:9590:24)
+#5: _flushPendingScripts (TestCombined.js:7778:24)
+#6: _dispatchResponse (TestCombined.js:7628:38)
+#7: dispatch (TestCombined.js:7567:35)
+#8: dispatch (TestCombined.js:7160:52)
+#9: (anonymous) (TestCombined.js:8098:34)
 
-URL: <style> on inspector/css/getAllStyleSheets.html
-OFFSET: (5, 7)
-PASS: Stylesheet should have a frame.
+Evaluated Code:
+function test()
+{
+    WI.cssManager.fetchStyleSheetsIfNeeded();
+    InspectorBackend.runAfterPendingDispatches(function() {
+        let styleSheets = WI.cssManager.styleSheets;
+        InspectorTest.expectThat(styleSheets.length === 4, "There should be four stylesheets.");
 
-URL: <style> on inspector/css/getAllStyleSheets.html
-OFFSET: (6, 7)
-PASS: Stylesheet should have a frame.
+        for (let styleSheet of styleSheets) {
+            InspectorTest.log("");
+            InspectorTest.log(`URL: ${styleSheet.isInlineStyleTag() ? "<style> on " : ""}${sanitizeURL(styleSheet.url)}`);
+            InspectorTest.log(`OFFSET: (${styleSheet.startLineNumber}, ${styleSheet.startColumnNumber})`);
+            InspectorTest.expectThat(styleSheet.parentFrame, "Stylesheet should have a frame.");
+        }
 
-URL: inspector/css/getAllStyleSheets.html
-OFFSET: (0, 0)
-PASS: Stylesheet should have a frame.
+        InspectorTest.completeTest();
+    });
+}
Comment 1 Radar WebKit Bug Importer 2020-05-04 15:07:40 PDT
<rdar://problem/62863843>
Comment 2 Devin Rousso 2020-05-04 16:23:08 PDT
Committed r261122: <https://trac.webkit.org/changeset/261122>