Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(49)

Unified Diff: third_party/WebKit/LayoutTests/inspector-protocol/layout-fonts/resources/layout-font-test.js

Issue 2916283002: Revert of DevTools: require enabling CSS domain before running CSS.* commands. (Closed)
Patch Set: Created 3 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: third_party/WebKit/LayoutTests/inspector-protocol/layout-fonts/resources/layout-font-test.js
diff --git a/third_party/WebKit/LayoutTests/inspector-protocol/layout-fonts/resources/layout-font-test.js b/third_party/WebKit/LayoutTests/inspector-protocol/layout-fonts/resources/layout-font-test.js
index 996ca6b445380c5afe4ec10923c24807d16d32cb..f3efc867dbe447c503b6f2714929208aa9567414 100644
--- a/third_party/WebKit/LayoutTests/inspector-protocol/layout-fonts/resources/layout-font-test.js
+++ b/third_party/WebKit/LayoutTests/inspector-protocol/layout-fonts/resources/layout-font-test.js
@@ -57,13 +57,20 @@
}
- async function platformFontsForElementWithSelector(selector)
+ function platformFontsForElementWithSelector(selector)
{
- var nodeId = await InspectorTest.requestNodeId(documentNodeId, selector);
- await InspectorTest.sendCommandOrDie("CSS.enable", {});
- var response = await InspectorTest.sendCommandOrDie("CSS.getPlatformFontsForNode", { nodeId: nodeId });
- collectResults(response);
- testNextPageElement();
+ InspectorTest.requestNodeId(documentNodeId, selector, onNodeId);
+
+ function onNodeId(nodeId)
+ {
+ InspectorTest.sendCommandOrDie("CSS.getPlatformFontsForNode", { nodeId: nodeId }, onGotComputedFonts);
+ }
+
+ function onGotComputedFonts(response)
+ {
+ collectResults(response);
+ testNextPageElement();
+ }
}
function collectResults(response)

Powered by Google App Engine
This is Rietveld 408576698