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

Unified Diff: mojo/dart/test/handle_finalizer_test.dart

Issue 800523004: Dart: Simplifies the handle watcher. Various cleanups and bugfixes. (Closed) Base URL: git@github.com:domokit/mojo.git@master
Patch Set: cleanup Created 5 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:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « mojo/dart/test/core_test.dart ('k') | mojo/dart/test/handle_watcher_test.dart » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: mojo/dart/test/handle_finalizer_test.dart
diff --git a/mojo/dart/test/handle_finalizer_test.dart b/mojo/dart/test/handle_finalizer_test.dart
index 9e3bab97c197c7dc49a000cb6355a0064bad26cc..b2d8f53dede4e70de19feb02d221fe52987e613f 100644
--- a/mojo/dart/test/handle_finalizer_test.dart
+++ b/mojo/dart/test/handle_finalizer_test.dart
@@ -9,14 +9,14 @@ import 'dart:typed_data';
import 'package:mojo/dart/testing/expect.dart';
-RawMojoHandle leakMojoHandle() {
+MojoHandle leakMojoHandle() {
var pipe = new MojoMessagePipe();
Expect.isNotNull(pipe);
var endpoint = pipe.endpoints[0];
Expect.isTrue(endpoint.handle.isValid);
- var handle = new MojoHandle(endpoint.handle);
+ var eventStream = new MojoEventStream(endpoint.handle);
// After making a high-level MojoHandle, the underlying mojo handle will have
// the native MojoClose called on it when the MojoHandle is GC'd or the VM
// shuts down.
@@ -31,7 +31,7 @@ List<int> triggerGC() {
}
test() {
- RawMojoHandle handle = leakMojoHandle();
+ MojoHandle handle = leakMojoHandle();
triggerGC();
// The handle will be closed by the MojoHandle finalizer during GC, so the
« no previous file with comments | « mojo/dart/test/core_test.dart ('k') | mojo/dart/test/handle_watcher_test.dart » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698