Add IPC serialization support for base::Optional

Add IPC serialization support for base::Optional

As part of bug 623556, we need to update one of our IPC structs to use
Optional.

We unfortunately can't migrate to mojo at this time because our IPCs
depend on delivery order relative to other IPCs (in particular,
FrameHostMsg_DidCommitProvisionalLoad). Additional details on ordering
constraints are covered in
https://docs.google.com/document/d/1HJsJ5W2H_3qRdqPAUgAEo10AF8gXPTXZLUET4X4_sII/edit.
As a result, I'd like to add IPC serialization support for Optional
so we can unblock progress on bug 623556.

BUG=623556
Committed: https://crrev.com/18573e16ba6fd3240514d661da4b64ffc0e82da8
Cr-Commit-Position: refs/heads/master@{#403442}

[Bryan McQuade, 2016-06-25 13:41:05]

bmcquade@chromium.org changed reviewers:
+ jln@chromium.org

[Bryan McQuade, 2016-06-25 13:41:29]

bmcquade@chromium.org changed reviewers:
+ tsepez@chromium.org
- jln@chromium.org

[Bryan McQuade, 2016-06-25 13:41:57]

bmcquade@chromium.org changed reviewers:
- tsepez@chromium.org

[Bryan McQuade, 2016-06-27 13:41:49]

Description was changed from

==========
NOT FOR REVIEW: Add IPC serialization support for base::Optional

BUG=
==========

to

==========
NOT FOR REVIEW: Add IPC serialization support for base::Optional

BUG=623556
==========

[Bryan McQuade, 2016-06-27 14:41:57]

Description was changed from

==========
NOT FOR REVIEW: Add IPC serialization support for base::Optional

BUG=623556
==========

to

==========
Add IPC serialization support for base::Optional

As part of bug 623556, we need to update one of our IPC structs to use Optional.

We unfortunately can't migrate to mojo at this time because our IPCs depend on
delivery order relative to other IPCs (in particular,
FrameHostMsg_DidCommitProvisionalLoad). Additional details on ordering
constraints are covered in
https://docs.google.com/document/d/1HJsJ5W2H_3qRdqPAUgAEo10AF8gXPTXZLUET4X4_s....

BUG=623556
==========

[Bryan McQuade, 2016-06-27 14:44:42]

Description was changed from

==========
Add IPC serialization support for base::Optional

As part of bug 623556, we need to update one of our IPC structs to use Optional.

We unfortunately can't migrate to mojo at this time because our IPCs depend on
delivery order relative to other IPCs (in particular,
FrameHostMsg_DidCommitProvisionalLoad). Additional details on ordering
constraints are covered in
https://docs.google.com/document/d/1HJsJ5W2H_3qRdqPAUgAEo10AF8gXPTXZLUET4X4_s....

BUG=623556
==========

to

==========
Add IPC serialization support for base::Optional

As part of bug 623556, we need to update one of our IPC structs to use
Optional.

We unfortunately can't migrate to mojo at this time because our IPCs
depend on delivery order relative to other IPCs (in particular,
FrameHostMsg_DidCommitProvisionalLoad). Additional details on ordering
constraints are covered in
https://docs.google.com/document/d/1HJsJ5W2H_3qRdqPAUgAEo10AF8gXPTXZLUET4X4_s....

BUG=623556
==========

[Bryan McQuade, 2016-06-27 14:48:43]

Description was changed from

==========
Add IPC serialization support for base::Optional

As part of bug 623556, we need to update one of our IPC structs to use
Optional.

We unfortunately can't migrate to mojo at this time because our IPCs
depend on delivery order relative to other IPCs (in particular,
FrameHostMsg_DidCommitProvisionalLoad). Additional details on ordering
constraints are covered in
https://docs.google.com/document/d/1HJsJ5W2H_3qRdqPAUgAEo10AF8gXPTXZLUET4X4_s....

BUG=623556
==========

to

==========
Add IPC serialization support for base::Optional

As part of bug 623556, we need to update one of our IPC structs to use
Optional.

We unfortunately can't migrate to mojo at this time because our IPCs
depend on delivery order relative to other IPCs (in particular,
FrameHostMsg_DidCommitProvisionalLoad). Additional details on ordering
constraints are covered in
https://docs.google.com/document/d/1HJsJ5W2H_3qRdqPAUgAEo10AF8gXPTXZLUET4X4_s....
As a result, I'd like to add IPC serialization support for Optional
so we can unblock progress on bug 623556.

BUG=623556
==========

[Bryan McQuade, 2016-06-27 14:48:53]

Description was changed from

==========
Add IPC serialization support for base::Optional

As part of bug 623556, we need to update one of our IPC structs to use
Optional.

We unfortunately can't migrate to mojo at this time because our IPCs
depend on delivery order relative to other IPCs (in particular,
FrameHostMsg_DidCommitProvisionalLoad). Additional details on ordering
constraints are covered in
https://docs.google.com/document/d/1HJsJ5W2H_3qRdqPAUgAEo10AF8gXPTXZLUET4X4_s....
As a result, I'd like to add IPC serialization support for Optional
so we can unblock progress on bug 623556.

BUG=623556
==========

to

==========
Add IPC serialization support for base::Optional

As part of bug 623556, we need to update one of our IPC structs to use
Optional.

We unfortunately can't migrate to mojo at this time because our IPCs
depend on delivery order relative to other IPCs (in particular,
FrameHostMsg_DidCommitProvisionalLoad). Additional details on ordering
constraints are covered in
https://docs.google.com/document/d/1HJsJ5W2H_3qRdqPAUgAEo10AF8gXPTXZLUET4X4_s....
As a result, I'd like to add IPC serialization support for Optional
so we can unblock progress on bug 623556.

BUG=623556
==========

[Bryan McQuade, 2016-06-30 11:56:04]

bmcquade@chromium.org changed reviewers:
+ dcheng@chromium.org

[Bryan McQuade, 2016-06-30 11:56:33]

dcheng, here's the patch I'd mentioned over email. Please take a look. Thanks!

[dcheng, 2016-07-01 02:24:10]

lgtm

[Bryan McQuade, 2016-07-01 11:56:25]

The CQ bit was checked by bmcquade@chromium.org

[commit-bot: I haz the power, 2016-07-01 11:56:29]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-07-01 13:14:02]

