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

Unified Diff: remoting/client/plugin/chromoting_plugin.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 | « remoting/client/plugin/chromoting_instance.cc ('k') | remoting/client/plugin/chromoting_plugin.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: remoting/client/plugin/chromoting_plugin.h
===================================================================
--- remoting/client/plugin/chromoting_plugin.h (revision 55094)
+++ remoting/client/plugin/chromoting_plugin.h (working copy)
@@ -1,85 +0,0 @@
-// 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_PLUGIN_H_
-#define REMOTING_CLIENT_PLUGIN_CHROMOTING_PLUGIN_H_
-
-#include <string>
-
-#include "base/at_exit.h"
-#include "base/gtest_prod_util.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 ChromotingPlugin : public pp::Instance {
- public:
- // The mimetype for which this plugin is registered.
- static const char *kMimeType;
-
- ChromotingPlugin(PP_Instance instance);
- virtual ~ChromotingPlugin();
-
- 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_ALL_PREFIXES(ChromotingPluginTest, 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(ChromotingPlugin);
-};
-
-} // namespace remoting
-
-#endif // REMOTING_CLIENT_PLUGIN_CHROMOTING_PLUGIN_H_
« no previous file with comments | « remoting/client/plugin/chromoting_instance.cc ('k') | remoting/client/plugin/chromoting_plugin.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698