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

Side by Side Diff: chrome/browser/metrics/chrome_browser_main_extra_parts_metrics.cc

Issue 2028593004: Add new InputDeviceManager interface. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@remove_keyboard
Patch Set: Fix windows compile error. Created 4 years, 6 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
OLDNEW
1 // Copyright (c) 2013 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2013 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 "chrome/browser/metrics/chrome_browser_main_extra_parts_metrics.h" 5 #include "chrome/browser/metrics/chrome_browser_main_extra_parts_metrics.h"
6 6
7 #include <string> 7 #include <string>
8 8
9 #include "base/bind.h" 9 #include "base/bind.h"
10 #include "base/command_line.h" 10 #include "base/command_line.h"
(...skipping 28 matching lines...) Expand all
39 #if defined(OS_LINUX) && !defined(OS_CHROMEOS) 39 #if defined(OS_LINUX) && !defined(OS_CHROMEOS)
40 #include <gnu/libc-version.h> 40 #include <gnu/libc-version.h>
41 41
42 #include "base/version.h" 42 #include "base/version.h"
43 #if defined(USE_X11) 43 #if defined(USE_X11)
44 #include "ui/base/x/x11_util.h" 44 #include "ui/base/x/x11_util.h"
45 #endif 45 #endif
46 #endif // defined(OS_LINUX) && !defined(OS_CHROMEOS) 46 #endif // defined(OS_LINUX) && !defined(OS_CHROMEOS)
47 47
48 #if defined(USE_OZONE) || defined(USE_X11) 48 #if defined(USE_OZONE) || defined(USE_X11)
49 #include "ui/events/devices/device_data_manager.h"
50 #include "ui/events/devices/input_device_event_observer.h" 49 #include "ui/events/devices/input_device_event_observer.h"
50 #include "ui/events/devices/input_device_manager.h"
51 #endif // defined(USE_OZONE) || defined(USE_X11) 51 #endif // defined(USE_OZONE) || defined(USE_X11)
52 52
53 #if defined(OS_WIN) 53 #if defined(OS_WIN)
54 #include "base/win/windows_version.h" 54 #include "base/win/windows_version.h"
55 #include "chrome/installer/util/google_update_settings.h" 55 #include "chrome/installer/util/google_update_settings.h"
56 #endif // defined(OS_WIN) 56 #endif // defined(OS_WIN)
57 57
58 namespace { 58 namespace {
59 59
60 enum UMALinuxGlibcVersion { 60 enum UMALinuxGlibcVersion {
(...skipping 220 matching lines...) Expand 10 before | Expand all | Expand 10 after
281 ~AsynchronousTouchEventStateRecorder() override; 281 ~AsynchronousTouchEventStateRecorder() override;
282 282
283 // ui::InputDeviceEventObserver overrides. 283 // ui::InputDeviceEventObserver overrides.
284 void OnDeviceListsComplete() override; 284 void OnDeviceListsComplete() override;
285 285
286 private: 286 private:
287 DISALLOW_COPY_AND_ASSIGN(AsynchronousTouchEventStateRecorder); 287 DISALLOW_COPY_AND_ASSIGN(AsynchronousTouchEventStateRecorder);
288 }; 288 };
289 289
290 AsynchronousTouchEventStateRecorder::AsynchronousTouchEventStateRecorder() { 290 AsynchronousTouchEventStateRecorder::AsynchronousTouchEventStateRecorder() {
291 ui::DeviceDataManager::GetInstance()->AddObserver(this); 291 ui::InputDeviceManager::GetInstance()->AddObserver(this);
292 } 292 }
293 293
294 AsynchronousTouchEventStateRecorder::~AsynchronousTouchEventStateRecorder() { 294 AsynchronousTouchEventStateRecorder::~AsynchronousTouchEventStateRecorder() {
295 ui::DeviceDataManager::GetInstance()->RemoveObserver(this); 295 ui::InputDeviceManager::GetInstance()->RemoveObserver(this);
296 } 296 }
297 297
298 void AsynchronousTouchEventStateRecorder::OnDeviceListsComplete() { 298 void AsynchronousTouchEventStateRecorder::OnDeviceListsComplete() {
299 ui::DeviceDataManager::GetInstance()->RemoveObserver(this); 299 ui::InputDeviceManager::GetInstance()->RemoveObserver(this);
300 RecordTouchEventState(); 300 RecordTouchEventState();
301 } 301 }
302 302
303 #endif // defined(USE_OZONE) || defined(USE_X11) 303 #endif // defined(USE_OZONE) || defined(USE_X11)
304 304
305 } // namespace 305 } // namespace
306 306
307 ChromeBrowserMainExtraPartsMetrics::ChromeBrowserMainExtraPartsMetrics() 307 ChromeBrowserMainExtraPartsMetrics::ChromeBrowserMainExtraPartsMetrics()
308 : display_count_(0), is_screen_observer_(false) { 308 : display_count_(0), is_screen_observer_(false) {
309 } 309 }
(...skipping 18 matching lines...) Expand all
328 #if defined(USE_X11) && !defined(OS_CHROMEOS) 328 #if defined(USE_X11) && !defined(OS_CHROMEOS)
329 UMA_HISTOGRAM_ENUMERATION("Linux.WindowManager", 329 UMA_HISTOGRAM_ENUMERATION("Linux.WindowManager",
330 GetLinuxWindowManager(), 330 GetLinuxWindowManager(),
331 UMA_LINUX_WINDOW_MANAGER_COUNT); 331 UMA_LINUX_WINDOW_MANAGER_COUNT);
332 #endif 332 #endif
333 333
334 #if defined(USE_OZONE) || defined(USE_X11) 334 #if defined(USE_OZONE) || defined(USE_X11)
335 // The touch event state for X11 and Ozone based event sub-systems are based 335 // The touch event state for X11 and Ozone based event sub-systems are based
336 // on device scans that happen asynchronously. So we may need to attach an 336 // on device scans that happen asynchronously. So we may need to attach an
337 // observer to wait until these scans complete. 337 // observer to wait until these scans complete.
338 if (ui::DeviceDataManager::GetInstance()->device_lists_complete()) { 338 if (ui::InputDeviceManager::GetInstance()->AreDeviceListsComplete()) {
339 RecordTouchEventState(); 339 RecordTouchEventState();
340 } else { 340 } else {
341 input_device_event_observer_.reset( 341 input_device_event_observer_.reset(
342 new AsynchronousTouchEventStateRecorder()); 342 new AsynchronousTouchEventStateRecorder());
343 } 343 }
344 #else 344 #else
345 RecordTouchEventState(); 345 RecordTouchEventState();
346 #endif // defined(USE_OZONE) || defined(USE_X11) 346 #endif // defined(USE_OZONE) || defined(USE_X11)
347 347
348 #if defined(OS_MACOSX) 348 #if defined(OS_MACOSX)
(...skipping 38 matching lines...) Expand 10 before | Expand all | Expand 10 after
387 } 387 }
388 } 388 }
389 389
390 namespace chrome { 390 namespace chrome {
391 391
392 void AddMetricsExtraParts(ChromeBrowserMainParts* main_parts) { 392 void AddMetricsExtraParts(ChromeBrowserMainParts* main_parts) {
393 main_parts->AddParts(new ChromeBrowserMainExtraPartsMetrics()); 393 main_parts->AddParts(new ChromeBrowserMainExtraPartsMetrics());
394 } 394 }
395 395
396 } // namespace chrome 396 } // namespace chrome
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698