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

Side by Side Diff: pkg/compiler/lib/src/native/native.dart

Issue 2907383002: Don't tag fromEnvironment constructors as native (Closed)
Patch Set: Updated cf. comments Created 3 years, 6 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 unified diff | Download patch
OLDNEW
1 // Copyright (c) 2012, the Dart project authors. Please see the AUTHORS file 1 // Copyright (c) 2012, 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 4
5 library native; 5 library native;
6 6
7 import '../compiler.dart' show Compiler;
8 import '../elements/entities.dart';
9
10 export 'behavior.dart'; 7 export 'behavior.dart';
11 export 'enqueue.dart'; 8 export 'enqueue.dart';
12 export 'js.dart'; 9 export 'js.dart';
13 export 'scanner.dart'; 10 export 'scanner.dart';
14 export 'ssa.dart'; 11 export 'ssa.dart';
15 12
16 const Iterable<String> _allowedDartSchemePaths = const <String>[ 13 const Iterable<String> _allowedDartSchemePaths = const <String>[
17 'async', 14 'async',
18 'html', 15 'html',
19 'html_common', 16 'html_common',
20 'indexed_db', 17 'indexed_db',
21 'js', 18 'js',
22 'js_util', 19 'js_util',
23 'svg', 20 'svg',
24 '_native_typed_data', 21 '_native_typed_data',
25 'web_audio', 22 'web_audio',
26 'web_gl', 23 'web_gl',
27 'web_sql' 24 'web_sql'
28 ]; 25 ];
29 26
30 bool maybeEnableNative(Compiler compiler, LibraryEntity library) { 27 bool maybeEnableNative(Uri uri, {bool allowNativeExtensions: false}) {
31 bool allowedTestLibrary() { 28 bool allowedTestLibrary() {
32 Uri uri = library.canonicalUri;
33 String scriptName = uri.path; 29 String scriptName = uri.path;
34 return scriptName.contains('sdk/tests/compiler/dart2js_native') || 30 return scriptName.contains('sdk/tests/compiler/dart2js_native') ||
35 scriptName.contains('sdk/tests/compiler/dart2js_extra'); 31 scriptName.contains('sdk/tests/compiler/dart2js_extra');
36 } 32 }
37 33
38 bool allowedDartLibary() { 34 bool allowedDartLibary() {
39 Uri uri = library.canonicalUri;
40 if (uri.scheme != 'dart') return false; 35 if (uri.scheme != 'dart') return false;
41 return _allowedDartSchemePaths.contains(uri.path); 36 return _allowedDartSchemePaths.contains(uri.path);
42 } 37 }
43 38
44 return allowedTestLibrary() || 39 return allowedTestLibrary() || allowedDartLibary() || allowNativeExtensions;
45 allowedDartLibary() ||
46 compiler.options.allowNativeExtensions;
47 } 40 }
OLDNEW
« no previous file with comments | « pkg/compiler/lib/src/kernel/element_map_impl.dart ('k') | tests/compiler/dart2js/dart2js.status » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698