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

Unified Diff: tests/html/fileapi_test.dart

Issue 11881007: Revert "Adding support checks for FileSystem APIs and making APIs not webkit-specific." (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Created 7 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
« no previous file with comments | « sdk/lib/html/dartium/html_dartium.dart ('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/fileapi_test.dart
diff --git a/tests/html/fileapi_test.dart b/tests/html/fileapi_test.dart
index dbd89ff5686a5e75168e1badd39738dbfacd9c3c..f7155b615824eb1d92f5dd3c962daccc825302b2 100644
--- a/tests/html/fileapi_test.dart
+++ b/tests/html/fileapi_test.dart
@@ -1,6 +1,6 @@
library fileapi;
import '../../pkg/unittest/lib/unittest.dart';
-import '../../pkg/unittest/lib/html_individual_config.dart';
+import '../../pkg/unittest/lib/html_config.dart';
import 'dart:html';
void fail(message) {
@@ -12,91 +12,68 @@ void fail(message) {
FileSystem fs;
main() {
- useHtmlIndividualConfiguration();
-
- group('supported', () {
- test('supported', () {
- expect(FileSystem.supported, true);
- });
- });
-
- getFileSystem() {
- window.requestFileSystem(Window.TEMPORARY, 100,
- expectAsync1((FileSystem fileSystem) {
- fs = fileSystem;
- }),
- (e) {
- fail('Got file error: ${e.code}');
- });
- }
-
- group('fileSystem', () {
- test('getFileSystem', () {
- var expectation = FileSystem.supported ? returnsNormally : throws;
- expect(() {
- getFileSystem();
- }, expectation);
- });
+ useHtmlConfiguration();
+ test('getFileSystem', () {
+ window.webkitRequestFileSystem(Window.TEMPORARY, 100, expectAsync1(
+ (FileSystem fileSystem) {
+ fs = fileSystem;
+ }),
+ (e) {
+ fail('Got file error: ${e.code}');
+ });
});
-
group('getDirectory', () {
- if (FileSystem.supported) {
- test('getFileSystem', getFileSystem);
- test('directoryDoesntExist', () {
- fs.root.getDirectory(
- 'directory2',
- options: {},
- successCallback: (e) {
- fail('Should not be reached');
- },
- errorCallback: expectAsync1((FileError e) {
- expect(e.code, equals(FileError.NOT_FOUND_ERR));
- }));
- });
+ test('directoryDoesntExist', () {
+ fs.root.getDirectory(
+ 'directory2',
+ options: {},
+ successCallback: (e) {
+ fail('Should not be reached');
+ },
+ errorCallback: expectAsync1((FileError e) {
+ expect(e.code, equals(FileError.NOT_FOUND_ERR));
+ }));
+ });
- test('directoryCreate', () {
- fs.root.getDirectory(
- 'directory3',
- options: {'create': true},
- successCallback: expectAsync1((DirectoryEntry e) {
- expect(e.name, equals('directory3'));
- }),
- errorCallback: (e) {
- fail('Got file error: ${e.code}');
- });
- });
- }
+ test('directoryCreate', () {
+ fs.root.getDirectory(
+ 'directory3',
+ options: {'create': true},
+ successCallback: expectAsync1((DirectoryEntry e) {
+ expect(e.name, equals('directory3'));
+ }),
+ errorCallback: (e) {
+ fail('Got file error: ${e.code}');
+ });
+ });
});
group('getFile', () {
- if (FileSystem.supported) {
- test('getFileSystem', getFileSystem);
- test('fileDoesntExist', () {
- fs.root.getFile(
- 'file2',
- options: {},
- successCallback: (e) {
- fail('Should not be reached');
- },
- errorCallback: expectAsync1((FileError e) {
- expect(e.code, equals(FileError.NOT_FOUND_ERR));
- }));
- });
+ test('fileDoesntExist', () {
+ fs.root.getFile(
+ 'file2',
+ options: {},
+ successCallback: (e) {
+ fail('Should not be reached');
+ },
+ errorCallback: expectAsync1((FileError e) {
+ expect(e.code, equals(FileError.NOT_FOUND_ERR));
+ }));
+ });
- test('fileCreate', () {
- fs.root.getFile(
- 'file4',
- options: {'create': true},
- successCallback: expectAsync1((FileEntry e) {
- expect(e.name, equals('file4'));
- expect(e.isFile, isTrue);
- }),
- errorCallback: (e) {
- fail('Got file error: ${e.code}');
- });
- });
- }
+ test('fileCreate', () {
+ fs.root.getFile(
+ 'file4',
+ options: {'create': true},
+ successCallback: expectAsync1((FileEntry e) {
+ expect(e.name, equals('file4'));
+ expect(e.isFile, isTrue);
+ }),
+ errorCallback: (e) {
+ fail('Got file error: ${e.code}');
+ });
+ });
});
}
« no previous file with comments | « sdk/lib/html/dartium/html_dartium.dart ('k') | tests/html/html.status » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698