commit 833e71a438e83bd654b9e71af1676a2be93459fc
author Viet-Trung Luu <viettrungluu@chromium.org> Tue Apr 05 14:17:48 2016 -0700
committer Viet-Trung Luu <viettrungluu@chromium.org> Tue Apr 05 14:17:48 2016 -0700
tree a0aba3d8b456dd1f5b6b3d229c327983bbd4e7d7
parent 15c6fa026b23a71407f4597132e60a5a73f8dd25

Let MojoSetDataPipeConsumerOptions() take null options.

The header defining it says that doing so means reset back to the
default options. I forgot about this when implementing it. Oops. :-/

R=vardhan@google.com
BUG=#442

Review URL: https://codereview.chromium.org/1865663002 .

mojo/edk/system/core_unittest.cc

diff --git a/mojo/edk/system/core_unittest.cc b/mojo/edk/system/core_unittest.cc
index 0fdd5ac..bb8967a 100644
--- a/mojo/edk/system/core_unittest.cc
+++ b/mojo/edk/system/core_unittest.cc
@@ -1250,11 +1250,9 @@
   EXPECT_EQ(kCoptsSize, copts.struct_size);
   EXPECT_EQ(16u, copts.read_threshold_num_bytes);
 
-  // Default read threshold.
-  copts.struct_size = kCoptsSize;
-  copts.read_threshold_num_bytes = 0;
+  // Can also set to default by passing null.
   EXPECT_EQ(MOJO_RESULT_OK,
-            core()->SetDataPipeConsumerOptions(ch, MakeUserPointer(&copts)));
+            core()->SetDataPipeConsumerOptions(ch, NullUserPointer()));
   copts = MojoDataPipeConsumerOptions();
   EXPECT_EQ(MOJO_RESULT_OK, core()->GetDataPipeConsumerOptions(
                                 ch, MakeUserPointer(&copts), kCoptsSize));