Load fonts before running screenshotter (#1891)

This commit is contained in:
ylemkimon
2019-04-07 18:15:05 +09:00
committed by GitHub
parent 7d7c55c4c7
commit b9deec098f
2 changed files with 40 additions and 12 deletions

View File

@@ -384,18 +384,17 @@ function takeScreenshot(key) {
function loadMath() {
if (!opts.reload && driverReady) {
driver.executeAsyncScript(
"var callback = arguments[arguments.length - 1]; " +
driver.executeScript(
"handle_search_string(" +
JSON.stringify("?" + itm.query) + ", callback);")
JSON.stringify("?" + itm.query) + ");")
.then(waitThenScreenshot);
} else if (opts.coverage) {
// collect coverage before reloading
collectCoverage().then(function() {
return driver.get(url).then(waitThenScreenshot);
return driver.get(url).then(loadFonts);
});
} else {
driver.get(url).then(waitThenScreenshot);
driver.get(url).then(loadFonts);
}
}
@@ -408,6 +407,13 @@ function takeScreenshot(key) {
});
}
function loadFonts() {
driver.executeAsyncScript(
"var callback = arguments[arguments.length - 1]; " +
"load_fonts(callback);")
.then(waitThenScreenshot);
}
function waitThenScreenshot() {
driverReady = true;
if (opts.wait) {