PPAPI: implement GetSupportedPreviewSizes

ppapi/proxy/ppapi_messages.h

Index: ppapi/proxy/ppapi_messages.h
diff --git a/ppapi/proxy/ppapi_messages.h b/ppapi/proxy/ppapi_messages.h
index d23a0773e29f272f6f14258bd6d5d9269bb6c9da..74ba363961fbe5db4a80f3061ceb2b8dbd82f3c4 100644
--- a/ppapi/proxy/ppapi_messages.h
+++ b/ppapi/proxy/ppapi_messages.h
@@ -1541,7 +1541,23 @@ IPC_MESSAGE_CONTROL2(PpapiHostMsg_Graphics2D_ReadImageData,
                      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 */)
@@ -2243,6 +2259,8 @@ IPC_MESSAGE_CONTROL1(PpapiHostMsg_PDF_SetSelectedText,
 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)