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

Side by Side Diff: pkg/compiler/lib/src/js_emitter/old_emitter/container_builder.dart

Issue 744823004: Revert "Encode super calls via extra properties on prototypes." (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: keep desirable changes Created 6 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
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 4
5 part of dart2js.js_emitter; 5 part of dart2js.js_emitter;
6 6
7 /// This class should morph into something that makes it easy to build 7 /// This class should morph into something that makes it easy to build
8 /// JavaScript representations of libraries, class-sides, and instance-sides. 8 /// JavaScript representations of libraries, class-sides, and instance-sides.
9 /// Initially, it is just a placeholder for code that is moved from 9 /// Initially, it is just a placeholder for code that is moved from
10 /// [CodeEmitterTask]. 10 /// [CodeEmitterTask].
(...skipping 396 matching lines...) Expand 10 before | Expand all | Expand 10 after
407 isClosure: isClosure, 407 isClosure: isClosure,
408 tearOffName: tearOffName, 408 tearOffName: tearOffName,
409 canBeReflected: canBeReflected, 409 canBeReflected: canBeReflected,
410 canBeApplied: canBeApplied, 410 canBeApplied: canBeApplied,
411 needStructuredInfo: needStructuredInfo); 411 needStructuredInfo: needStructuredInfo);
412 412
413 } 413 }
414 414
415 void addMemberMethodFromInfo(MemberInfo info, ClassBuilder builder) { 415 void addMemberMethodFromInfo(MemberInfo info, ClassBuilder builder) {
416 final FunctionElement member = info.member; 416 final FunctionElement member = info.member;
417 String name = info.name; 417 final String name = info.name;
418 final FunctionSignature parameters = info.parameters; 418 final FunctionSignature parameters = info.parameters;
419 jsAst.Expression code = info.code; 419 jsAst.Expression code = info.code;
420 final bool needsStubs = info.needsStubs; 420 final bool needsStubs = info.needsStubs;
421 final bool canTearOff = info.canTearOff; 421 final bool canTearOff = info.canTearOff;
422 final bool isClosure = info.isClosure; 422 final bool isClosure = info.isClosure;
423 final String tearOffName = info.tearOffName; 423 final String tearOffName = info.tearOffName;
424 final bool canBeReflected = info.canBeReflected; 424 final bool canBeReflected = info.canBeReflected;
425 final bool canBeApplied = info.canBeApplied; 425 final bool canBeApplied = info.canBeApplied;
426 final bool needStructuredInfo = info.needStructuredInfo; 426 final bool needStructuredInfo = info.needStructuredInfo;
427 427
428 emitter.interceptorEmitter.recordMangledNameOfMemberMethod(member, name); 428 emitter.interceptorEmitter.recordMangledNameOfMemberMethod(member, name);
429 429
430 // If this member is called from a subclass via super, we have to add a
431 // renaming. This is encoded by using two names separated with a : as
432 // the property name.
433 if (backend.isAliasedSuperMember(member)) {
434 name = "$name:${namer.getNameOfAliasedSuperMember(member)}";
435 }
436
437 if (!needStructuredInfo) { 430 if (!needStructuredInfo) {
438 compiler.dumpInfoTask.registerElementAst(member, 431 compiler.dumpInfoTask.registerElementAst(member,
439 builder.addProperty(name, code)); 432 builder.addProperty(name, code));
440 if (needsStubs) { 433 if (needsStubs) {
441 addParameterStubs( 434 addParameterStubs(
442 member, 435 member,
443 (Selector selector, jsAst.Fun function) { 436 (Selector selector, jsAst.Fun function) {
444 compiler.dumpInfoTask.registerElementAst(member, 437 compiler.dumpInfoTask.registerElementAst(member,
445 builder.addProperty(namer.invocationName(selector), function)) ; 438 builder.addProperty(namer.invocationName(selector), function)) ;
446 }); 439 });
(...skipping 193 matching lines...) Expand 10 before | Expand all | Expand 10 after
640 assert(code != null); 633 assert(code != null);
641 assert(needsStubs != null); 634 assert(needsStubs != null);
642 assert(canTearOff != null); 635 assert(canTearOff != null);
643 assert(isClosure != null); 636 assert(isClosure != null);
644 assert(tearOffName != null || !canTearOff); 637 assert(tearOffName != null || !canTearOff);
645 assert(canBeReflected != null); 638 assert(canBeReflected != null);
646 assert(canBeApplied != null); 639 assert(canBeApplied != null);
647 assert(needStructuredInfo != null); 640 assert(needStructuredInfo != null);
648 } 641 }
649 } 642 }
OLDNEW
« no previous file with comments | « pkg/compiler/lib/src/js_backend/namer.dart ('k') | pkg/compiler/lib/src/js_emitter/old_emitter/emitter.dart » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698