commit | c1deace1f39cbc605fa86973daeedb4cd70e1562 | [log] [tgz] |
---|---|---|
author | Han Leon <leon.han@intel.com> | Fri Aug 03 03:52:53 2018 |
committer | Commit Bot <commit-bot@chromium.org> | Fri Aug 03 03:52:53 2018 |
tree | 29d1fa47b269702c5be32c3e739999a0b7aced99 | |
parent | d50b7b5d2fac65722d7cc3936cb8c7da827cb1dd [diff] |
[OnionSoup] Remove struct content::PushEventPayload Currently we have a [Native] mojom struct content.mojom.PushEventPayload with a typemapping to the native struct content::PushEventPayload, we use them to pass payload data when calling the Mojo call content.mojom.ServiceWorker.DispatchPushEvent(). However, as content::PushEventPayload contains only 2 members: a string containing real data and a bool indicating data's validity, and it has no expectations to be expanded for now, so, we can just use a base::Optional<string> to replace it, and correspondingly in mojom we just needs a 'string? payload' for our content.mojom.ServiceWorker.DispatchPushEvent(). This CL removes content::PushEventPayload by doing the above replacement. BUG=789854 Change-Id: I8df737165b3004db820e3b758c1bfa8bdb7b5690 Reviewed-on: https://chromium-review.googlesource.com/1149705 Reviewed-by: Peter Beverloo <peter@chromium.org> Reviewed-by: Makoto Shimazu <shimazu@chromium.org> Reviewed-by: Matt Falkenhagen <falken@chromium.org> Reviewed-by: Kinuko Yasuda <kinuko@chromium.org> Commit-Queue: Han Leon <leon.han@intel.com> Cr-Commit-Position: refs/heads/master@{#580445}
Chromium is an open-source browser project that aims to build a safer, faster, and more stable way for all users to experience the web.
The project's web site is https://www.chromium.org.
Documentation in the source is rooted in docs/README.md.
Learn how to Get Around the Chromium Source Code Directory Structure .