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

Unified Diff: content/common/frame_messages.h

Issue 1619363002: Add compile time checks against longs being used in IPC structs on 32 bit Android. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: one more per Dmitry Created 4 years, 11 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: content/common/frame_messages.h
diff --git a/content/common/frame_messages.h b/content/common/frame_messages.h
index 6e9f362f6d5604430f3fb1a417adbd83f871dfba..81ef777f3c81eb09a64ead007a79ca50e3dcb942 100644
--- a/content/common/frame_messages.h
+++ b/content/common/frame_messages.h
@@ -728,7 +728,7 @@ IPC_MESSAGE_ROUTED0(FrameMsg_DeleteProxy)
// Request the text surrounding the selection with a |max_length|. The response
// will be sent via FrameHostMsg_TextSurroundingSelectionResponse.
IPC_MESSAGE_ROUTED1(FrameMsg_TextSurroundingSelectionRequest,
- size_t /* max_length */)
+ uint32_t /* max_length */)
// Tells the renderer to insert a link to the specified stylesheet. This is
// needed to support navigation transitions.
@@ -1300,8 +1300,8 @@ IPC_MESSAGE_ROUTED1(FrameHostMsg_DidChangeThemeColor,
// |endOffset| are the offsets of the selection in the returned |content|.
IPC_MESSAGE_ROUTED3(FrameHostMsg_TextSurroundingSelectionResponse,
base::string16, /* content */
- size_t, /* startOffset */
- size_t /* endOffset */)
+ uint32_t, /* startOffset */
+ uint32_t/* endOffset */)
// Register a new handler for URL requests with the given scheme.
IPC_MESSAGE_ROUTED4(FrameHostMsg_RegisterProtocolHandler,

Powered by Google App Engine
This is Rietveld 408576698