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

Unified Diff: ash/wm/tablet_mode/tablet_mode_controller.cc

Issue 2906803002: Rename MaximizeMode to TabletMode (Closed)
Patch Set: updated filter Created 3 years, 7 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: ash/wm/tablet_mode/tablet_mode_controller.cc
diff --git a/ash/wm/maximize_mode/maximize_mode_controller.cc b/ash/wm/tablet_mode/tablet_mode_controller.cc
similarity index 71%
rename from ash/wm/maximize_mode/maximize_mode_controller.cc
rename to ash/wm/tablet_mode/tablet_mode_controller.cc
index 39b21cd26d09a52c5d54fffbf0a9fdb7ab4c83c0..5525e4635472b90a3d184e5d35f3e4b0f67cdc5a 100644
--- a/ash/wm/maximize_mode/maximize_mode_controller.cc
+++ b/ash/wm/tablet_mode/tablet_mode_controller.cc
@@ -2,15 +2,15 @@
// Use of this source code is governed by a BSD-style license that can be
// found in the LICENSE file.
-#include "ash/wm/maximize_mode/maximize_mode_controller.h"
+#include "ash/wm/tablet_mode/tablet_mode_controller.h"
#include <utility>
#include "ash/ash_switches.h"
#include "ash/shell.h"
#include "ash/shell_port.h"
-#include "ash/wm/maximize_mode/maximize_mode_window_manager.h"
-#include "ash/wm/maximize_mode/scoped_disable_internal_mouse_and_keyboard.h"
+#include "ash/wm/tablet_mode/scoped_disable_internal_mouse_and_keyboard.h"
+#include "ash/wm/tablet_mode/tablet_mode_window_manager.h"
#include "base/bind.h"
#include "base/command_line.h"
#include "base/metrics/histogram_macros.h"
@@ -28,12 +28,12 @@ namespace ash {
namespace {
-// The hinge angle at which to enter maximize mode.
-const float kEnterMaximizeModeAngle = 200.0f;
+// The hinge angle at which to enter tablet mode.
+const float kEnterTabletModeAngle = 200.0f;
-// The angle at which to exit maximize mode, this is specifically less than the
-// angle to enter maximize mode to prevent rapid toggling when near the angle.
-const float kExitMaximizeModeAngle = 160.0f;
+// The angle at which to exit tablet mode, this is specifically less than the
+// angle to enter tablet mode to prevent rapid toggling when near the angle.
+const float kExitTabletModeAngle = 160.0f;
// Defines a range for which accelerometer readings are considered accurate.
// When the lid is near open (or near closed) the accelerometer readings may be
@@ -43,7 +43,7 @@ const float kMinStableAngle = 20.0f;
const float kMaxStableAngle = 340.0f;
// The time duration to consider the lid to be recently opened.
-// This is used to prevent entering maximize mode if an erroneous accelerometer
+// This is used to prevent entering tablet mode if an erroneous accelerometer
// reading makes the lid appear to be fully open when the user is opening the
// lid from a closed position.
const int kLidRecentlyOpenedDurationSeconds = 2;
@@ -85,25 +85,25 @@ bool IsEnabled() {
switches::kAshEnableTouchView);
}
-// Checks the command line to see which force maximize mode is turned on, if
+// Checks the command line to see which force tablet mode is turned on, if
// any.
-MaximizeModeController::ForceTabletMode GetMaximizeMode() {
+TabletModeController::ForceTabletMode GetTabletMode() {
base::CommandLine* command_line = base::CommandLine::ForCurrentProcess();
if (command_line->HasSwitch(switches::kAshForceTabletMode)) {
std::string switch_value =
command_line->GetSwitchValueASCII(switches::kAshForceTabletMode);
if (switch_value == switches::kAshForceTabletModeClamshell)
- return MaximizeModeController::ForceTabletMode::CLAMSHELL;
+ return TabletModeController::ForceTabletMode::CLAMSHELL;
if (switch_value == switches::kAshForceTabletModeTouchView)
- return MaximizeModeController::ForceTabletMode::TOUCHVIEW;
+ return TabletModeController::ForceTabletMode::TOUCHVIEW;
}
- return MaximizeModeController::ForceTabletMode::NONE;
+ return TabletModeController::ForceTabletMode::NONE;
}
} // namespace
-MaximizeModeController::MaximizeModeController()
+TabletModeController::TabletModeController()
: have_seen_accelerometer_data_(false),
can_detect_lid_angle_(false),
touchview_usage_interval_start_time_(base::Time::Now()),
@@ -116,9 +116,9 @@ MaximizeModeController::MaximizeModeController()
ShellPort::Get()->RecordUserMetricsAction(
UMA_MAXIMIZE_MODE_INITIALLY_DISABLED);
- // TODO(jonross): Do not create MaximizeModeController if the flag is
+ // TODO(jonross): Do not create TabletModeController if the flag is
// unavailable. This will require refactoring
- // IsMaximizeModeWindowManagerEnabled to check for the existance of the
+ // IsTabletModeWindowManagerEnabled to check for the existance of the
// controller.
if (IsEnabled()) {
ShellPort::Get()->AddDisplayObserver(this);
@@ -128,10 +128,10 @@ MaximizeModeController::MaximizeModeController()
chromeos::DBusThreadManager::Get()->GetPowerManagerClient();
power_manager_client->AddObserver(this);
power_manager_client->GetSwitchStates(base::Bind(
- &MaximizeModeController::OnGetSwitchStates, weak_factory_.GetWeakPtr()));
+ &TabletModeController::OnGetSwitchStates, weak_factory_.GetWeakPtr()));
}
-MaximizeModeController::~MaximizeModeController() {
+TabletModeController::~TabletModeController() {
Shell::Get()->RemoveShellObserver(this);
if (IsEnabled()) {
@@ -142,38 +142,37 @@ MaximizeModeController::~MaximizeModeController() {
this);
}
-bool MaximizeModeController::CanEnterMaximizeMode() {
+bool TabletModeController::CanEnterTabletMode() {
// If we have ever seen accelerometer data, then HandleHingeRotation may
- // trigger maximize mode at some point in the future.
- // All TouchView-enabled devices can enter maximized mode.
+ // trigger tablet mode at some point in the future.
+ // All TouchView-enabled devices can enter tablet mode.
return have_seen_accelerometer_data_ || IsEnabled();
}
-// TODO(jcliang): Hide or remove EnableMaximizeModeWindowManager
+// TODO(jcliang): Hide or remove EnableTabletModeWindowManager
// (http://crbug.com/620241).
-void MaximizeModeController::EnableMaximizeModeWindowManager(
- bool should_enable) {
- bool is_enabled = !!maximize_mode_window_manager_.get();
+void TabletModeController::EnableTabletModeWindowManager(bool should_enable) {
+ bool is_enabled = !!tablet_mode_window_manager_.get();
if (should_enable == is_enabled)
return;
if (should_enable) {
- maximize_mode_window_manager_.reset(new MaximizeModeWindowManager());
- // TODO(jonross): Move the maximize mode notifications from ShellObserver
- // to MaximizeModeController::Observer
+ tablet_mode_window_manager_.reset(new TabletModeWindowManager());
+ // TODO(jonross): Move the tablet mode notifications from ShellObserver
+ // to TabletModeController::Observer
ShellPort::Get()->RecordUserMetricsAction(UMA_MAXIMIZE_MODE_ENABLED);
- Shell::Get()->NotifyMaximizeModeStarted();
+ Shell::Get()->NotifyTabletModeStarted();
observers_.ForAllPtrs([](mojom::TouchViewObserver* observer) {
observer->OnTouchViewToggled(true);
});
} else {
- maximize_mode_window_manager_->SetIgnoreWmEventsForExit();
- Shell::Get()->NotifyMaximizeModeEnding();
- maximize_mode_window_manager_.reset();
+ tablet_mode_window_manager_->SetIgnoreWmEventsForExit();
+ Shell::Get()->NotifyTabletModeEnding();
+ tablet_mode_window_manager_.reset();
ShellPort::Get()->RecordUserMetricsAction(UMA_MAXIMIZE_MODE_DISABLED);
- Shell::Get()->NotifyMaximizeModeEnded();
+ Shell::Get()->NotifyTabletModeEnded();
observers_.ForAllPtrs([](mojom::TouchViewObserver* observer) {
observer->OnTouchViewToggled(false);
@@ -181,23 +180,22 @@ void MaximizeModeController::EnableMaximizeModeWindowManager(
}
}
-bool MaximizeModeController::IsMaximizeModeWindowManagerEnabled() const {
- return maximize_mode_window_manager_.get() != NULL;
+bool TabletModeController::IsTabletModeWindowManagerEnabled() const {
+ return tablet_mode_window_manager_.get() != NULL;
}
-void MaximizeModeController::AddWindow(aura::Window* window) {
- if (IsMaximizeModeWindowManagerEnabled())
- maximize_mode_window_manager_->AddWindow(window);
+void TabletModeController::AddWindow(aura::Window* window) {
+ if (IsTabletModeWindowManagerEnabled())
+ tablet_mode_window_manager_->AddWindow(window);
}
-void MaximizeModeController::BindRequest(
- mojom::TouchViewManagerRequest request) {
+void TabletModeController::BindRequest(mojom::TouchViewManagerRequest request) {
bindings_.AddBinding(this, std::move(request));
}
-void MaximizeModeController::OnAccelerometerUpdated(
+void TabletModeController::OnAccelerometerUpdated(
scoped_refptr<const chromeos::AccelerometerUpdate> update) {
- if (!AllowEnterExitMaximizeMode())
+ if (!AllowEnterExitTabletMode())
return;
have_seen_accelerometer_data_ = true;
@@ -216,8 +214,8 @@ void MaximizeModeController::OnAccelerometerUpdated(
return;
}
- // Whether or not we enter maximize mode affects whether we handle screen
- // rotation, so determine whether to enter maximize mode first.
+ // Whether or not we enter tablet mode affects whether we handle screen
+ // rotation, so determine whether to enter tablet mode first.
if (ui::IsAccelerometerReadingStable(*update,
chromeos::ACCELEROMETER_SOURCE_SCREEN) &&
ui::IsAccelerometerReadingStable(
@@ -231,23 +229,23 @@ void MaximizeModeController::OnAccelerometerUpdated(
}
}
-void MaximizeModeController::LidEventReceived(
+void TabletModeController::LidEventReceived(
chromeos::PowerManagerClient::LidState state,
const base::TimeTicks& time) {
- if (!AllowEnterExitMaximizeMode())
+ if (!AllowEnterExitTabletMode())
return;
const bool open = state == chromeos::PowerManagerClient::LidState::OPEN;
if (open)
last_lid_open_time_ = time;
lid_is_closed_ = !open;
- LeaveMaximizeMode();
+ LeaveTabletMode();
}
-void MaximizeModeController::TabletModeEventReceived(
+void TabletModeController::TabletModeEventReceived(
chromeos::PowerManagerClient::TabletMode mode,
const base::TimeTicks& time) {
- if (!AllowEnterExitMaximizeMode())
+ if (!AllowEnterExitTabletMode())
return;
const bool on = mode == chromeos::PowerManagerClient::TabletMode::ON;
@@ -259,29 +257,28 @@ void MaximizeModeController::TabletModeEventReceived(
return;
}
// The tablet mode switch activates at 300 degrees, so it is always reliable
- // when |on|. However we wish to exit maximize mode at a smaller angle, so
+ // when |on|. However we wish to exit tablet mode at a smaller angle, so
// when |on| is false we ignore if it is possible to calculate the lid angle.
- if (on && !IsMaximizeModeWindowManagerEnabled()) {
- EnterMaximizeMode();
- } else if (!on && IsMaximizeModeWindowManagerEnabled() &&
+ if (on && !IsTabletModeWindowManagerEnabled()) {
+ EnterTabletMode();
+ } else if (!on && IsTabletModeWindowManagerEnabled() &&
!can_detect_lid_angle_) {
- LeaveMaximizeMode();
+ LeaveTabletMode();
}
}
-void MaximizeModeController::SuspendImminent() {
+void TabletModeController::SuspendImminent() {
// The system is about to suspend, so record TouchView usage interval metrics
// based on whether TouchView mode is currently active.
RecordTouchViewUsageInterval(CurrentTouchViewIntervalType());
}
-void MaximizeModeController::SuspendDone(
- const base::TimeDelta& sleep_duration) {
+void TabletModeController::SuspendDone(const base::TimeDelta& sleep_duration) {
// We do not want TouchView usage metrics to include time spent in suspend.
touchview_usage_interval_start_time_ = base::Time::Now();
}
-void MaximizeModeController::HandleHingeRotation(
+void TabletModeController::HandleHingeRotation(
scoped_refptr<const chromeos::AccelerometerUpdate> update) {
static const gfx::Vector3dF hinge_vector(1.0f, 0.0f, 0.0f);
gfx::Vector3dF base_reading(ui::ConvertAccelerometerReadingToVector3dF(
@@ -338,72 +335,71 @@ void MaximizeModeController::HandleHingeRotation(
if (is_angle_stable)
last_lid_open_time_ = base::TimeTicks();
- // Toggle maximize mode on or off when corresponding thresholds are passed.
- if (IsMaximizeModeWindowManagerEnabled() && is_angle_stable &&
- lid_angle <= kExitMaximizeModeAngle) {
- LeaveMaximizeMode();
- } else if (!IsMaximizeModeWindowManagerEnabled() && !lid_is_closed_ &&
- lid_angle >= kEnterMaximizeModeAngle &&
+ // Toggle tablet mode on or off when corresponding thresholds are passed.
+ if (IsTabletModeWindowManagerEnabled() && is_angle_stable &&
+ lid_angle <= kExitTabletModeAngle) {
+ LeaveTabletMode();
+ } else if (!IsTabletModeWindowManagerEnabled() && !lid_is_closed_ &&
+ lid_angle >= kEnterTabletModeAngle &&
(is_angle_stable || !WasLidOpenedRecently())) {
- EnterMaximizeMode();
+ EnterTabletMode();
}
}
-void MaximizeModeController::EnterMaximizeMode() {
+void TabletModeController::EnterTabletMode() {
// Always reset first to avoid creation before destruction of a previous
// object.
event_blocker_ =
ShellPort::Get()->CreateScopedDisableInternalMouseAndKeyboard();
- if (IsMaximizeModeWindowManagerEnabled())
+ if (IsTabletModeWindowManagerEnabled())
return;
- EnableMaximizeModeWindowManager(true);
+ EnableTabletModeWindowManager(true);
}
-void MaximizeModeController::LeaveMaximizeMode() {
+void TabletModeController::LeaveTabletMode() {
event_blocker_.reset();
- if (!IsMaximizeModeWindowManagerEnabled())
+ if (!IsTabletModeWindowManagerEnabled())
return;
- EnableMaximizeModeWindowManager(false);
+ EnableTabletModeWindowManager(false);
}
-// Called after maximize mode has started, windows might still animate though.
-void MaximizeModeController::OnMaximizeModeStarted() {
+// Called after tablet mode has started, windows might still animate though.
+void TabletModeController::OnTabletModeStarted() {
RecordTouchViewUsageInterval(TOUCH_VIEW_INTERVAL_INACTIVE);
}
-// Called after maximize mode has ended, windows might still be returning to
+// Called after tablet mode has ended, windows might still be returning to
// their original position.
-void MaximizeModeController::OnMaximizeModeEnded() {
+void TabletModeController::OnTabletModeEnded() {
RecordTouchViewUsageInterval(TOUCH_VIEW_INTERVAL_ACTIVE);
}
-void MaximizeModeController::OnShellInitialized() {
- force_tablet_mode_ = GetMaximizeMode();
+void TabletModeController::OnShellInitialized() {
+ force_tablet_mode_ = GetTabletMode();
if (force_tablet_mode_ == ForceTabletMode::TOUCHVIEW)
- EnterMaximizeMode();
+ EnterTabletMode();
}
-void MaximizeModeController::OnDisplayConfigurationChanged() {
+void TabletModeController::OnDisplayConfigurationChanged() {
if (!display::Display::HasInternalDisplay() ||
!ShellPort::Get()->IsActiveDisplayId(
display::Display::InternalDisplayId())) {
- LeaveMaximizeMode();
- } else if (tablet_mode_switch_is_on_ &&
- !IsMaximizeModeWindowManagerEnabled()) {
+ LeaveTabletMode();
+ } else if (tablet_mode_switch_is_on_ && !IsTabletModeWindowManagerEnabled()) {
// The internal display has returned, as we are exiting docked mode.
- // The device is still in tablet mode, so trigger maximize mode, as this
+ // The device is still in tablet mode, so trigger tablet mode, as this
// switch leads to the ignoring of accelerometer events. When the switch is
// not set the next stable accelerometer readings will trigger maximize
// mode.
- EnterMaximizeMode();
+ EnterTabletMode();
}
}
-void MaximizeModeController::RecordTouchViewUsageInterval(
+void TabletModeController::RecordTouchViewUsageInterval(
TouchViewIntervalType type) {
- if (!CanEnterMaximizeMode())
+ if (!CanEnterTabletMode())
return;
base::Time current_time = base::Time::Now();
@@ -422,28 +418,28 @@ void MaximizeModeController::RecordTouchViewUsageInterval(
touchview_usage_interval_start_time_ = current_time;
}
-MaximizeModeController::TouchViewIntervalType
-MaximizeModeController::CurrentTouchViewIntervalType() {
- if (IsMaximizeModeWindowManagerEnabled())
+TabletModeController::TouchViewIntervalType
+TabletModeController::CurrentTouchViewIntervalType() {
+ if (IsTabletModeWindowManagerEnabled())
return TOUCH_VIEW_INTERVAL_ACTIVE;
return TOUCH_VIEW_INTERVAL_INACTIVE;
}
-void MaximizeModeController::AddObserver(mojom::TouchViewObserverPtr observer) {
- observer->OnTouchViewToggled(IsMaximizeModeWindowManagerEnabled());
+void TabletModeController::AddObserver(mojom::TouchViewObserverPtr observer) {
+ observer->OnTouchViewToggled(IsTabletModeWindowManagerEnabled());
observers_.AddPtr(std::move(observer));
}
-bool MaximizeModeController::AllowEnterExitMaximizeMode() const {
+bool TabletModeController::AllowEnterExitTabletMode() const {
return force_tablet_mode_ == ForceTabletMode::NONE;
}
-void MaximizeModeController::OnChromeTerminating() {
+void TabletModeController::OnChromeTerminating() {
// The system is about to shut down, so record TouchView usage interval
// metrics based on whether TouchView mode is currently active.
RecordTouchViewUsageInterval(CurrentTouchViewIntervalType());
- if (CanEnterMaximizeMode()) {
+ if (CanEnterTabletMode()) {
UMA_HISTOGRAM_CUSTOM_COUNTS("Ash.TouchView.TouchViewActiveTotal",
total_touchview_time_.InMinutes(), 1,
base::TimeDelta::FromDays(7).InMinutes(), 50);
@@ -460,14 +456,14 @@ void MaximizeModeController::OnChromeTerminating() {
}
}
-void MaximizeModeController::OnGetSwitchStates(
+void TabletModeController::OnGetSwitchStates(
chromeos::PowerManagerClient::LidState lid_state,
chromeos::PowerManagerClient::TabletMode tablet_mode) {
LidEventReceived(lid_state, base::TimeTicks::Now());
TabletModeEventReceived(tablet_mode, base::TimeTicks::Now());
}
-bool MaximizeModeController::WasLidOpenedRecently() const {
+bool TabletModeController::WasLidOpenedRecently() const {
if (last_lid_open_time_.is_null())
return false;
@@ -477,7 +473,7 @@ bool MaximizeModeController::WasLidOpenedRecently() const {
return elapsed_time.InSeconds() <= kLidRecentlyOpenedDurationSeconds;
}
-void MaximizeModeController::SetTickClockForTest(
+void TabletModeController::SetTickClockForTest(
std::unique_ptr<base::TickClock> tick_clock) {
DCHECK(tick_clock_);
tick_clock_ = std::move(tick_clock);

Powered by Google App Engine
This is Rietveld 408576698