Committed patchset #7 (id:120001)

[commit-bot: I haz the power, 2016-07-01 13:14:09]

CQ bit was unchecked.

[commit-bot: I haz the power, 2016-07-01 13:15:40]

Description was changed from

==========
Add IPC serialization support for base::Optional

As part of bug 623556, we need to update one of our IPC structs to use
Optional.

We unfortunately can't migrate to mojo at this time because our IPCs
depend on delivery order relative to other IPCs (in particular,
FrameHostMsg_DidCommitProvisionalLoad). Additional details on ordering
constraints are covered in
https://docs.google.com/document/d/1HJsJ5W2H_3qRdqPAUgAEo10AF8gXPTXZLUET4X4_s....
As a result, I'd like to add IPC serialization support for Optional
so we can unblock progress on bug 623556.

BUG=623556
==========

to

==========
Add IPC serialization support for base::Optional

As part of bug 623556, we need to update one of our IPC structs to use
Optional.

We unfortunately can't migrate to mojo at this time because our IPCs
depend on delivery order relative to other IPCs (in particular,
FrameHostMsg_DidCommitProvisionalLoad). Additional details on ordering
constraints are covered in
https://docs.google.com/document/d/1HJsJ5W2H_3qRdqPAUgAEo10AF8gXPTXZLUET4X4_s....
As a result, I'd like to add IPC serialization support for Optional
so we can unblock progress on bug 623556.

BUG=623556

Committed: https://crrev.com/18573e16ba6fd3240514d661da4b64ffc0e82da8
Cr-Commit-Position: refs/heads/master@{#403442}
==========

[commit-bot: I haz the power, 2016-07-01 13:15:41]

Patchset 7 (id:??) landed as
https://crrev.com/18573e16ba6fd3240514d661da4b64ffc0e82da8
Cr-Commit-Position: refs/heads/master@{#403442}