Fix a bunch of mojo_public_*_unittests with the new EDK.

mojo/public/c/system/tests/core_unittest.cc

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 a5cb2d7d9e4a2072d22314272cdb50e0b6ea1dc8..ef44a8748816157d354e673ed39937ed263c2aaf 100644
--- a/mojo/public/c/system/tests/core_unittest.cc
+++ b/mojo/public/c/system/tests/core_unittest.cc
@@ -156,6 +156,10 @@ TEST(CoreTest, BasicMessagePipe) {
   // Close |h0|.
   EXPECT_EQ(MOJO_RESULT_OK, MojoClose(h0));
 
+  EXPECT_EQ(MOJO_RESULT_OK,
+            MojoWait(h1, MOJO_HANDLE_SIGNAL_PEER_CLOSED,
+                     MOJO_DEADLINE_INDEFINITE, &state));
+
   // |h1| should no longer be readable or writable.
   EXPECT_EQ(
       MOJO_RESULT_FAILED_PRECONDITION,
@@ -250,7 +254,8 @@ TEST(CoreTest, BasicDataPipe) {
 
   // |hc| should still be readable.

[yzshen1, 2016/01/04 18:54:25]

nit: Maybe we could consider keeping the wait for READABLE call (and adding a
wait for PEER_CLOSED as you did), or update this comment?

[jam, 2016/01/04 21:00:06]

well, we still do that check for readable in line 260.

   EXPECT_EQ(MOJO_RESULT_OK,
-            MojoWait(hc, MOJO_HANDLE_SIGNAL_READABLE, 0, &state));
+            MojoWait(hc, MOJO_HANDLE_SIGNAL_PEER_CLOSED,
+                     MOJO_DEADLINE_INDEFINITE, &state));
 
   EXPECT_EQ(MOJO_HANDLE_SIGNAL_READABLE | MOJO_HANDLE_SIGNAL_PEER_CLOSED,
             state.satisfied_signals);