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

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

Issue 2914663002: DevTools: require enabling CSS domain before running CSS.* commands. (Closed)
Patch Set: same Created 3 years, 6 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 unified diff | Download patch
OLDNEW
1 function test() 1 function test()
2 { 2 {
3 var documentNodeId; 3 var documentNodeId;
4 var documentNodeSelector; 4 var documentNodeSelector;
5 var testSelectors = []; 5 var testSelectors = [];
6 var collectedFontUsage = {}; 6 var collectedFontUsage = {};
7 7
8 InspectorTest.requestDocumentNodeId(onDocumentNodeId); 8 InspectorTest.requestDocumentNodeId(onDocumentNodeId);
9 9
10 function onDocumentNodeId(nodeId) 10 function onDocumentNodeId(nodeId)
(...skipping 39 matching lines...) Expand 10 before | Expand all | Expand 10 after
50 // the content_shell text dump is easier to validate. 50 // the content_shell text dump is easier to validate.
51 InspectorTest.evaluateInInspectedPage("injectCollectedResultsInPage(" + 51 InspectorTest.evaluateInInspectedPage("injectCollectedResultsInPage(" +
52 JSON.stringify(collectedFontUsage) + 52 JSON.stringify(collectedFontUsage) +
53 ")"); 53 ")");
54 InspectorTest.evaluateInInspectedPage("postTestHookWithFontResults(" + 54 InspectorTest.evaluateInInspectedPage("postTestHookWithFontResults(" +
55 JSON.stringify(collectedFontUsage) + 55 JSON.stringify(collectedFontUsage) +
56 ")"); 56 ")");
57 57
58 } 58 }
59 59
60 function platformFontsForElementWithSelector(selector) 60 async function platformFontsForElementWithSelector(selector)
61 { 61 {
62 InspectorTest.requestNodeId(documentNodeId, selector, onNodeId); 62 var nodeId = await InspectorTest.requestNodeId(documentNodeId, selector) ;
63 63 await InspectorTest.sendCommandOrDie("CSS.enable", {});
64 function onNodeId(nodeId) 64 var response = await InspectorTest.sendCommandOrDie("CSS.getPlatformFont sForNode", { nodeId: nodeId });
65 { 65 collectResults(response);
66 InspectorTest.sendCommandOrDie("CSS.getPlatformFontsForNode", { node Id: nodeId }, onGotComputedFonts); 66 testNextPageElement();
67 }
68
69 function onGotComputedFonts(response)
70 {
71 collectResults(response);
72 testNextPageElement();
73 }
74 } 67 }
75 68
76 function collectResults(response) 69 function collectResults(response)
77 { 70 {
78 var usedFonts = response.fonts; 71 var usedFonts = response.fonts;
79 usedFonts.sort(function(a, b) { 72 usedFonts.sort(function(a, b) {
80 return b.glyphCount - a.glyphCount; 73 return b.glyphCount - a.glyphCount;
81 }); 74 });
82 collectedFontUsage[documentNodeSelector] = usedFonts; 75 collectedFontUsage[documentNodeSelector] = usedFonts;
83 } 76 }
(...skipping 25 matching lines...) Expand all
109 var resultTextNode = document.createTextNode(resultText); 102 var resultTextNode = document.createTextNode(resultText);
110 resultTextPre.appendChild(resultTextNode); 103 resultTextPre.appendChild(resultTextNode);
111 resultElement.appendChild(resultTextPre); 104 resultElement.appendChild(resultTextPre);
112 testNode.parentNode.insertBefore(resultElement, testNode.nextSibling); 105 testNode.parentNode.insertBefore(resultElement, testNode.nextSibling);
113 } 106 }
114 } 107 }
115 108
116 window.addEventListener("DOMContentLoaded", function () { 109 window.addEventListener("DOMContentLoaded", function () {
117 runTest(); 110 runTest();
118 }, false); 111 }, false);
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698