Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(45)

Issue 1977643002: Generate param traits size methods for IPC files in chrome/ (and traits it depends on). (Closed)

Created:
4 years, 7 months ago by jam
Modified:
4 years, 7 months ago
CC:
android-webview-reviews_chromium.org, bondd+autofillwatch_chromium.org, browser-components-watch_chromium.org, chromium-apps-reviews_chromium.org, chromium-reviews, chromoting-reviews_chromium.org, csharrison+watch_chromium.org, darin-cc_chromium.org, eme-reviews_chromium.org, estade+watch_chromium.org, extensions-reviews_chromium.org, feature-media-reviews_chromium.org, grt+watch_chromium.org, jam, jdonnelly+autofillwatch_chromium.org, loading-reviews+metrics_chromium.org, markusheintz_, msramek+watch_chromium.org, piman+watch_chromium.org, posciak+watch_chromium.org, raymes+watch_chromium.org, rouslan+autofill_chromium.org, tfarina, tracing+reviews_chromium.org, vabr+watchlistautofill_chromium.org, wfh+watch_chromium.org
Base URL:
https://chromium.googlesource.com/chromium/src.git@master
Target Ref:
refs/pending/heads/master
Project:
chromium
Visibility:
Public.

Description

Generate param traits size methods for IPC files in chrome/ (and traits it depends on). This allows us to reuse these traits in Mojo if needed. I didn't add GetSize to every single trait in chrome/ and below; just enough to get the chrome message generator to compile with the size traits. This is a followup to r393058. Committed: https://crrev.com/3db6b6d9b24c3c0d44df4929eb8b44605d338cbf Cr-Commit-Position: refs/heads/master@{#393526}

Patch Set 1 #

Patch Set 2 : fix #

