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

Side by Side Diff: ash/common/system/chromeos/screen_security/screen_share_tray_item.cc

Issue 2132773002: Added UMA metrics for screen capture UI in the status area. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Merge branch 'master' into uma_screen_capture Created 4 years, 5 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 2013 The Chromium Authors. All rights reserved. 1 // Copyright 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 "ash/common/system/chromeos/screen_security/screen_share_tray_item.h" 5 #include "ash/common/system/chromeos/screen_security/screen_share_tray_item.h"
6 6
7 #include <utility> 7 #include <utility>
8 8
9 #include "ash/common/system/system_notifier.h" 9 #include "ash/common/system/system_notifier.h"
10 #include "ash/common/system/tray/system_tray_notifier.h" 10 #include "ash/common/system/tray/system_tray_notifier.h"
(...skipping 59 matching lines...) Expand 10 before | Expand all | Expand 10 after
70 data, new tray::ScreenNotificationDelegate(this))); 70 data, new tray::ScreenNotificationDelegate(this)));
71 notification->SetSystemPriority(); 71 notification->SetSystemPriority();
72 message_center::MessageCenter::Get()->AddNotification( 72 message_center::MessageCenter::Get()->AddNotification(
73 std::move(notification)); 73 std::move(notification));
74 } 74 }
75 75
76 std::string ScreenShareTrayItem::GetNotificationId() { 76 std::string ScreenShareTrayItem::GetNotificationId() {
77 return kScreenShareNotificationId; 77 return kScreenShareNotificationId;
78 } 78 }
79 79
80 void ScreenShareTrayItem::RecordStoppedFromDefaultViewMetric() {
81 // Intentionally not recording a metric.
82 }
83
84 void ScreenShareTrayItem::RecordStoppedFromNotificationViewMetric() {
85 // Intentionally not recording a metric.
86 }
87
80 void ScreenShareTrayItem::OnScreenShareStart( 88 void ScreenShareTrayItem::OnScreenShareStart(
81 const base::Closure& stop_callback, 89 const base::Closure& stop_callback,
82 const base::string16& helper_name) { 90 const base::string16& helper_name) {
83 helper_name_ = helper_name; 91 helper_name_ = helper_name;
84 Start(stop_callback); 92 Start(stop_callback);
85 } 93 }
86 94
87 void ScreenShareTrayItem::OnScreenShareStop() { 95 void ScreenShareTrayItem::OnScreenShareStop() {
88 // We do not need to run the stop callback 96 // We do not need to run the stop callback
89 // when screening is stopped externally. 97 // when screening is stopped externally.
90 set_is_started(false); 98 set_is_started(false);
91 Update(); 99 Update();
92 } 100 }
93 101
94 } // namespace ash 102 } // namespace ash
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698