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

Side by Side Diff: tools/testing/dart/browser_controller.dart

Issue 49043003: Run all tests inside an iframe + restrict number of chrome browser processes on mac (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Created 7 years, 1 month 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 unified diff | Download patch | Annotate | Revision Log
« no previous file with comments | « tools/test.dart ('k') | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright (c) 2013, the Dart project authors. Please see the AUTHORS file 1 // Copyright (c) 2013, the Dart project authors. Please see the AUTHORS file
2 // for details. All rights reserved. Use of this source code is governed by a 2 // for details. All rights reserved. Use of this source code is governed by a
3 // BSD-style license that can be found in the LICENSE file. 3 // BSD-style license that can be found in the LICENSE file.
4 library browser; 4 library browser;
5 5
6 import "dart:async"; 6 import "dart:async";
7 import "dart:convert" show LineSplitter, UTF8; 7 import "dart:convert" show LineSplitter, UTF8;
8 import "dart:core"; 8 import "dart:core";
9 import "dart:io"; 9 import "dart:io";
10 10
(...skipping 57 matching lines...) Expand 10 before | Expand all | Expand 10 after
68 } else if (name.startsWith('ie')) { 68 } else if (name.startsWith('ie')) {
69 return new IE(); 69 return new IE();
70 } else { 70 } else {
71 throw "Non supported browser"; 71 throw "Non supported browser";
72 } 72 }
73 } 73 }
74 74
75 static const List<String> SUPPORTED_BROWSERS = 75 static const List<String> SUPPORTED_BROWSERS =
76 const ['safari', 'ff', 'firefox', 'chrome', 'ie9', 'ie10', 'dartium']; 76 const ['safari', 'ff', 'firefox', 'chrome', 'ie9', 'ie10', 'dartium'];
77 77
78 static const List<String> BROWSERS_WITH_WINDOW_SUPPORT = 78 static const List<String> BROWSERS_WITH_WINDOW_SUPPORT = const [];
79 const ['safari', 'ff', 'firefox', 'chrome'];
ricow1 2013/10/28 16:29:56 this really should not have an effect
kustermann 2013/10/28 16:31:08 I think it will make it faster and hopefully more
80 79
81 // TODO(kustermann): add standard support for chrome on android 80 // TODO(kustermann): add standard support for chrome on android
82 static bool supportedBrowser(String name) { 81 static bool supportedBrowser(String name) {
83 return SUPPORTED_BROWSERS.contains(name); 82 return SUPPORTED_BROWSERS.contains(name);
84 } 83 }
85 84
86 void _logEvent(String event) { 85 void _logEvent(String event) {
87 String toLog = "$this ($id) - $event \n"; 86 String toLog = "$this ($id) - $event \n";
88 if (debugPrint) print("usageLog: $toLog"); 87 if (debugPrint) print("usageLog: $toLog");
89 if (logger != null) logger(toLog); 88 if (logger != null) logger(toLog);
(...skipping 1261 matching lines...) Expand 10 before | Expand all | Expand 10 after
1351 Dart test driver, number of tests: <div id="number"></div><br> 1350 Dart test driver, number of tests: <div id="number"></div><br>
1352 Currently executing: <div id="currently_executing"></div><br> 1351 Currently executing: <div id="currently_executing"></div><br>
1353 Unhandled error: <div id="unhandled_error"></div> 1352 Unhandled error: <div id="unhandled_error"></div>
1354 <iframe id="embedded_iframe"></iframe> 1353 <iframe id="embedded_iframe"></iframe>
1355 </body> 1354 </body>
1356 </html> 1355 </html>
1357 """; 1356 """;
1358 return driverContent; 1357 return driverContent;
1359 } 1358 }
1360 } 1359 }
OLDNEW
« no previous file with comments | « tools/test.dart ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698