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

Unified Diff: tests/html/fontface_loaded_test.dart

Issue 1345083002: Revert "Dartium JS Interop enabled." (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 8a80f5d4d500fc8cd7938e107f2b6c2a6f8358ba..d1926880ae89c66f7f052293d236fb2906193cb7 100644
--- a/tests/html/fontface_loaded_test.dart
+++ b/tests/html/fontface_loaded_test.dart
@@ -8,18 +8,17 @@ 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(/root_dart/tests/html/Ahem.ttf);
+ src: url(../../resources/Ahem.ttf);
font-style: italic;
font-weight: 300;
unicode-range: U+0-3FF;
@@ -28,26 +27,20 @@ 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;
- loaded.add(fontFace.load());
- loaded.add(f1);
- loaded.add(f2);
+ expect(f1, equals(f2)); // Repeated calls should answer the same Future.
+
+ expect(fontFace.load(), throws);
});
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