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

Unified Diff: ppapi/native_client/src/trusted/plugin/plugin.cc

Issue 10827456: Remove PPB_Scrollbar_Dev and PPB_Widget_Dev from native client (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Also remove PPB_PDF and associated tests Created 8 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 | « ppapi/native_client/src/trusted/plugin/plugin.h ('k') | ppapi/native_client/tests/examples.html » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: ppapi/native_client/src/trusted/plugin/plugin.cc
diff --git a/ppapi/native_client/src/trusted/plugin/plugin.cc b/ppapi/native_client/src/trusted/plugin/plugin.cc
index 957fd91c3371e04920e5d358271eb2be0c9475db..e811e758bf176da64211b33051bdf7c8bd4f1c26 100644
--- a/ppapi/native_client/src/trusted/plugin/plugin.cc
+++ b/ppapi/native_client/src/trusted/plugin/plugin.cc
@@ -46,9 +46,7 @@
#include "ppapi/c/dev/ppb_console_dev.h"
#include "ppapi/c/dev/ppp_find_dev.h"
#include "ppapi/c/dev/ppp_printing_dev.h"
-#include "ppapi/c/dev/ppp_scrollbar_dev.h"
#include "ppapi/c/dev/ppp_selection_dev.h"
-#include "ppapi/c/dev/ppp_widget_dev.h"
#include "ppapi/c/dev/ppp_zoom_dev.h"
#include "ppapi/c/pp_errors.h"
#include "ppapi/c/ppb_var.h"
@@ -59,11 +57,9 @@
#include "ppapi/c/private/ppb_uma_private.h"
#include "ppapi/cpp/dev/find_dev.h"
#include "ppapi/cpp/dev/printing_dev.h"
-#include "ppapi/cpp/dev/scrollbar_dev.h"
#include "ppapi/cpp/dev/selection_dev.h"
#include "ppapi/cpp/dev/text_input_dev.h"
#include "ppapi/cpp/dev/url_util_dev.h"
-#include "ppapi/cpp/dev/widget_client_dev.h"
#include "ppapi/cpp/dev/zoom_dev.h"
#include "ppapi/cpp/image_data.h"
#include "ppapi/cpp/input_event.h"
@@ -403,54 +399,6 @@ class SelectionAdapter : public pp::Selection_Dev {
};
-// Derive a class from pp::WidgetClient_Dev to forward PPP_Widget_Dev
-// and PPP_Scrollbar_Dev calls to the plugin.
-class WidgetClientAdapter : public pp::WidgetClient_Dev {
- public:
- explicit WidgetClientAdapter(Plugin* plugin)
- : pp::WidgetClient_Dev(plugin),
- plugin_(plugin) {
- BrowserPpp* proxy = plugin_->ppapi_proxy();
- CHECK(proxy != NULL);
- ppp_widget_ = static_cast<const PPP_Widget_Dev*>(
- proxy->GetPluginInterface(PPP_WIDGET_DEV_INTERFACE));
- ppp_scrollbar_ = static_cast<const PPP_Scrollbar_Dev*>(
- proxy->GetPluginInterface(PPP_SCROLLBAR_DEV_INTERFACE));
- }
-
- void InvalidateWidget(pp::Widget_Dev widget, const pp::Rect& dirty_rect) {
- if (ppp_widget_ != NULL) {
- ppp_widget_->Invalidate(plugin_->pp_instance(),
- widget.pp_resource(),
- &dirty_rect.pp_rect());
- }
- }
-
- void ScrollbarValueChanged(pp::Scrollbar_Dev scrollbar, uint32_t value) {
- if (ppp_scrollbar_ != NULL) {
- ppp_scrollbar_->ValueChanged(plugin_->pp_instance(),
- scrollbar.pp_resource(),
- value);
- }
- }
-
- void ScrollbarOverlayChanged(pp::Scrollbar_Dev scrollbar, bool overlay) {
- if (ppp_scrollbar_ != NULL) {
- ppp_scrollbar_->OverlayChanged(plugin_->pp_instance(),
- scrollbar.pp_resource(),
- PP_FromBool(overlay));
- }
- }
-
- private:
- Plugin* plugin_;
- const PPP_Widget_Dev* ppp_widget_;
- const PPP_Scrollbar_Dev* ppp_scrollbar_;
-
- NACL_DISALLOW_COPY_AND_ASSIGN(WidgetClientAdapter);
-};
-
-
// Derive a class from pp::Zoom_Dev to forward PPP_Zoom_Dev calls to
// the plugin.
class ZoomAdapter : public pp::Zoom_Dev {
@@ -1326,7 +1274,6 @@ bool Plugin::StartProxiedExecution(NaClSrpcChannel* srpc_channel,
mouse_lock_adapter_.reset(new MouseLockAdapter(this));
printing_adapter_.reset(new PrintingAdapter(this));
selection_adapter_.reset(new SelectionAdapter(this));
- widget_client_adapter_.reset(new WidgetClientAdapter(this));
zoom_adapter_.reset(new ZoomAdapter(this));
// Replay missed events.
« no previous file with comments | « ppapi/native_client/src/trusted/plugin/plugin.h ('k') | ppapi/native_client/tests/examples.html » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698