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

Unified Diff: tests/html/fontface_loaded_test.dart

Issue 1348173002: Dartium JS Enabled take 2 (Closed) Base URL: https://github.com/dart-lang/sdk.git@master
Patch Set: Created 5 years, 3 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
« no previous file with comments | « tests/co19/co19-dartium.status ('k') | tests/html/html.status » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: tests/html/fontface_loaded_test.dart
diff --git a/tests/html/fontface_loaded_test.dart b/tests/html/fontface_loaded_test.dart
index d1926880ae89c66f7f052293d236fb2906193cb7..8a80f5d4d500fc8cd7938e107f2b6c2a6f8358ba 100644
--- a/tests/html/fontface_loaded_test.dart
+++ b/tests/html/fontface_loaded_test.dart
@@ -8,17 +8,18 @@ import 'dart:isolate';
import 'dart:html';
class NullTreeSanitizer implements NodeTreeSanitizer {
- void sanitizeTree(Node node) {}
+ void sanitizeTree(Node node) {}
}
main() {
useHtmlConfiguration();
- var style = new Element.html('''
+ var style = new Element.html(
+ '''
<style>
@font-face {
font-family: 'Ahem';
- src: url(../../resources/Ahem.ttf);
+ src: url(/root_dart/tests/html/Ahem.ttf);
font-style: italic;
font-weight: 300;
unicode-range: U+0-3FF;
@@ -27,20 +28,26 @@ main() {
/* font-stretch property is not supported */
}
</style>
- ''', treeSanitizer: new NullTreeSanitizer());
+ ''',
+ treeSanitizer: new NullTreeSanitizer());
document.head.append(style);
-
test('document fonts - temporary', () {
var atLeastOneFont = false;
+ var loaded = [];
document.fonts.forEach((FontFace fontFace, _, __) {
atLeastOneFont = true;
Future f1 = fontFace.loaded;
Future f2 = fontFace.loaded;
- expect(f1, equals(f2)); // Repeated calls should answer the same Future.
-
- expect(fontFace.load(), throws);
+ loaded.add(fontFace.load());
+ loaded.add(f1);
+ loaded.add(f2);
});
expect(atLeastOneFont, isTrue);
+ return Future.wait(loaded).then(expectAsync((_) {
+ document.fonts.forEach((fontFace, _, __) {
+ expect(fontFace.status, 'loaded');
+ });
+ }));
});
}
« no previous file with comments | « tests/co19/co19-dartium.status ('k') | tests/html/html.status » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698