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

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

Issue 799233004: Add touch events to the protocol, the stub layer, and to the client plugin. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: removed todo that no longer applies Created 5 years, 10 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
Index: remoting/client/plugin/chromoting_instance.cc
diff --git a/remoting/client/plugin/chromoting_instance.cc b/remoting/client/plugin/chromoting_instance.cc
index ff8a06a1824a7631d82e06dffaeedb2f654d7092..047b270edc203463a327d82ede543c3c7168c29b 100644
--- a/remoting/client/plugin/chromoting_instance.cc
+++ b/remoting/client/plugin/chromoting_instance.cc
@@ -187,7 +187,8 @@ ChromotingInstance::ChromotingInstance(PP_Instance pp_instance)
plugin_task_runner_(new PluginThreadTaskRunner(&plugin_thread_delegate_)),
context_(plugin_task_runner_.get()),
input_tracker_(&mouse_input_filter_),
- key_mapper_(&input_tracker_),
+ touch_input_scaler_(&input_tracker_),
+ key_mapper_(&touch_input_scaler_),
cursor_setter_(this),
empty_cursor_filter_(&cursor_setter_),
text_input_controller_(this),
@@ -350,6 +351,8 @@ void ChromotingInstance::HandleMessage(const pp::Var& message) {
HandleSendMouseInputWhenUnfocused();
} else if (method == "delegateLargeCursors") {
HandleDelegateLargeCursors();
+ } else if (method == "enableTouchEvents") {
+ HandleEnableTouchEvents();
}
}
@@ -368,8 +371,10 @@ void ChromotingInstance::DidChangeView(const pp::View& view) {
DCHECK(plugin_task_runner_->BelongsToCurrentThread());
plugin_view_ = view;
- mouse_input_filter_.set_input_size(
+ webrtc::DesktopSize size(
webrtc::DesktopSize(view.GetRect().width(), view.GetRect().height()));
+ mouse_input_filter_.set_input_size(size);
+ touch_input_scaler_.set_input_size(size);
if (video_renderer_)
video_renderer_->OnViewChanged(view);
@@ -401,8 +406,9 @@ void ChromotingInstance::OnVideoFirstFrameReceived() {
}
void ChromotingInstance::OnVideoSize(const webrtc::DesktopSize& size,
- const webrtc::DesktopVector& dpi) {
+ const webrtc::DesktopVector& dpi) {
mouse_input_filter_.set_output_size(size);
+ touch_input_scaler_.set_output_size(size);
scoped_ptr<base::DictionaryValue> data(new base::DictionaryValue());
data->SetInteger("width", size.width());
@@ -664,9 +670,12 @@ void ChromotingInstance::HandleConnect(const base::DictionaryValue& data) {
// Connect the input pipeline to the protocol stub & initialize components.
mouse_input_filter_.set_input_stub(client_->input_stub());
+ touch_input_scaler_.set_input_stub(client_->input_stub());
if (!plugin_view_.is_null()) {
- mouse_input_filter_.set_input_size(webrtc::DesktopSize(
- plugin_view_.GetRect().width(), plugin_view_.GetRect().height()));
+ webrtc::DesktopSize size(plugin_view_.GetRect().width(),
+ plugin_view_.GetRect().height());
+ mouse_input_filter_.set_input_size(size);
+ touch_input_scaler_.set_input_size(size);
}
// Setup the signal strategy.
@@ -944,6 +953,10 @@ void ChromotingInstance::HandleDelegateLargeCursors() {
cursor_setter_.set_delegate_stub(this);
}
+void ChromotingInstance::HandleEnableTouchEvents() {
+ RequestInputEvents(PP_INPUTEVENT_CLASS_TOUCH);
+}
+
void ChromotingInstance::Disconnect() {
DCHECK(plugin_task_runner_->BelongsToCurrentThread());

Powered by Google App Engine
This is Rietveld 408576698