Total comments: 1
Unified diffs Side-by-side diffs Delta from patch set Stats (+411 lines, -12 lines) Patch
M android_webview/common/android_webview_message_generator.cc View 1 chunk +6 lines, -0 lines 0 comments Download
M chrome/chrome_common.gypi View 1 chunk +1 line, -0 lines 0 comments Download
M chrome/common/cast_messages.h View 2 chunks +2 lines, -0 lines 0 comments Download
M chrome/common/cast_messages.cc View 2 chunks +10 lines, -0 lines 1 comment Download
M chrome/common/common_message_generator.cc View 1 chunk +7 lines, -0 lines 0 comments Download
M chrome/common/common_param_traits.cc View 1 chunk +7 lines, -0 lines 0 comments Download
M chrome/common/content_settings_pattern_serializer.h View 1 chunk +5 lines, -0 lines 0 comments Download
M chrome/common/content_settings_pattern_serializer.cc View 1 chunk +7 lines, -0 lines 0 comments Download
M chrome/common/importer/profile_import_process_messages.cc View 1 chunk +6 lines, -0 lines 0 comments Download
M chrome/common/render_messages.h View 1 chunk +1 line, -0 lines 0 comments Download
M chrome/common/render_messages.cc View 1 chunk +6 lines, -0 lines 0 comments Download
M chrome/common/safe_browsing/ipc_protobuf_message_macros.h View 1 chunk +1 line, -0 lines 0 comments Download
M chrome/common/safe_browsing/ipc_protobuf_message_unittest.cc View 1 chunk +5 lines, -0 lines 0 comments Download
M chrome/common/safe_browsing/protobuf_message_param_traits.h View 1 chunk +6 lines, -0 lines 0 comments Download
A + chrome/common/safe_browsing/protobuf_message_size_macros.h View 2 chunks +10 lines, -12 lines 0 comments Download
M components/autofill/content/common/autofill_message_generator.cc View 1 chunk +8 lines, -0 lines 0 comments Download
M components/cdm/common/cdm_message_generator.cc View 1 chunk +6 lines, -0 lines 0 comments Download
M components/content_settings/content/common/content_settings_message_generator.cc View 1 chunk +6 lines, -0 lines 0 comments Download
M components/data_reduction_proxy/content/common/data_reduction_proxy_messages.cc View 1 chunk +6 lines, -0 lines 0 comments Download
M components/guest_view/common/guest_view_message_generator.cc View 1 chunk +6 lines, -0 lines 0 comments Download
M components/nacl/common/nacl_host_messages.cc View 1 chunk +6 lines, -0 lines 0 comments Download
M components/nacl/common/nacl_messages.cc View 1 chunk +6 lines, -0 lines 0 comments Download
M components/nacl/common/nacl_renderer_messages.cc View 1 chunk +6 lines, -0 lines 0 comments Download
M components/nacl/common/nacl_types_param_traits.cc View 1 chunk +7 lines, -0 lines 0 comments Download
M components/network_hints/common/network_hints_message_generator.cc View 1 chunk +6 lines, -0 lines 0 comments Download
M components/page_load_metrics/common/page_load_metrics_messages.cc View 1 chunk +6 lines, -0 lines 0 comments Download
M components/pdf/common/pdf_message_generator.cc View 1 chunk +6 lines, -0 lines 0 comments Download
M components/printing/common/print_messages.cc View 1 chunk +6 lines, -0 lines 0 comments Download
M components/startup_metric_utils/common/startup_metric_message_generator.cc View 1 chunk +6 lines, -0 lines 0 comments Download
M components/tracing/tracing_messages.cc View 1 chunk +6 lines, -0 lines 0 comments Download
M components/translate/content/common/data_file_cld_data_provider_messages.cc View 1 chunk +6 lines, -0 lines 0 comments Download
M components/translate/content/common/translate_messages.cc View 1 chunk +6 lines, -0 lines 0 comments Download
M components/visitedlink/common/visitedlink_message_generator.cc View 1 chunk +6 lines, -0 lines 0 comments Download
M extensions/common/extension_message_generator.cc View 1 chunk +6 lines, -0 lines 0 comments Download
M extensions/common/extension_messages.h View 7 chunks +7 lines, -0 lines 0 comments Download
M extensions/common/extension_messages.cc View 7 chunks +51 lines, -0 lines 0 comments Download
M extensions/common/permissions/api_permission.h View 1 chunk +3 lines, -0 lines 0 comments Download
M extensions/common/permissions/api_permission.cc View 1 chunk +2 lines, -0 lines 0 comments Download
M extensions/common/permissions/manifest_permission.h View 1 chunk +3 lines, -0 lines 0 comments Download
M extensions/common/permissions/manifest_permission.cc View 1 chunk +7 lines, -0 lines 0 comments Download
M extensions/common/permissions/set_disjunction_permission.h View 1 chunk +4 lines, -0 lines 0 comments Download
M extensions/common/permissions/settings_override_permission.h View 1 chunk +1 line, -0 lines 0 comments Download
M extensions/common/permissions/settings_override_permission.cc View 1 chunk +2 lines, -0 lines 0 comments Download
M gpu/ipc/common/gpu_message_generator.cc View 1 chunk +6 lines, -0 lines 0 comments Download
M gpu/ipc/common/gpu_param_traits.cc View 1 chunk +7 lines, -0 lines 0 comments Download
M ipc/ipc_message_utils.h View 1 chunk +1 line, -0 lines 0 comments Download
M ipc/ipc_message_utils.cc View 1 chunk +8 lines, -0 lines 0 comments Download
M media/gpu/ipc/common/media_message_generator.cc View 1 chunk +6 lines, -0 lines 0 comments Download
M media/gpu/ipc/common/media_messages.h View 1 chunk +1 line, -0 lines 0 comments Download
M media/gpu/ipc/common/media_messages.cc View 1 chunk +14 lines, -0 lines 0 comments Download
M ppapi/proxy/ppapi_messages.cc View 1 chunk +6 lines, -0 lines 0 comments Download
M ppapi/proxy/ppapi_param_traits.h View 5 chunks +5 lines, -0 lines 0 comments Download
M ppapi/proxy/ppapi_param_traits.cc View 5 chunks +33 lines, -0 lines 0 comments Download
M remoting/host/chromoting_messages.cc View 1 chunk +6 lines, -0 lines 0 comments Download
M remoting/host/chromoting_param_traits.h View 1 5 chunks +5 lines, -0 lines 0 comments Download
M remoting/host/chromoting_param_traits.cc View 1 5 chunks +36 lines, -0 lines 0 comments Download

Messages

Total messages: 17 (10 generated)
jam
4 years, 7 months ago (2016-05-12 23:28:23 UTC) #3
dcheng
LGTM with nits. https://codereview.chromium.org/1977643002/diff/20001/chrome/common/cast_messages.cc File chrome/common/cast_messages.cc (right): https://codereview.chromium.org/1977643002/diff/20001/chrome/common/cast_messages.cc#newcode11 chrome/common/cast_messages.cc:11: ParamTraits<uint64_t>::GetSize(s, p.SerializeForIPC()); Nit: does GetParamSize() work ...
4 years, 7 months ago (2016-05-13 03:23:10 UTC) #5
Andrew Hayden (chromium.org)
components/translate/* LGTM
4 years, 7 months ago (2016-05-13 08:57:26 UTC) #7
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1977643002/20001 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/1977643002/20001
4 years, 7 months ago (2016-05-13 15:03:01 UTC) #9
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1977643002/20001 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/1977643002/20001
4 years, 7 months ago (2016-05-13 15:05:51 UTC) #13
commit-bot: I haz the power
Committed patchset #2 (id:20001)
4 years, 7 months ago (2016-05-13 15:10:09 UTC) #15
commit-bot: I haz the power
4 years, 7 months ago (2016-05-13 15:11:06 UTC) #17
Message was sent while issue was closed.
Patchset 2 (id:??) landed as
https://crrev.com/3db6b6d9b24c3c0d44df4929eb8b44605d338cbf
Cr-Commit-Position: refs/heads/master@{#393526}

Powered by Google App Engine
This is Rietveld 408576698