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

Unified Diff: chrome/browser/ui/webui/chromeos/login/signin_screen_handler.cc

Issue 2505403003: TouchView Mojom (Closed)
Patch Set: Minor param rename Created 4 years 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: chrome/browser/ui/webui/chromeos/login/signin_screen_handler.cc
diff --git a/chrome/browser/ui/webui/chromeos/login/signin_screen_handler.cc b/chrome/browser/ui/webui/chromeos/login/signin_screen_handler.cc
index 9efae546cc454471732adae2f27b24c30607005e..d86e7b57de4b864e011afd5a4460076d48a448a4 100644
--- a/chrome/browser/ui/webui/chromeos/login/signin_screen_handler.cc
+++ b/chrome/browser/ui/webui/chromeos/login/signin_screen_handler.cc
@@ -92,7 +92,10 @@
#include "components/version_info/version_info.h"
#include "content/public/browser/render_frame_host.h"
#include "content/public/browser/web_contents.h"
+#include "content/public/common/service_manager_connection.h"
+#include "content/public/common/service_names.mojom.h"
#include "google_apis/gaia/gaia_auth_util.h"
+#include "services/service_manager/public/cpp/connector.h"
#include "third_party/cros_system_api/dbus/service_constants.h"
#include "ui/base/ime/chromeos/ime_keyboard.h"
#include "ui/base/ime/chromeos/input_method_descriptor.h"
@@ -265,6 +268,7 @@ SigninScreenHandler::SigninScreenHandler(
->CapsLockIsEnabled()),
proxy_auth_dialog_reload_times_(kMaxGaiaReloadForProxyAuthDialog),
gaia_screen_handler_(gaia_screen_handler),
+ touch_view_binding_(this),
histogram_helper_(new ErrorScreensHistogramHelper("Signin")),
weak_factory_(this) {
DCHECK(network_state_informer_.get());
@@ -291,12 +295,17 @@ SigninScreenHandler::SigninScreenHandler(
if (keyboard)
keyboard->AddObserver(this);
+ service_manager::Connector* connector =
+ content::ServiceManagerConnection::GetForProcess()->GetConnector();
if (!chrome::IsRunningInMash()) {
- max_mode_delegate_.reset(new TouchViewControllerDelegate());
- max_mode_delegate_->AddObserver(this);
+ connector->ConnectToInterface(content::mojom::kBrowserServiceName,
+ &touch_view_manager_ptr_);
} else {
- NOTIMPLEMENTED();
+ connector->ConnectToInterface("ash", &touch_view_manager_ptr_);
}
+
+ touch_view_manager_ptr_->AddObserver(
+ touch_view_binding_.CreateInterfacePtrAndBind());
}
SigninScreenHandler::~SigninScreenHandler() {
@@ -313,10 +322,6 @@ SigninScreenHandler::~SigninScreenHandler() {
if (delegate_)
delegate_->SetWebUIHandler(nullptr);
network_state_informer_->RemoveObserver(this);
- if (max_mode_delegate_) {
- max_mode_delegate_->RemoveObserver(this);
- max_mode_delegate_.reset(nullptr);
- }
proximity_auth::ScreenlockBridge::Get()->SetLockHandler(nullptr);
proximity_auth::ScreenlockBridge::Get()->SetFocusedUser(EmptyAccountId());
}
@@ -1066,12 +1071,9 @@ void SigninScreenHandler::SuspendDone(const base::TimeDelta& sleep_duration) {
}
}
-void SigninScreenHandler::OnMaximizeModeStarted() {
- CallJS("login.AccountPickerScreen.setTouchViewState", true);
-}
-
-void SigninScreenHandler::OnMaximizeModeEnded() {
- CallJS("login.AccountPickerScreen.setTouchViewState", false);
+void SigninScreenHandler::OnTouchViewToggled(bool enabled) {
+ touch_view_enabled_ = enabled;
+ CallJS("login.AccountPickerScreen.setTouchViewState", enabled);
}
bool SigninScreenHandler::ShouldLoadGaia() const {
@@ -1423,10 +1425,7 @@ void SigninScreenHandler::HandleLaunchArcKioskApp(
}
void SigninScreenHandler::HandleGetTouchViewState() {
- if (max_mode_delegate_) {
- CallJS("login.AccountPickerScreen.setTouchViewState",
- max_mode_delegate_->IsMaximizeModeEnabled());
- }
+ CallJS("login.AccountPickerScreen.setTouchViewState", touch_view_enabled_);
}
void SigninScreenHandler::HandleLogRemoveUserWarningShown() {

Powered by Google App Engine
This is Rietveld 408576698