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

Unified Diff: mojo/public/js/core_unittests.js

Issue 716453004: Mojo JS Bindings: revert the Mojo handle toString() method (Closed) Base URL: https://github.com/domokit/mojo.git@master
Patch Set: 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « mojo/edk/js/handle.cc ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: mojo/public/js/core_unittests.js
diff --git a/mojo/public/js/core_unittests.js b/mojo/public/js/core_unittests.js
index 44f24d32ed4a273cdf08ec7350db47f856b96bff..eaeaa4f74aeb9ada6e199bb3408fb89fcec1d8ff 100644
--- a/mojo/public/js/core_unittests.js
+++ b/mojo/public/js/core_unittests.js
@@ -15,7 +15,6 @@ define([
runWithDataPipe(testReadAndWriteDataPipe);
runWithDataPipeWithOptions(testNop);
runWithDataPipeWithOptions(testReadAndWriteDataPipe);
- testHandleToString();
gc.collectGarbage(); // should not crash
this.result = "PASS";
@@ -126,18 +125,4 @@ define([
expect(memory[i]).toBe((i * i) & 0xFF);
}
- function testHandleToString() {
- var pipe = core.createDataPipe();
- expect(pipe.consumerHandle.toString).toBeDefined();
-
- var openHandleRE = /^\[mojo\:\:Handle \d+\]$/ // e.g. "[mojo::Handle 123]"
- var openHandleString = pipe.consumerHandle.toString();
- expect(openHandleString.match(openHandleRE)[0]).toEqual(openHandleString);
-
- expect(core.close(pipe.producerHandle)).toBe(core.RESULT_OK);
- expect(core.close(pipe.consumerHandle)).toBe(core.RESULT_OK);
-
- expect(pipe.consumerHandle.toString()).toEqual("[mojo::Handle null]");
- }
-
});
« no previous file with comments | « mojo/edk/js/handle.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698