Let's start mojofying from FrameHostMsg's. Start with - FrameHostMsg_DidChangeName - FrameHostMsg_DidSetFeaturePolicyHeader - FrameHostMsg_EnforceInsecureRequestPolicy and then we can go for the harder ones.
I'm getting with these 2 too: FrameMsg_ExtractSmartClipData FrameHostMsg_SmartClipDataExtracted
I'm taking FrameHostMsg_UpdateEncoding and FrameHostMsg_CancelInitialHistoryLoad
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/8def47f83bb6413eee451db95b15ceb96c24dba8 commit 8def47f83bb6413eee451db95b15ceb96c24dba8 Author: Luciano Pacheco <lucmult@chromium.org> Date: Thu Dec 07 06:52:30 2017 Convert 3 Frame IPC calls to mojo Convert FrameHostMsg_DidChangeName, FrameHostMsg_DidSetFramePolicyHeaders and FrameHostMsg_EnforceInsecureRequestPolicy from IPC calls to mojo. Create mojo conversions for enum FeaturePolicyFeature and struct ParsedFeaturePolicyDeclaration. Update third_party/WebKit/common/feature_policy/OWNERS with IPC SECURITY_OWNERS settings for new mojo files. Bug: 786836 Change-Id: I18d18956197d3c7c5a713c61dd9856c8a31c06f7 Reviewed-on: https://chromium-review.googlesource.com/784372 Commit-Queue: Luciano Pacheco <lucmult@chromium.org> Reviewed-by: Mike West <mkwst@chromium.org> Reviewed-by: Alex Moshchuk <alexmos@chromium.org> Reviewed-by: Ian Clelland <iclelland@chromium.org> Reviewed-by: Sam McNally <sammc@chromium.org> Reviewed-by: Joel Hockey <joelhockey@chromium.org> Cr-Commit-Position: refs/heads/master@{#522370} [modify] https://crrev.com/8def47f83bb6413eee451db95b15ceb96c24dba8/content/browser/frame_host/render_frame_host_feature_policy_unittest.cc [modify] https://crrev.com/8def47f83bb6413eee451db95b15ceb96c24dba8/content/browser/frame_host/render_frame_host_impl.cc [modify] https://crrev.com/8def47f83bb6413eee451db95b15ceb96c24dba8/content/browser/frame_host/render_frame_host_impl.h [modify] https://crrev.com/8def47f83bb6413eee451db95b15ceb96c24dba8/content/common/BUILD.gn [modify] https://crrev.com/8def47f83bb6413eee451db95b15ceb96c24dba8/content/common/frame.mojom [modify] https://crrev.com/8def47f83bb6413eee451db95b15ceb96c24dba8/content/common/frame_messages.h [modify] https://crrev.com/8def47f83bb6413eee451db95b15ceb96c24dba8/content/renderer/render_frame_impl.cc [modify] https://crrev.com/8def47f83bb6413eee451db95b15ceb96c24dba8/content/test/test_render_frame.cc [modify] https://crrev.com/8def47f83bb6413eee451db95b15ceb96c24dba8/content/test/test_render_frame_host.cc [modify] https://crrev.com/8def47f83bb6413eee451db95b15ceb96c24dba8/third_party/WebKit/common/BUILD.gn [modify] https://crrev.com/8def47f83bb6413eee451db95b15ceb96c24dba8/third_party/WebKit/common/feature_policy/OWNERS [add] https://crrev.com/8def47f83bb6413eee451db95b15ceb96c24dba8/third_party/WebKit/common/feature_policy/feature_policy.mojom [add] https://crrev.com/8def47f83bb6413eee451db95b15ceb96c24dba8/third_party/WebKit/common/feature_policy/feature_policy.typemap [add] https://crrev.com/8def47f83bb6413eee451db95b15ceb96c24dba8/third_party/WebKit/common/feature_policy/feature_policy_struct_traits.cc [add] https://crrev.com/8def47f83bb6413eee451db95b15ceb96c24dba8/third_party/WebKit/common/feature_policy/feature_policy_struct_traits.h [modify] https://crrev.com/8def47f83bb6413eee451db95b15ceb96c24dba8/third_party/WebKit/common/typemaps.gni
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/8528c16f6bfdd4c106ebc95fdd5bf261eb81479d commit 8528c16f6bfdd4c106ebc95fdd5bf261eb81479d Author: Luciano Pacheco <lucmult@chromium.org> Date: Thu Dec 14 07:10:56 2017 Convert FrameMsg_ExtractSmartClipData to Mojo Convert and merge FrameHostMsg_SmartClipDataExtracted now using Mojo callback directly. Bug: 786836 Change-Id: If2925e9e019854558f8c97cf51e6b0d32b521345 Reviewed-on: https://chromium-review.googlesource.com/804804 Reviewed-by: Kinuko Yasuda <kinuko@chromium.org> Reviewed-by: Will Harris <wfh@chromium.org> Reviewed-by: Sam McNally <sammc@chromium.org> Commit-Queue: Luciano Pacheco <lucmult@chromium.org> Cr-Commit-Position: refs/heads/master@{#524017} [modify] https://crrev.com/8528c16f6bfdd4c106ebc95fdd5bf261eb81479d/content/browser/frame_host/render_frame_host_impl.cc [modify] https://crrev.com/8528c16f6bfdd4c106ebc95fdd5bf261eb81479d/content/browser/frame_host/render_frame_host_impl.h [modify] https://crrev.com/8528c16f6bfdd4c106ebc95fdd5bf261eb81479d/content/browser/web_contents/web_contents_android.cc [modify] https://crrev.com/8528c16f6bfdd4c106ebc95fdd5bf261eb81479d/content/common/frame.mojom [modify] https://crrev.com/8528c16f6bfdd4c106ebc95fdd5bf261eb81479d/content/common/frame_messages.h [modify] https://crrev.com/8528c16f6bfdd4c106ebc95fdd5bf261eb81479d/content/renderer/render_frame_impl.cc [modify] https://crrev.com/8528c16f6bfdd4c106ebc95fdd5bf261eb81479d/content/renderer/render_frame_impl.h
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/10d648365c9ba94b792b0204637940d748d63b37 commit 10d648365c9ba94b792b0204637940d748d63b37 Author: Austin Tankiang <austinct@google.com> Date: Wed Dec 20 02:43:55 2017 Convert 2 Frame IPC messages to mojo Convert FrameHostMsg_CancelInitialHistoryLoad and FrameHostMsg_UpdateEncoding from the old IPC system to mojo. Bug: 786836 Change-Id: I80c250d7b086b34a2709c70d89b9ecce8e07ec75 Reviewed-on: https://chromium-review.googlesource.com/826829 Reviewed-by: Sam McNally <sammc@chromium.org> Reviewed-by: Kinuko Yasuda <kinuko@chromium.org> Reviewed-by: Daniel Cheng <dcheng@chromium.org> Commit-Queue: Austin Tankiang <austinct@google.com> Cr-Commit-Position: refs/heads/master@{#525236} [modify] https://crrev.com/10d648365c9ba94b792b0204637940d748d63b37/content/browser/frame_host/render_frame_host_impl.cc [modify] https://crrev.com/10d648365c9ba94b792b0204637940d748d63b37/content/browser/frame_host/render_frame_host_impl.h [modify] https://crrev.com/10d648365c9ba94b792b0204637940d748d63b37/content/common/frame.mojom [modify] https://crrev.com/10d648365c9ba94b792b0204637940d748d63b37/content/common/frame_messages.h [modify] https://crrev.com/10d648365c9ba94b792b0204637940d748d63b37/content/renderer/render_frame_impl.cc [modify] https://crrev.com/10d648365c9ba94b792b0204637940d748d63b37/content/test/test_render_frame.cc
Comment 1 by sa...@chromium.org
, Nov 22 2017