Refactor Pnacl coordinator

ppapi/native_client/src/trusted/plugin/pnacl_translate_thread.h

Index: ppapi/native_client/src/trusted/plugin/pnacl_translate_thread.h
diff --git a/ppapi/native_client/src/trusted/plugin/pnacl_translate_thread.h b/ppapi/native_client/src/trusted/plugin/pnacl_translate_thread.h
new file mode 100644
index 0000000000000000000000000000000000000000..06691f3a2f3e2d9f9971fd10db2a4de801e969fc
--- /dev/null
+++ b/ppapi/native_client/src/trusted/plugin/pnacl_translate_thread.h
@@ -0,0 +1,80 @@
+// 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.
+
+#ifndef NATIVE_CLIENT_SRC_TRUSTED_PLUGIN_PNACL_TRANSLATE_THREAD_H_
+#define NATIVE_CLIENT_SRC_TRUSTED_PLUGIN_PNACL_TRANSLATE_THREAD_H_
+
+#include "native_client/src/include/nacl_scoped_ptr.h"
+#include "native_client/src/include/nacl_string.h"
+#include "native_client/src/shared/platform/nacl_threads.h"
+#include "native_client/src/shared/platform/nacl_sync_checked.h"
+
+#include "ppapi/cpp/completion_callback.h"
+
+namespace nacl {
+class DescWrapper;
+}
+
+
+namespace plugin {
+
+class ErrorInfo;
+class LocalTempFile;
+class Manifest;
+class NaClSubprocess;
+class Plugin;
+class PnaclResources;
+
+class PnaclTranslateThread {
+ public:
+  PnaclTranslateThread();
+  virtual ~PnaclTranslateThread();
+  // TODO(jvoung/dschuff): handle surfaway issues when coordinator/plugin
+  // goes away. This data may have to be refcounted not touched in that case.
+  void RunTranslate(pp::CompletionCallback finish_callback,
+                    const Manifest* manifest,
+                    const Manifest* ld_manifest,
+                    LocalTempFile* obj_file,
+                    LocalTempFile* nexe_file,
+                    nacl::DescWrapper* pexe_wrapper,
+                    ErrorInfo* error_info,
+                    PnaclResources* resources,
+                    Plugin* plugin);
+  bool SubprocessesShouldDie();
+  // Signal the translate thread and subprocesses that they should stop.
+  void SetSubprocessesShouldDie(bool subprocesses_should_die);
+
+ private:

[sehr (please use chromium), 2012/05/16 17:19:56]

DISALLOW_COPY_AND_ASSIGN(PnaclTranslateThread);

[Derek Schuff, 2012/05/16 17:48:09]

Done.

+  // Starts an individual llc or ld subprocess used for translation.
+  NaClSubprocess* StartSubprocess(const nacl::string& url,
+                                  const Manifest* manifest);
+  // Creates a helper thread to allow translations to be
+  // invoked via SRPC.  This is the helper thread function for translation.
+  static void WINAPI DoTranslateThread(void* arg);
+  // Signal that Pnacl translation failed, from the translation thread only.
+  void TranslateFailed(const nacl::string& error_string);
+  // Returns true if a the translate thread and subprocesses should stop.
+
+  // Callback to run when tasks are completed or an error has occurred.
+  pp::CompletionCallback report_translate_finished_;
+  // True if the translation thread and related subprocesses should exit.
+  bool subprocesses_should_die_;
+  // Used to guard and publish subprocesses_should_die_.
+  struct NaClMutex subprocess_mu_;
+
+  nacl::scoped_ptr<NaClThread> translate_thread_;
+
+  // Data about the translation files, owned by the coordinator
+  const Manifest* manifest_;
+  const Manifest* ld_manifest_;
+  LocalTempFile* obj_file_;
+  LocalTempFile* nexe_file_;
+  nacl::DescWrapper* pexe_wrapper_;
+  ErrorInfo *error_info_;
+  PnaclResources* resources_;
+  Plugin* plugin_;
+};
+
+}
+#endif // NATIVE_CLIENT_SRC_TRUSTED_PLUGIN_PNACL_TRANSLATE_THREAD_H_