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

Side by Side Diff: ui/base/material_design/material_design_controller.cc

Issue 1720273002: Set material hybrid by default on ChromeOS touch devices (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 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 unified diff | Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright 2015 The Chromium Authors. All rights reserved. 1 // Copyright 2015 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include <string> 5 #include <string>
6 6
7 #include "base/command_line.h" 7 #include "base/command_line.h"
8 #include "base/logging.h" 8 #include "base/logging.h"
9 #include "ui/base/material_design/material_design_controller.h" 9 #include "ui/base/material_design/material_design_controller.h"
10 #include "ui/base/touch/touch_device.h"
10 #include "ui/base/ui_base_switches.h" 11 #include "ui/base/ui_base_switches.h"
11 12
12 namespace ui { 13 namespace ui {
13 14
14 bool MaterialDesignController::is_mode_initialized_ = false; 15 bool MaterialDesignController::is_mode_initialized_ = false;
15 16
16 MaterialDesignController::Mode MaterialDesignController::mode_ = 17 MaterialDesignController::Mode MaterialDesignController::mode_ =
17 MaterialDesignController::Mode::NON_MATERIAL; 18 MaterialDesignController::Mode::NON_MATERIAL;
18 19
19 MaterialDesignController::Mode MaterialDesignController::GetMode() { 20 MaterialDesignController::Mode MaterialDesignController::GetMode() {
20 if (!is_mode_initialized_) 21 if (!is_mode_initialized_)
21 InitializeMode(); 22 InitializeMode();
22 CHECK(is_mode_initialized_); 23 CHECK(is_mode_initialized_);
23 return mode_; 24 return mode_;
24 } 25 }
25 26
26 bool MaterialDesignController::IsModeMaterial() { 27 bool MaterialDesignController::IsModeMaterial() {
27 return GetMode() == Mode::MATERIAL_NORMAL || 28 return GetMode() == Mode::MATERIAL_NORMAL ||
28 GetMode() == Mode::MATERIAL_HYBRID; 29 GetMode() == Mode::MATERIAL_HYBRID;
29 } 30 }
30 31
31 MaterialDesignController::Mode MaterialDesignController::DefaultMode() { 32 MaterialDesignController::Mode MaterialDesignController::DefaultMode() {
32 #if defined(OS_CHROMEOS) 33 #if defined(OS_CHROMEOS)
34 if (GetTouchScreensAvailability() == TouchScreensAvailability::ENABLED)
tdanderson 2016/02/22 23:47:40 Doesn't look like this will work as-is, since the
35 return Mode::MATERIAL_HYBRID;
33 return Mode::MATERIAL_NORMAL; 36 return Mode::MATERIAL_NORMAL;
34 #else 37 #else
35 return Mode::NON_MATERIAL; 38 return Mode::NON_MATERIAL;
36 #endif // defined(OS_CHROMEOS) 39 #endif // defined(OS_CHROMEOS)
37 } 40 }
38 41
39 void MaterialDesignController::InitializeMode() { 42 void MaterialDesignController::InitializeMode() {
40 #if !defined(ENABLE_TOPCHROME_MD) 43 #if !defined(ENABLE_TOPCHROME_MD)
41 SetMode(Mode::NON_MATERIAL); 44 SetMode(Mode::NON_MATERIAL);
42 #else 45 #else
(...skipping 23 matching lines...) Expand all
66 MaterialDesignController::Mode::NON_MATERIAL); 69 MaterialDesignController::Mode::NON_MATERIAL);
67 is_mode_initialized_ = false; 70 is_mode_initialized_ = false;
68 } 71 }
69 72
70 void MaterialDesignController::SetMode(MaterialDesignController::Mode mode) { 73 void MaterialDesignController::SetMode(MaterialDesignController::Mode mode) {
71 mode_ = mode; 74 mode_ = mode;
72 is_mode_initialized_ = true; 75 is_mode_initialized_ = true;
73 } 76 }
74 77
75 } // namespace ui 78 } // namespace ui
OLDNEW
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698