PPAPI: implement GetSupportedPreviewSizes

ppapi/proxy/ppapi_messages.h

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 // Multiply-included message header, no traditional include guard.
 #include <map>
 #include <string>
 #include <vector>
 
 #include "base/basictypes.h"
@@ skipping 1523 matching lines @@
 // Graphics2D, plugin -> host -> plugin
 IPC_MESSAGE_CONTROL1(PpapiHostMsg_Graphics2D_Flush,
                      std::vector<ui::LatencyInfo> /* latency_info */)
 IPC_MESSAGE_CONTROL0(PpapiPluginMsg_Graphics2D_FlushAck)
 
 IPC_MESSAGE_CONTROL2(PpapiHostMsg_Graphics2D_ReadImageData,
                      PP_Resource /* image */,
                      PP_Point /* top_left */)
 IPC_MESSAGE_CONTROL0(PpapiPluginMsg_Graphics2D_ReadImageDataAck)
 
-// IsolatedFileSystem
+// ImageCapture ----------------------------------------------------------------
+// ImageCapture_Private, plugin -> host
+IPC_MESSAGE_CONTROL0(PpapiHostMsg_ImageCapture_Create)
+IPC_MESSAGE_CONTROL0(PpapiHostMsg_ImageCapture_Close)
+
+// ImageCapture_Private, plugin -> host -> plugin
+IPC_MESSAGE_CONTROL1(PpapiHostMsg_ImageCapture_Open,
+                     std::string /* camera_source_id */)
+IPC_MESSAGE_CONTROL0(PpapiPluginMsg_ImageCapture_OpenReply)
+
+IPC_MESSAGE_CONTROL0(PpapiPluginMsg_ImageCapture_GetSupportedPreviewSizes)

[dmichael (off chromium), 2015/02/10 16:06:59]

nit: This should be PpapiHostMsg_...
sorry I missed it before.

[Justin Chuang, 2015/02/10 17:44:09]

Done. Sorry I missed it.

+IPC_MESSAGE_CONTROL1(PpapiPluginMsg_ImageCapture_GetSupportedPreviewSizesReply,
+                     std::vector<PP_Size> /* preview_sizes */)
+
+// ImageCapture_Private, host -> plugin

[dmichael (off chromium), 2015/02/09 22:27:00]

This and the above "plugin -> host" message aren't really accurate, since you
don't have the requests and replies ordered that way. Please remove those
comments (the message ordering is fine).

[Justin Chuang, 2015/02/10 16:01:03]

Removed the comments. 

But I don't really understand. I think the requests and replies are indeed
'plugin -> host -> plugin' as in above comments?

[dmichael (off chromium), 2015/02/10 16:06:59]

I see, I missed the comment on 1549. This last comment threw me off. I still
think the comments were probably more confusing than helpful, so thanks for
removing them.

+
+// IsolatedFileSystem ----------------------------------------------------------
 IPC_MESSAGE_CONTROL0(PpapiHostMsg_IsolatedFileSystem_Create)
 IPC_MESSAGE_CONTROL1(PpapiHostMsg_IsolatedFileSystem_BrowserOpen,
                      PP_IsolatedFileSystemType_Private /* type */)
 IPC_MESSAGE_CONTROL1(PpapiPluginMsg_IsolatedFileSystem_BrowserOpenReply,
                      std::string /* fsid */)
 
 // MediaStream -----------------------------------------------------------------
 IPC_MESSAGE_CONTROL1(PpapiPluginMsg_MediaStreamAudioTrack_CreateFromPendingHost,
                      std::string /* track_id */)
 IPC_MESSAGE_CONTROL1(
@@ skipping 681 matching lines @@
                      PP_ImageDataDesc /* image_data_desc */)
 
 // Called by the plugin when its selection changes.
 IPC_MESSAGE_CONTROL1(PpapiHostMsg_PDF_SetSelectedText,
                      base::string16 /* selected_text */)
 
 // Called by the plugin to set the link under the cursor.
 IPC_MESSAGE_CONTROL1(PpapiHostMsg_PDF_SetLinkUnderCursor,
                      std::string /* url */)
 
+// VideoCapture ----------------------------------------------------------------
+
 // VideoCapture_Dev, plugin -> host
 IPC_MESSAGE_CONTROL0(PpapiHostMsg_VideoCapture_Create)
 IPC_MESSAGE_CONTROL0(PpapiHostMsg_VideoCapture_StartCapture)
 IPC_MESSAGE_CONTROL1(PpapiHostMsg_VideoCapture_ReuseBuffer,
                      uint32_t /* buffer */)
 IPC_MESSAGE_CONTROL0(PpapiHostMsg_VideoCapture_StopCapture)
 IPC_MESSAGE_CONTROL0(PpapiHostMsg_VideoCapture_Close)
 
 // VideoCapture_Dev, plugin -> host -> plugin
 IPC_MESSAGE_CONTROL3(PpapiHostMsg_VideoCapture_Open,
@@ skipping 20 matching lines @@
 IPC_MESSAGE_CONTROL1(PpapiHostMsg_Talk_RequestPermission,
                      PP_TalkPermission /* permission */)
 IPC_MESSAGE_CONTROL0(PpapiPluginMsg_Talk_RequestPermissionReply)
 IPC_MESSAGE_CONTROL0(PpapiHostMsg_Talk_StartRemoting)
 IPC_MESSAGE_CONTROL0(PpapiPluginMsg_Talk_StartRemotingReply)
 IPC_MESSAGE_CONTROL0(PpapiHostMsg_Talk_StopRemoting)
 IPC_MESSAGE_CONTROL0(PpapiPluginMsg_Talk_StopRemotingReply)
 IPC_MESSAGE_CONTROL1(PpapiPluginMsg_Talk_NotifyEvent, PP_TalkEvent /* event */)
 
 #endif  // !defined(OS_NACL) && !defined(NACL_WIN64)