[NaCl SDK] Add NetworkMonitor example.

[NaCl SDK] Add NetworkMonitor example.

BUG=none
R=sbc@chromium.org

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=241723

[Sam Clegg, 2013-12-18 00:17:56]

lgtm

https://codereview.chromium.org/100213012/diff/1/native_client_sdk/src/exampl...
File native_client_sdk/src/examples/api/network_monitor/network_monitor.cc
(right):

https://codereview.chromium.org/100213012/diff/1/native_client_sdk/src/exampl...
native_client_sdk/src/examples/api/network_monitor/network_monitor.cc:53: if
(result < PP_OK_COMPLETIONPENDING) {
Why not != ?

https://codereview.chromium.org/100213012/diff/1/native_client_sdk/src/exampl...
native_client_sdk/src/examples/api/network_monitor/network_monitor.cc:65: if
(result < 0)
Send the error the javascript too?

https://codereview.chromium.org/100213012/diff/1/native_client_sdk/src/exampl...
native_client_sdk/src/examples/api/network_monitor/network_monitor.cc:107:
return "unknown";
invalid?

https://codereview.chromium.org/100213012/diff/1/native_client_sdk/src/exampl...
native_client_sdk/src/examples/api/network_monitor/network_monitor.cc:126:
return "unknown";
Distinguish "invalid" from "unknown"?

[binji, 2013-12-18 01:11:41]

https://codereview.chromium.org/100213012/diff/1/native_client_sdk/src/exampl...
File native_client_sdk/src/examples/api/network_monitor/network_monitor.cc
(right):

https://codereview.chromium.org/100213012/diff/1/native_client_sdk/src/exampl...
native_client_sdk/src/examples/api/network_monitor/network_monitor.cc:53: if
(result < PP_OK_COMPLETIONPENDING) {
On 2013/12/18 00:17:56, Sam Clegg wrote:
> Why not != ?

Done.

https://codereview.chromium.org/100213012/diff/1/native_client_sdk/src/exampl...
native_client_sdk/src/examples/api/network_monitor/network_monitor.cc:65: if
(result < 0)
On 2013/12/18 00:17:56, Sam Clegg wrote:
> Send the error the javascript too?

Done.

https://codereview.chromium.org/100213012/diff/1/native_client_sdk/src/exampl...
native_client_sdk/src/examples/api/network_monitor/network_monitor.cc:107:
return "unknown";
On 2013/12/18 00:17:56, Sam Clegg wrote:
> invalid?

Done.

https://codereview.chromium.org/100213012/diff/1/native_client_sdk/src/exampl...
native_client_sdk/src/examples/api/network_monitor/network_monitor.cc:126:
return "unknown";
On 2013/12/18 00:17:56, Sam Clegg wrote:
> Distinguish "invalid" from "unknown"?

Done.

[Sam Clegg, 2013-12-18 17:29:19]

slgtm

[commit-bot: I haz the power, 2013-12-18 21:42:11]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/binji@chromium.org/100213012/20001

[commit-bot: I haz the power, 2013-12-19 00:34:18]

Change committed as 241723