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

Unified Diff: remoting/client/plugin/chromoting_instance.h

Issue 3078005: Rename ChromotingPlugin -> ChromotingInstance to be more consistent with... (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: '' Created 10 years, 4 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
« no previous file with comments | « no previous file | remoting/client/plugin/chromoting_instance.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: remoting/client/plugin/chromoting_instance.h
===================================================================
--- remoting/client/plugin/chromoting_instance.h (revision 0)
+++ remoting/client/plugin/chromoting_instance.h (revision 0)
@@ -0,0 +1,84 @@
+// Copyright (c) 2010 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.
+
+// TODO(ajwong): We need to come up with a better description of the
+// responsibilities for each thread.
+
+#ifndef REMOTING_CLIENT_PLUGIN_CHROMOTING_INSTANCE_H_
+#define REMOTING_CLIENT_PLUGIN_CHROMOTING_INSTANCE_H_
+
+#include <string>
+
+#include "base/at_exit.h"
+#include "base/scoped_ptr.h"
+#include "remoting/client/client_context.h"
+#include "remoting/client/host_connection.h"
+#include "testing/gtest/include/gtest/gtest_prod.h"
+#include "third_party/ppapi/c/pp_event.h"
+#include "third_party/ppapi/c/pp_instance.h"
+#include "third_party/ppapi/c/pp_rect.h"
+#include "third_party/ppapi/c/pp_resource.h"
+#include "third_party/ppapi/cpp/instance.h"
+#include "third_party/ppapi/cpp/device_context_2d.h"
+#include "third_party/ppapi/cpp/var.h"
+
+class MessageLoop;
+
+namespace base {
+class Thread;
+} // namespace base
+
+namespace pp {
+class Module;
+} // namespace pp
+
+namespace remoting {
+
+class ChromotingClient;
+class ClientContext;
+class HostConnection;
+class InputHandler;
+class JingleThread;
+class PepperView;
+
+class ChromotingInstance : public pp::Instance {
+ public:
+ // The mimetype for which this plugin is registered.
+ static const char *kMimeType;
+
+ explicit ChromotingInstance(PP_Instance instance);
+ virtual ~ChromotingInstance();
+
+ virtual bool Init(uint32_t argc, const char* argn[], const char* argv[]);
+ virtual void Connect(const ClientConfig& config);
+ virtual bool HandleEvent(const PP_Event& event);
+ virtual pp::Var GetInstanceObject();
+ virtual void ViewChanged(const pp::Rect& position, const pp::Rect& clip);
+
+ virtual bool CurrentlyOnPluginThread() const;
+
+ private:
+ FRIEND_TEST(ChromotingInstanceTest, TestCaseSetup);
+
+ // Since we're an internal plugin, we can just grab the message loop during
+ // init to figure out which thread we're on. This should only be used to
+ // sanity check which thread we're executing on. Do not post task here!
+ // Instead, use PPB_Core:CallOnMainThread() in the pepper api.
+ //
+ // TODO(ajwong): Think if there is a better way to safeguard this.
+ MessageLoop* pepper_main_loop_dont_post_to_me_;
+
+ ClientContext context_;
+ scoped_ptr<HostConnection> host_connection_;
+ scoped_ptr<PepperView> view_;
+ scoped_ptr<InputHandler> input_handler_;
+ scoped_ptr<ChromotingClient> client_;
+ pp::Var instance_object_; // JavaScript interface to control this instance.
+
+ DISALLOW_COPY_AND_ASSIGN(ChromotingInstance);
+};
+
+} // namespace remoting
+
+#endif // REMOTING_CLIENT_PLUGIN_CHROMOTING_INSTANCE_H_
Property changes on: remoting/client/plugin/chromoting_instance.h
___________________________________________________________________
Added: svn:eol-style
+ LF
« no previous file with comments | « no previous file | remoting/client/plugin/chromoting_instance.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698