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

Unified Diff: pkg/dev_compiler/lib/js/es6/dart_sdk.js

Issue 2666053004: Remove dynamic call from _LinkedHashSet. Remove some dcalls from the dart:html util library. (Closed)
Patch Set: Remove dynamic call from _LinkedHashSet. Remove some dcalls from the dart:html util library. Created 3 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:
Download patch
Index: pkg/dev_compiler/lib/js/es6/dart_sdk.js
diff --git a/pkg/dev_compiler/lib/js/es6/dart_sdk.js b/pkg/dev_compiler/lib/js/es6/dart_sdk.js
index e24a6640f69474f74912d63c4a260b4f4b434904..21ad7a734a533771dfe684d5d057b20325905923 100644
--- a/pkg/dev_compiler/lib/js/es6/dart_sdk.js
+++ b/pkg/dev_compiler/lib/js/es6/dart_sdk.js
@@ -25725,13 +25725,13 @@ collection._LinkedHashSetIterator$ = dart.generic(E => {
this[_modifications] = modifications;
this[_cell] = null;
this[_current] = null;
- this[_cell] = collection._LinkedHashSetCell._check(dart.dload(this[_set], _first));
+ this[_cell] = this[_set][_first];
}
get current() {
return this[_current];
}
moveNext() {
- if (!dart.equals(this[_modifications], dart.dload(this[_set], _modifications))) {
+ if (this[_modifications] != this[_set][_modifications]) {
dart.throw(new core.ConcurrentModificationError(this[_set]));
} else if (this[_cell] == null) {
this[_current] = null;
@@ -25746,9 +25746,9 @@ collection._LinkedHashSetIterator$ = dart.generic(E => {
dart.addTypeTests(_LinkedHashSetIterator);
_LinkedHashSetIterator[dart.implements] = () => [IteratorOfE()];
dart.setSignature(_LinkedHashSetIterator, {
- constructors: () => ({new: dart.definiteFunctionType(collection._LinkedHashSetIterator$(E), [dart.dynamic, core.int])}),
+ constructors: () => ({new: dart.definiteFunctionType(collection._LinkedHashSetIterator$(E), [collection._LinkedHashSet, core.int])}),
fields: () => ({
- [_set]: dart.dynamic,
+ [_set]: collection._LinkedHashSet,
[_modifications]: core.int,
[_cell]: collection._LinkedHashSetCell,
[_current]: E
@@ -99193,7 +99193,7 @@ html_common._StructuredClone = class _StructuredClone extends core.Object {
let copy = this.newJsList(length);
this.writeSlot(slot, copy);
for (; i < dart.notNull(length); i++) {
- dart.dsetindex(copy, i, this.walk(e[dartx._get](i)));
+ copy[dartx._set](i, this.walk(e[dartx._get](i)));
}
return copy;
}
@@ -99214,7 +99214,7 @@ dart.setSignature(html_common._StructuredClone, {
writeSlot: dart.definiteFunctionType(dart.dynamic, [core.int, dart.dynamic]),
cleanupSlots: dart.definiteFunctionType(dart.dynamic, []),
walk: dart.definiteFunctionType(dart.dynamic, [dart.dynamic]),
- copyList: dart.definiteFunctionType(dart.dynamic, [core.List, core.int]),
+ copyList: dart.definiteFunctionType(core.List, [core.List, core.int]),
convertDartToNative_PrepareForStructuredClone: dart.definiteFunctionType(dart.dynamic, [dart.dynamic])
})
});
@@ -99263,14 +99263,15 @@ html_common._AcceptStructuredClone = class _AcceptStructuredClone extends core.O
return copy;
}
if (dart.test(html_common.isJavaScriptArray(e))) {
- let slot = this.findSlot(e);
+ let l = e;
+ let slot = this.findSlot(l);
let copy = this.readSlot(slot);
if (copy != null) return copy;
- let length = core.int._check(dart.dload(e, 'length'));
- copy = dart.test(this.mustCopy) ? this.newDartList(length) : e;
+ let length = l[dartx.length];
+ copy = dart.test(this.mustCopy) ? this.newDartList(length) : l;
this.writeSlot(slot, copy);
for (let i = 0; i < dart.notNull(length); i++) {
- dart.dsetindex(copy, i, this.walk(dart.dindex(e, i)));
+ copy[dartx._set](i, this.walk(l[dartx._get](i)));
}
return copy;
}
@@ -99431,7 +99432,7 @@ dart.setSignature(html_common._StructuredCloneDart2Js, {
methods: () => ({
newJsMap: dart.definiteFunctionType(dart.dynamic, []),
putIntoMap: dart.definiteFunctionType(dart.void, [dart.dynamic, dart.dynamic, dart.dynamic]),
- newJsList: dart.definiteFunctionType(dart.dynamic, [dart.dynamic]),
+ newJsList: dart.definiteFunctionType(core.List, [dart.dynamic]),
cloneNotRequired: dart.definiteFunctionType(core.bool, [dart.dynamic])
})
});
@@ -99456,10 +99457,10 @@ html_common._AcceptStructuredCloneDart2Js = class _AcceptStructuredCloneDart2Js
};
dart.setSignature(html_common._AcceptStructuredCloneDart2Js, {
methods: () => ({
- newJsList: dart.definiteFunctionType(dart.dynamic, [dart.dynamic]),
- newDartList: dart.definiteFunctionType(dart.dynamic, [dart.dynamic]),
+ newJsList: dart.definiteFunctionType(core.List, [dart.dynamic]),
+ newDartList: dart.definiteFunctionType(core.List, [dart.dynamic]),
identicalInJs: dart.definiteFunctionType(core.bool, [dart.dynamic, dart.dynamic]),
- forEachJsField: dart.definiteFunctionType(dart.void, [dart.dynamic, dart.dynamic])
+ forEachJsField: dart.definiteFunctionType(dart.void, [dart.dynamic, dynamicAnddynamicTodynamic()])
})
});
html_common.isJavaScriptDate = function(value) {

Powered by Google App Engine
This is Rietveld 408576698