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

Unified Diff: ios/chrome/browser/ui/history/history_panel_view_controller.mm

Issue 2592843002: [ios] Adds history user action metrics (Closed)
Patch Set: 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 side-by-side diff with in-line comments
Download patch
Index: ios/chrome/browser/ui/history/history_panel_view_controller.mm
diff --git a/ios/chrome/browser/ui/history/history_panel_view_controller.mm b/ios/chrome/browser/ui/history/history_panel_view_controller.mm
index 41be2faa97542d813293e05832e1e87c99454cf5..3010ba729d80ff0bc0255b4e611f487a932d14ba 100644
--- a/ios/chrome/browser/ui/history/history_panel_view_controller.mm
+++ b/ios/chrome/browser/ui/history/history_panel_view_controller.mm
@@ -7,6 +7,8 @@
#include "base/ios/block_types.h"
#include "base/ios/ios_util.h"
#include "base/mac/scoped_nsobject.h"
+#include "base/metrics/user_metrics.h"
+#include "base/metrics/user_metrics_action.h"
#include "components/strings/grit/components_strings.h"
#import "ios/chrome/browser/ui/history/clear_browsing_bar.h"
#import "ios/chrome/browser/ui/history/history_collection_view_controller.h"
@@ -299,6 +301,8 @@ CGFloat kShadowOpacity = 0.2f;
- (void)openPrivacySettings {
[self exitSearchMode];
+ base::RecordAction(
+ base::UserMetricsAction("HistoryPage_InitClearBrowsingData"));
ShowClearBrowsingData();
}
@@ -324,6 +328,7 @@ CGFloat kShadowOpacity = 0.2f;
- (void)deleteSelectedItems {
[_historyCollectionController deleteSelectedItemsFromHistory];
+ base::RecordAction(base::UserMetricsAction("HistoryPage_RemoveSelected"));
[self exitEditingMode];
}
- (void)enterSearchMode {
@@ -337,6 +342,7 @@ CGFloat kShadowOpacity = 0.2f;
[self.view addSubview:searchBarView];
_historyCollectionController.get().searching = YES;
[_searchViewController didMoveToParentViewController:self];
+ base::RecordAction(base::UserMetricsAction("HistoryPage_Search"));
// Constraints to make search bar cover header.
[searchBarView setTranslatesAutoresizingMaskIntoConstraints:NO];

Powered by Google App Engine
This is Rietveld 408576698