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

Unified Diff: ui/display/chromeos/display_change_observer.cc

Issue 2524863003: Move DisplayChangeObserver + tests from ash to ui. (Closed)
Patch Set: Replace with static method. Created 4 years, 1 month 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: ui/display/chromeos/display_change_observer.cc
diff --git a/ash/display/display_change_observer_chromeos.cc b/ui/display/chromeos/display_change_observer.cc
similarity index 75%
rename from ash/display/display_change_observer_chromeos.cc
rename to ui/display/chromeos/display_change_observer.cc
index 4b16348ddd96c410a6054d7fb3a8f42ac2fa769b..3a8c8efd2a68878c40c076fca27dd79cabbdfbab 100644
--- a/ash/display/display_change_observer_chromeos.cc
+++ b/ui/display/chromeos/display_change_observer.cc
@@ -2,7 +2,7 @@
// Use of this source code is governed by a BSD-style license that can be
// found in the LICENSE file.
-#include "ash/display/display_change_observer_chromeos.h"
+#include "ui/display/chromeos/display_change_observer.h"
#include <algorithm>
#include <map>
@@ -12,7 +12,6 @@
#include <vector>
#include "base/logging.h"
-#include "ui/base/user_activity/user_activity_detector.h"
#include "ui/display/chromeos/touchscreen_util.h"
#include "ui/display/display.h"
#include "ui/display/manager/display_layout.h"
@@ -25,7 +24,7 @@
#include "ui/events/devices/input_device_manager.h"
#include "ui/events/devices/touchscreen_device.h"
-namespace ash {
+namespace display {
using ui::DisplayConfigurator;
@@ -45,9 +44,9 @@ void UpdateInternalDisplayId(
const ui::DisplayConfigurator::DisplayStateList& display_states) {
for (auto* state : display_states) {
if (state->type() == ui::DISPLAY_CONNECTION_TYPE_INTERNAL) {
- if (display::Display::HasInternalDisplay())
- DCHECK_EQ(display::Display::InternalDisplayId(), state->display_id());
- display::Display::SetInternalDisplayId(state->display_id());
+ if (Display::HasInternalDisplay())
+ DCHECK_EQ(Display::InternalDisplayId(), state->display_id());
+ Display::SetInternalDisplayId(state->display_id());
}
}
}
@@ -55,38 +54,35 @@ void UpdateInternalDisplayId(
} // namespace
// static
-display::ManagedDisplayInfo::ManagedDisplayModeList
+ManagedDisplayInfo::ManagedDisplayModeList
DisplayChangeObserver::GetInternalManagedDisplayModeList(
- const display::ManagedDisplayInfo& display_info,
+ const ManagedDisplayInfo& display_info,
const ui::DisplaySnapshot& output) {
const ui::DisplayMode* ui_native_mode = output.native_mode();
- scoped_refptr<display::ManagedDisplayMode> native_mode =
- new display::ManagedDisplayMode(ui_native_mode->size(),
- ui_native_mode->refresh_rate(),
- ui_native_mode->is_interlaced(), true,
- 1.0, display_info.device_scale_factor());
+ scoped_refptr<ManagedDisplayMode> native_mode = new ManagedDisplayMode(
+ ui_native_mode->size(), ui_native_mode->refresh_rate(),
+ ui_native_mode->is_interlaced(), true, 1.0,
+ display_info.device_scale_factor());
- return display::CreateInternalManagedDisplayModeList(native_mode);
+ return CreateInternalManagedDisplayModeList(native_mode);
}
// static
-display::ManagedDisplayInfo::ManagedDisplayModeList
+ManagedDisplayInfo::ManagedDisplayModeList
DisplayChangeObserver::GetExternalManagedDisplayModeList(
const ui::DisplaySnapshot& output) {
using DisplayModeMap =
- std::map<std::pair<int, int>, scoped_refptr<display::ManagedDisplayMode>>;
+ std::map<std::pair<int, int>, scoped_refptr<ManagedDisplayMode>>;
DisplayModeMap display_mode_map;
- scoped_refptr<display::ManagedDisplayMode> native_mode =
- new display::ManagedDisplayMode();
+ scoped_refptr<ManagedDisplayMode> native_mode = new ManagedDisplayMode();
for (const auto& mode_info : output.modes()) {
const std::pair<int, int> size(mode_info->size().width(),
mode_info->size().height());
- scoped_refptr<display::ManagedDisplayMode> display_mode =
- new display::ManagedDisplayMode(
- mode_info->size(), mode_info->refresh_rate(),
- mode_info->is_interlaced(), output.native_mode() == mode_info.get(),
- 1.0, 1.0);
+ scoped_refptr<ManagedDisplayMode> display_mode = new ManagedDisplayMode(
+ mode_info->size(), mode_info->refresh_rate(),
+ mode_info->is_interlaced(), output.native_mode() == mode_info.get(),
+ 1.0, 1.0);
if (display_mode->native())
native_mode = display_mode;
@@ -100,7 +96,7 @@ DisplayChangeObserver::GetExternalManagedDisplayModeList(
display_mode_it->second = std::move(display_mode);
}
- display::ManagedDisplayInfo::ManagedDisplayModeList display_mode_list;
+ ManagedDisplayInfo::ManagedDisplayModeList display_mode_list;
for (const auto& display_mode_pair : display_mode_map)
display_mode_list.push_back(std::move(display_mode_pair.second));
@@ -118,11 +114,10 @@ DisplayChangeObserver::GetExternalManagedDisplayModeList(
if (native_mode->size().width() >= kMinimumWidthFor4K) {
for (size_t i = 0; i < arraysize(kAdditionalDeviceScaleFactorsFor4k); ++i) {
- scoped_refptr<display::ManagedDisplayMode> mode =
- new display::ManagedDisplayMode(
- native_mode->size(), native_mode->refresh_rate(),
- native_mode->is_interlaced(), false /* native */,
- native_mode->ui_scale(), kAdditionalDeviceScaleFactorsFor4k[i]);
+ scoped_refptr<ManagedDisplayMode> mode = new ManagedDisplayMode(
+ native_mode->size(), native_mode->refresh_rate(),
+ native_mode->is_interlaced(), false /* native */,
+ native_mode->ui_scale(), kAdditionalDeviceScaleFactorsFor4k[i]);
display_mode_list.push_back(mode);
}
}
@@ -131,7 +126,7 @@ DisplayChangeObserver::GetExternalManagedDisplayModeList(
}
DisplayChangeObserver::DisplayChangeObserver(
- std::unique_ptr<display::DisplayStringProvider> string_provider)
+ std::unique_ptr<DisplayStringProvider> string_provider)
: string_provider_(std::move(string_provider)) {
ui::InputDeviceManager::GetInstance()->AddObserver(this);
}
@@ -145,13 +140,13 @@ ui::MultipleDisplayState DisplayChangeObserver::GetStateForDisplayIds(
UpdateInternalDisplayId(display_states);
if (display_states.size() == 1)
return ui::MULTIPLE_DISPLAY_STATE_SINGLE;
- display::DisplayIdList list = display::GenerateDisplayIdList(
- display_states.begin(), display_states.end(),
- [](const ui::DisplaySnapshot* display_state) {
- return display_state->display_id();
- });
+ DisplayIdList list =
+ GenerateDisplayIdList(display_states.begin(), display_states.end(),
+ [](const ui::DisplaySnapshot* display_state) {
+ return display_state->display_id();
+ });
- const display::DisplayLayout& layout =
+ const DisplayLayout& layout =
display_manager_->layout_store()->GetRegisteredDisplayLayout(list);
return layout.mirrored ? ui::MULTIPLE_DISPLAY_STATE_DUAL_MIRROR
: ui::MULTIPLE_DISPLAY_STATE_DUAL_EXTENDED;
@@ -159,7 +154,7 @@ ui::MultipleDisplayState DisplayChangeObserver::GetStateForDisplayIds(
bool DisplayChangeObserver::GetResolutionForDisplayId(int64_t display_id,
gfx::Size* size) const {
- scoped_refptr<display::ManagedDisplayMode> mode =
+ scoped_refptr<ManagedDisplayMode> mode =
display_manager_->GetSelectedModeForDisplayId(display_id);
if (!mode)
return false;
@@ -171,7 +166,7 @@ void DisplayChangeObserver::OnDisplayModeChanged(
const ui::DisplayConfigurator::DisplayStateList& display_states) {
UpdateInternalDisplayId(display_states);
- std::vector<display::ManagedDisplayInfo> displays;
+ std::vector<ManagedDisplayInfo> displays;
std::set<int64_t> ids;
for (const ui::DisplaySnapshot* state : display_states) {
const ui::DisplayMode* mode_info = state->current_mode();
@@ -180,7 +175,7 @@ void DisplayChangeObserver::OnDisplayModeChanged(
float device_scale_factor = 1.0f;
// Sets dpi only if the screen size is not blacklisted.
- float dpi = display::IsDisplaySizeBlackListed(state->physical_size())
+ float dpi = IsDisplaySizeBlackListed(state->physical_size())
? 0
: kInchInMm * mode_info->size().width() /
state->physical_size().width();
@@ -188,7 +183,7 @@ void DisplayChangeObserver::OnDisplayModeChanged(
if (dpi)
device_scale_factor = FindDeviceScaleFactor(dpi);
} else {
- scoped_refptr<display::ManagedDisplayMode> mode =
+ scoped_refptr<ManagedDisplayMode> mode =
display_manager_->GetSelectedModeForDisplayId(state->display_id());
if (mode) {
device_scale_factor = mode->device_scale_factor();
@@ -229,8 +224,8 @@ void DisplayChangeObserver::OnDisplayModeChanged(
int64_t id = state->display_id();
ids.insert(id);
- displays.push_back(display::ManagedDisplayInfo(id, name, has_overscan));
- display::ManagedDisplayInfo& new_info = displays.back();
+ displays.push_back(ManagedDisplayInfo(id, name, has_overscan));
+ ManagedDisplayInfo& new_info = displays.back();
new_info.set_sys_path(state->sys_path());
new_info.set_device_scale_factor(device_scale_factor);
new_info.SetBounds(display_bounds);
@@ -240,7 +235,7 @@ void DisplayChangeObserver::OnDisplayModeChanged(
if (dpi)
new_info.set_device_dpi(dpi);
- display::ManagedDisplayInfo::ManagedDisplayModeList display_modes =
+ ManagedDisplayInfo::ManagedDisplayModeList display_modes =
(state->type() == ui::DISPLAY_CONNECTION_TYPE_INTERNAL)
? GetInternalManagedDisplayModeList(new_info, *state)
: GetExternalManagedDisplayModeList(*state);
@@ -251,17 +246,15 @@ void DisplayChangeObserver::OnDisplayModeChanged(
new_info.set_maximum_cursor_size(state->maximum_cursor_size());
}
- display::AssociateTouchscreens(
+ AssociateTouchscreens(
&displays,
ui::InputDeviceManager::GetInstance()->GetTouchscreenDevices());
display_manager_->OnNativeDisplaysChanged(displays);
// For the purposes of user activity detection, ignore synthetic mouse events
// that are triggered by screen resizes: http://crbug.com/360634
- ui::UserActivityDetector* user_activity_detector =
- ui::UserActivityDetector::Get();
- if (user_activity_detector)
- user_activity_detector->OnDisplayPowerChanging();
+ if (!display_changing_callback_.is_null())
+ display_changing_callback_.Run();
}
void DisplayChangeObserver::OnDisplayModeChangeFailed(
@@ -288,4 +281,4 @@ float DisplayChangeObserver::FindDeviceScaleFactor(float dpi) {
return 1.0f;
}
-} // namespace ash
+} // namespace display

Powered by Google App Engine
This is Rietveld 408576698