Skip to content

Commit

Permalink
remove #177 workaround
Browse files Browse the repository at this point in the history
this issue was fixed

R=vsm@google.com

Review URL: https://codereview.chromium.org/1142553002
  • Loading branch information
John Messerly committed May 14, 2015
1 parent 3377d2b commit 995e0f0
Show file tree
Hide file tree
Showing 3 changed files with 7 additions and 11 deletions.
6 changes: 3 additions & 3 deletions pkg/dev_compiler/lib/runtime/dart/_isolate_helper.js
Original file line number Diff line number Diff line change
Expand Up @@ -778,12 +778,13 @@ var async = dart.import(async);
}
[_runHelper]() {
if (exports.globalWindow != null) {
// Function next: () → dynamic
let next = (() => {
if (!dart.notNull(this.runIteration()))
return;
async.Timer.run(dart.as(next, __CastType2));
async.Timer.run(next);
}).bind(this);
dart.dcall(next);
next();
} else {
while (this.runIteration()) {
}
Expand Down Expand Up @@ -1381,7 +1382,6 @@ var async = dart.import(async);
}
CapabilityImpl[dart.implements] = () => [isolate.Capability];
dart.defineNamedConstructor(CapabilityImpl, '_internal');
let __CastType2 = dart.typedef('__CastType2', () => dart.functionType(dart.void, []));
// Exports:
exports.enterJsAsync = enterJsAsync;
exports.leaveJsAsync = leaveJsAsync;
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -652,12 +652,10 @@ class _EventLoop {
void _runHelper() {
if (globalWindow != null) {
// Run each iteration from the browser's top event loop.
// TODO(vsm): Revert to original pattern.
// See: https://github.com/dart-lang/dev_compiler/issues/177
Function next = () {
next() {
if (!runIteration()) return;
Timer.run(next);
};
}
next();
} else {
// Run synchronously until no more iterations are available.
Expand Down
6 changes: 2 additions & 4 deletions pkg/dev_compiler/tool/input_sdk/private/isolate_helper.dart
Original file line number Diff line number Diff line change
Expand Up @@ -652,12 +652,10 @@ class _EventLoop {
void _runHelper() {
if (globalWindow != null) {
// Run each iteration from the browser's top event loop.
// TODO(vsm): Revert to original pattern.
// See: https://github.com/dart-lang/dev_compiler/issues/177
Function next = () {
next() {
if (!runIteration()) return;
Timer.run(next);
};
}
next();
} else {
// Run synchronously until no more iterations are available.
Expand Down

0 comments on commit 995e0f0

Please sign in to comment.