mojo::RawChannelWin: Use GetOverlappedResult() to retrive read bytes.

mojo::RawChannelWin: Use GetOverlappedResult() to retrive read bytes.

The MSDN document suggests that with overlapped I/O, |lpNumberOfBytesRead|
parameter of ReadFile() API isn't reliable and shouldn't be used.
GetOverlappedResult() should be used instead, according to the doc.
See: http://msdn.microsoft.com/en-us/library/windows/desktop/aa365467(v=vs.85).aspx

This is a speculative crash fix for crbug.com/41509. The crash happens
only on Windows. The report also implies that there might be some
message duplication.It is possible if the value of |lpNumberOfBytesRead|
is wrong.

BUG=415059
R=viettrungluu@chromium.org

Committed: https://crrev.com/f7ef6dd233c8c0b7d237e35476c44b2c26d2bab0
Cr-Commit-Position: refs/heads/master@{#297500}

[Hajime Morrita, 2014-09-30 18:13:39]

Yuzhu pointed that the MSDN link was wrong. Fixed.

[viettrungluu, 2014-09-30 19:07:40]

lgtm w/nit

https://codereview.chromium.org/616043003/diff/1/mojo/system/raw_channel_win.cc
File mojo/system/raw_channel_win.cc (right):

https://codereview.chromium.org/616043003/diff/1/mojo/system/raw_channel_win....
mojo/system/raw_channel_win.cc:377: NULL,
nullptr

[Hajime Morrita, 2014-09-30 19:34:10]

The CQ bit was checked by morrita@chromium.org

[commit-bot: I haz the power, 2014-09-30 19:34:41]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/616043003/20001

[commit-bot: I haz the power, 2014-09-30 21:25:08]

Committed patchset #2 (id:20001) as b2558938ed65332ec1caf36d81bcdcdd7ae90913

[commit-bot: I haz the power, 2014-09-30 21:25:50]

Patchset 2 (id:??) landed as
https://crrev.com/f7ef6dd233c8c0b7d237e35476c44b2c26d2bab0
Cr-Commit-Position: refs/heads/master@{#297500}