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

Side by Side Diff: chrome/browser/android/vr_shell/vr_usage_monitor.cc

Issue 2510803003: Pass RapporService to content/browser/ (Closed)
Patch Set: Fix an Android compile error 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 unified diff | Download patch
OLDNEW
1 // Copyright 2016 The Chromium Authors. All rights reserved. 1 // Copyright 2016 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 "vr_usage_monitor.h" 5 #include "vr_usage_monitor.h"
6 #include "base/logging.h" 6 #include "base/logging.h"
7 #include "base/metrics/histogram_macros.h" 7 #include "base/metrics/histogram_macros.h"
8 #include "chrome/browser/browser_process.h" 8 #include "chrome/browser/browser_process.h"
9 #include "components/rappor/rappor_utils.h" 9 #include "components/rappor/public/rappor_utils.h"
10 #include "content/public/browser/browser_thread.h" 10 #include "content/public/browser/browser_thread.h"
11 11
12 // There is a dependency cycle between chrome/brower and 12 // There is a dependency cycle between chrome/brower and
13 // chrome/browser/android/vr_shell. To temporarily fix and unblock tests, 13 // chrome/browser/android/vr_shell. To temporarily fix and unblock tests,
14 // don't send rappor events. 14 // don't send rappor events.
15 #define ALLOW_SENDING_RAPPOR_EVENTS 0 15 #define ALLOW_SENDING_RAPPOR_EVENTS 0
16 16
17 namespace vr_shell { 17 namespace vr_shell {
18 18
19 // minimum duration: 7 seconds for video, no minimum for headset/vr modes 19 // minimum duration: 7 seconds for video, no minimum for headset/vr modes
(...skipping 341 matching lines...) Expand 10 before | Expand all | Expand 10 after
361 } 361 }
362 362
363 void VrMetricsHelper::DidToggleFullscreenModeForTab(bool entered_fullscreen, 363 void VrMetricsHelper::DidToggleFullscreenModeForTab(bool entered_fullscreen,
364 bool will_cause_resize) { 364 bool will_cause_resize) {
365 DCHECK_EQ(thread_id_, base::PlatformThread::CurrentId()); 365 DCHECK_EQ(thread_id_, base::PlatformThread::CurrentId());
366 is_fullscreen_ = entered_fullscreen; 366 is_fullscreen_ = entered_fullscreen;
367 UpdateMode(); 367 UpdateMode();
368 } 368 }
369 369
370 } // namespace vr_shell 370 } // namespace vr_shell
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698