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

Unified Diff: services/ui/ime/ime_server_impl.cc

Issue 2626983003: IME for Mus: Send TextInputClient information to IMEDriver. (Closed)
Patch Set: fix mojo dependencies. Created 3 years, 11 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 | « services/ui/ime/ime_server_impl.h ('k') | services/ui/ime/ime_unittest.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: services/ui/ime/ime_server_impl.cc
diff --git a/services/ui/ime/ime_server_impl.cc b/services/ui/ime/ime_server_impl.cc
index 0731d02a996b1a8e04aa8bd5603db25b9430b472..a5a10d37c80965690f171e765590ba0f12382343 100644
--- a/services/ui/ime/ime_server_impl.cc
+++ b/services/ui/ime/ime_server_impl.cc
@@ -4,6 +4,7 @@
#include "services/ui/ime/ime_server_impl.h"
+#include "base/memory/ptr_util.h"
#include "services/catalog/public/interfaces/constants.mojom.h"
#include "services/service_manager/public/cpp/connector.h"
#include "services/ui/ime/ime_registrar_impl.h"
@@ -46,26 +47,20 @@ void IMEServerImpl::OnDriverChanged(mojom::IMEDriverPtr driver) {
driver_ = std::move(driver);
while (!pending_requests_.empty()) {
- driver_->StartSession(current_id_++,
- std::move(pending_requests_.front().first),
- std::move(pending_requests_.front().second));
+ driver_->StartSession(current_id_++, std::move(pending_requests_.front()));
pending_requests_.pop();
}
}
-void IMEServerImpl::StartSession(
- mojom::TextInputClientPtr client,
- mojom::InputMethodRequest input_method_request) {
+void IMEServerImpl::StartSession(mojom::StartSessionDetailsPtr details) {
if (driver_.get()) {
// TODO(moshayedi): crbug.com/634431. This will forward all calls from
// clients to the driver as they are. We may need to check |caret_bounds|
// parameter of InputMethod::OnCaretBoundsChanged() here and limit them to
// client's focused window.
- driver_->StartSession(current_id_++, std::move(client),
- std::move(input_method_request));
+ driver_->StartSession(current_id_++, std::move(details));
} else {
- pending_requests_.push(
- std::make_pair(std::move(client), std::move(input_method_request)));
+ pending_requests_.push(std::move(details));
}
}
« no previous file with comments | « services/ui/ime/ime_server_impl.h ('k') | services/ui/ime/ime_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698