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

Unified Diff: pkg/compiler/lib/src/resolution/resolution_strategy.dart

Issue 2850143002: Reuse RuntimeTypesNeedBuilderImpl and _RuntimeTypesNeed for kernel based elements (Closed)
Patch Set: Fix comment Created 3 years, 8 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
Index: pkg/compiler/lib/src/resolution/resolution_strategy.dart
diff --git a/pkg/compiler/lib/src/resolution/resolution_strategy.dart b/pkg/compiler/lib/src/resolution/resolution_strategy.dart
index c173ca30b861911e49c20b03dafa76fd15c92b39..2400bd99d121d7c45ce668f35138d7fce8d33433 100644
--- a/pkg/compiler/lib/src/resolution/resolution_strategy.dart
+++ b/pkg/compiler/lib/src/resolution/resolution_strategy.dart
@@ -105,7 +105,8 @@ class ResolutionFrontEndStrategy implements FrontEndStrategy {
new MirrorsResolutionAnalysisImpl(backend, _compiler.resolution);
RuntimeTypesNeedBuilder createRuntimeTypesNeedBuilder() {
- return new RuntimeTypesNeedBuilderImpl();
+ return new ResolutionRuntimeTypesNeedBuilderImpl(
+ elementEnvironment, _compiler.types);
}
ResolutionWorldBuilder createResolutionWorldBuilder(
@@ -229,6 +230,11 @@ class _CompilerElementEnvironment implements ElementEnvironment {
}
@override
+ bool isGenericClass(ClassEntity cls) {
+ return getThisType(cls).typeArguments.isNotEmpty;
+ }
+
+ @override
ResolutionDartType getTypeVariableBound(TypeVariableElement typeVariable) {
return typeVariable.bound;
}

Powered by Google App Engine
This is Rietveld 408576698