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

Unified Diff: mojo/public/c/system/tests/core_unittest.cc

Issue 220113003: Mojo: Implement MojoUnmapBuffer(). (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: rebased Created 6 years, 9 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/mojo.gyp ('k') | mojo/system/constants.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: mojo/public/c/system/tests/core_unittest.cc
diff --git a/mojo/public/c/system/tests/core_unittest.cc b/mojo/public/c/system/tests/core_unittest.cc
index c9acd00cfa2d8dddaf93164aec0d57d6a56db32f..bb6c45883da978f192db5c10f8c6dba7e19b36e8 100644
--- a/mojo/public/c/system/tests/core_unittest.cc
+++ b/mojo/public/c/system/tests/core_unittest.cc
@@ -251,8 +251,7 @@ TEST(CoreTest, BasicSharedBuffer) {
static_cast<char*>(pointer)[51] = 'y';
// Unmap it.
- // TODO(vtl): Not yet implemented.
- EXPECT_EQ(MOJO_RESULT_UNIMPLEMENTED, MojoUnmapBuffer(pointer));
+ EXPECT_EQ(MOJO_RESULT_OK, MojoUnmapBuffer(pointer));
// Map half of |h1|.
pointer = NULL;
@@ -265,8 +264,7 @@ TEST(CoreTest, BasicSharedBuffer) {
EXPECT_EQ('y', static_cast<char*>(pointer)[1]);
// Unmap it.
- // TODO(vtl): Not yet implemented.
- EXPECT_EQ(MOJO_RESULT_UNIMPLEMENTED, MojoUnmapBuffer(pointer));
+ EXPECT_EQ(MOJO_RESULT_OK, MojoUnmapBuffer(pointer));
EXPECT_EQ(MOJO_RESULT_OK, MojoClose(h1));
}
« no previous file with comments | « mojo/mojo.gyp ('k') | mojo/system/constants.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698