Make the data pipe "cancel all state" functions cancel two-phase operations.

Make the data pipe "cancel all state" functions cancel two-phase operations.

Refactor some code paths to share more code. This leads to (very slight)
redundancy in a couple of code paths, because on closing a handle both
"cancel all state" and "close" get called.

See the commit message for c59a3473231b17af071fc401fdd66ec0adbcdd4c for
details about why this is being done. This CL doesn't actually change
any visible behavior.

R=azani@chromium.org
BUG=#782

Committed: https://chromium.googlesource.com/external/mojo/+/f54da366a84db70d2aa6a747b91f3e30b8e8df00

[viettrungluu, 2016-06-07 20:55:00]

Description was changed from

==========
Make the data pipe "cancel all state" functions cancel two-phase operations.

Refactor some code paths to share more code. This leads to (very slight)
redundancy in a couple of code paths, because on closing a handle both
"cancel all state" and "close" get called.

See the commit message for c59a3473231b17af071fc401fdd66ec0adbcdd4c for
details about why this is being done. This CL doesn't actually change
any visible behavior.

R=azani@chromium.org
==========

to

==========
Make the data pipe "cancel all state" functions cancel two-phase operations.

Refactor some code paths to share more code. This leads to (very slight)
redundancy in a couple of code paths, because on closing a handle both
"cancel all state" and "close" get called.

See the commit message for c59a3473231b17af071fc401fdd66ec0adbcdd4c for
details about why this is being done. This CL doesn't actually change
any visible behavior.

R=azani@chromium.org
BUG=#782
==========

[azani, 2016-06-07 23:22:10]

lgtm

[viettrungluu, 2016-06-08 00:15:29]

Description was changed from

==========
Make the data pipe "cancel all state" functions cancel two-phase operations.

Refactor some code paths to share more code. This leads to (very slight)
redundancy in a couple of code paths, because on closing a handle both
"cancel all state" and "close" get called.

See the commit message for c59a3473231b17af071fc401fdd66ec0adbcdd4c for
details about why this is being done. This CL doesn't actually change
any visible behavior.

R=azani@chromium.org
BUG=#782
==========

to

==========
Make the data pipe "cancel all state" functions cancel two-phase operations.

Refactor some code paths to share more code. This leads to (very slight)
redundancy in a couple of code paths, because on closing a handle both
"cancel all state" and "close" get called.

See the commit message for c59a3473231b17af071fc401fdd66ec0adbcdd4c for
details about why this is being done. This CL doesn't actually change
any visible behavior.

R=azani@chromium.org
BUG=#782

Committed:
https://chromium.googlesource.com/external/mojo/+/f54da366a84db70d2aa6a747b91...
==========

[viettrungluu, 2016-06-08 00:15:31]

Committed patchset #1 (id:1) manually as
f54da366a84db70d2aa6a747b91f3e30b8e8df00 (presubmit successful).