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

Unified Diff: third_party/WebKit/LayoutTests/fast/css/fontface-arraybuffer.html

Issue 2610593002: Make CSSFontFace::setLoadStatus post a task (Closed)
Patch Set: Change loading logic for fonts from array buffers Created 3 years, 11 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/fast/css/fontface-arraybuffer.html
diff --git a/third_party/WebKit/LayoutTests/fast/css/fontface-arraybuffer.html b/third_party/WebKit/LayoutTests/fast/css/fontface-arraybuffer.html
index 1bbcd1ca8eaa93157ef681dcd7cc6e2b8e8b0d34..aa060f51409793771a657df156bbdb2c7d6bf02b 100644
--- a/third_party/WebKit/LayoutTests/fast/css/fontface-arraybuffer.html
+++ b/third_party/WebKit/LayoutTests/fast/css/fontface-arraybuffer.html
@@ -16,11 +16,11 @@ for (var i = 0; i < ahemString.length; i++) {
}
var face1 = new FontFace('FontFromArrayBuffer', buffer);
-shouldBeEqualToString('face1.status', 'loaded');
+shouldBeEqualToString('face1.status', 'loading');
document.fonts.add(face1);
var face2 = new FontFace('FontFromArrayBufferView', uint8View);
-shouldBeEqualToString('face2.status', 'loaded');
+shouldBeEqualToString('face2.status', 'loading');
document.fonts.add(face2);
var face3 = new FontFace('FontFromEmptyArrayBuffer', new ArrayBuffer(0));
@@ -32,6 +32,8 @@ face3.load().catch(function(v) {
});
function verify() {
+ shouldBeEqualToString('face1.status', 'loaded');
+ shouldBeEqualToString('face2.status', 'loaded');
shouldBe("document.getElementById('FontFromArrayBuffer').offsetWidth", "document.getElementById('ref').offsetWidth");
shouldBe("document.getElementById('FontFromArrayBufferView').offsetWidth", "document.getElementById('ref').offsetWidth");
finishJSTest();

Powered by Google App Engine
This is Rietveld 408576698