Remove support for "all or none" two-phase data pipe read/write.

mojo/go/tests/system_test.go

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 package tests
 
 import (
         "bytes"
         "testing"
 
@@ skipping 187 matching lines @@
         if r != system.MOJO_RESULT_OK {
                 t.Fatalf("Failed ReadData on hc:%v", r)
         }
         if !bytes.Equal(data, kHello) {
                 t.Fatalf("Invalid data expected:%s, got:%s", kHello, data)
         }
 
         // Test two-phase read/write.
         // Writing.
         kHello = []byte("Hello, world!")
-        r, buf := hp.BeginWriteData(len(kHello), system.MOJO_WRITE_DATA_FLAG_NONE)
+        r, buf := hp.BeginWriteData(system.MOJO_WRITE_DATA_FLAG_NONE)
         if r != system.MOJO_RESULT_OK {
-                t.Fatalf("Failed BeginWriteData on hp:%v numBytes:%d", r, len(kHello))
+                t.Fatalf("Failed BeginWriteData on hp:%v", r)
         }
         if len(buf) < len(kHello) {
                 t.Fatalf("Buffer size(%d) should be at least %d", len(buf), len(kHello))
         }
         copy(buf, kHello)
         if r, _ := hp.WriteData(kHello, system.MOJO_WRITE_DATA_FLAG_NONE); r != system.MOJO_RESULT_BUSY {
                 t.Fatalf("hp should be busy during a two-phase write: %v", r)
         }
         if r, _ = hc.Wait(system.MOJO_HANDLE_SIGNAL_READABLE, 0); r != system.MOJO_RESULT_DEADLINE_EXCEEDED {
                 t.Fatalf("hc shouldn't be readable before EndWriteData on hp:%v", r)
         }
         if r := hp.EndWriteData(len(kHello)); r != system.MOJO_RESULT_OK {
                 t.Fatalf("Failed EndWriteData on hp:%v", r)
         }
         // Reading.
         if r, _ = hc.Wait(system.MOJO_HANDLE_SIGNAL_READABLE, system.MOJO_DEADLINE_INDEFINITE); r != system.MOJO_RESULT_OK {
                 t.Fatalf("hc should be readable after EndWriteData on hp:%v", r)
         }
-        if r, buf = hc.BeginReadData(len(kHello), system.MOJO_READ_DATA_FLAG_NONE); r != system.MOJO_RESULT_OK {
-                t.Fatalf("Failed BeginReadData on hc:%v numBytes:%d", r, len(kHello))
+        if r, buf = hc.BeginReadData(system.MOJO_READ_DATA_FLAG_NONE); r != system.MOJO_RESULT_OK {
+                t.Fatalf("Failed BeginReadData on hc:%v", r)
         }
         if len(buf) != len(kHello) {
                 t.Fatalf("Buffer size(%d) should be equal to %d", len(buf), len(kHello))
         }
         if r, _ := hc.ReadData(system.MOJO_READ_DATA_FLAG_NONE); r != system.MOJO_RESULT_BUSY {
                 t.Fatalf("hc should be busy during a two-phase read: %v", r)
         }
         if !bytes.Equal(buf, kHello) {
                 t.Fatalf("Invalid data expected:%s, got:%s", kHello, buf)
         }
@@ skipping 52 matching lines @@
         if buf[0] != 'x' || buf[1] != 'y' {
                 t.Fatalf("Failed to validate shared buffer. expected:x,y got:%s,%s", buf[0], buf[1])
         }
         if r = h1.UnmapBuffer(buf); r != system.MOJO_RESULT_OK {
                 t.Fatalf("UnmapBuffer failed:%v", r)
         }
         if r = h1.Close(); r != system.MOJO_RESULT_OK {
                 t.Fatalf("Close on h1 failed:%v", r)
         }
 }