PPAPI/NaCl: Move handle extraction code to ppapi/proxy

ppapi/proxy/handle_converter.h

-// Copyright (c) 2012 The Chromium Authors. All rights reserved.
+// Copyright (c) 2013 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.
 
-#ifndef PPAPI_PROXY_SERIALIZED_FLASH_MENU_H_
-#define PPAPI_PROXY_SERIALIZED_FLASH_MENU_H_
+#ifndef PPAPI_PROXY_HANDLE_CONVERTER_H_
+#define PPAPI_PROXY_HANDLE_CONVERTER_H_
 
-#include <string>
+#include <map>
 #include <vector>
 
 #include "base/basictypes.h"
+#include "base/memory/scoped_ptr.h"
 #include "ppapi/proxy/ppapi_proxy_export.h"
 
-class PickleIterator;
-
-struct PP_Flash_Menu;
-
 namespace IPC {
 class Message;
 }
 
 namespace ppapi {
 namespace proxy {
 
-class PPAPI_PROXY_EXPORT SerializedFlashMenu {
+class SerializedHandle;
+
+class PPAPI_PROXY_EXPORT HandleConverter {
  public:
-  SerializedFlashMenu();
-  ~SerializedFlashMenu();
+  HandleConverter();
 
-  bool SetPPMenu(const PP_Flash_Menu* menu);
+  // Convert the native handles in |msg| to NaCl style.
+  // In some cases (e.g., Windows), we need to re-write the contents of the
+  // message; in those cases, |new_msg_ptr| will be set to the new message.
+  // If |msg| is already in a good form for NaCl, |new_msg_ptr| is left NULL.
+  // See the explanation in the body of the method.
+  //
+  // In either case, all the handles in |msg| are extracted into |handles| so
+  // that they can be converted to NaClDesc handles.
+  // See chrome/nacl/nacl_ipc_adapter.cc for where this gets used.
+  bool ConvertNativeHandlesToPosix(const IPC::Message& msg,
+                                   std::vector<SerializedHandle>* handles,
+                                   scoped_ptr<IPC::Message>* new_msg_ptr);
 
-  const PP_Flash_Menu* pp_menu() const { return pp_menu_; }
-
-  void WriteToMessage(IPC::Message* m) const;
-  bool ReadFromMessage(const IPC::Message* m, PickleIterator* iter);
+  // |SendingSyncMessage| allows HandleConverter to know what sync messages are

[bbudge, 2013/01/16 17:47:28]

Method name doesn't match comment

+  // being sent, so that it can associate reply messages with their type.
+  //
+  // Users of HandleConverter must call this when they send a synchronous
+  // message, otherwise HandleConverter won't be able to convert handles in
+  // replies.
+  void RegisterSyncMessageForReply(const IPC::Message& msg);
 
  private:
-  const PP_Flash_Menu* pp_menu_;
-  bool own_menu_;
-  DISALLOW_COPY_AND_ASSIGN(SerializedFlashMenu);
+  // When we send a synchronous message (from untrusted to trusted), we store
+  // its type here, so that later we can associate the reply with its type
+  // and potentially translate handles in the message.
+  typedef std::map<int, uint32> PendingSyncMsgMap;
+  PendingSyncMsgMap pending_sync_msgs_;
+
+  DISALLOW_COPY_AND_ASSIGN(HandleConverter);
 };
 
 }  // namespace proxy
 }  // namespace ppapi
 
-#endif  // PPAPI_PROXY_SERIALIZED_FLASH_MENU_H_
+#endif  // PPAPI_PROXY_HANDLE_CONVERTER_H_