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

Side by Side Diff: chrome/browser/ui/cocoa/website_settings/website_settings_bubble_controller.mm

Issue 690483003: Log major user interactions with the WebsiteSettings (origin info) bubble. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Remove old actions. Created 6 years, 1 month 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 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 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 #import "chrome/browser/ui/cocoa/website_settings/website_settings_bubble_contro ller.h" 5 #import "chrome/browser/ui/cocoa/website_settings/website_settings_bubble_contro ller.h"
6 6
7 #include <cmath> 7 #include <cmath>
8 8
9 #import <AppKit/AppKit.h> 9 #import <AppKit/AppKit.h>
10 10
(...skipping 486 matching lines...) Expand 10 before | Expand all | Expand 10 after
497 toView:contentView]; 497 toView:contentView];
498 [cookiesButton_ setTarget:self]; 498 [cookiesButton_ setTarget:self];
499 [cookiesButton_ setAction:@selector(showCookiesAndSiteData:)]; 499 [cookiesButton_ setAction:@selector(showCookiesAndSiteData:)];
500 500
501 return contentView.get(); 501 return contentView.get();
502 } 502 }
503 503
504 // Handler for the link button below the list of cookies. 504 // Handler for the link button below the list of cookies.
505 - (void)showCookiesAndSiteData:(id)sender { 505 - (void)showCookiesAndSiteData:(id)sender {
506 DCHECK(webContents_); 506 DCHECK(webContents_);
507 content::RecordAction(
508 base::UserMetricsAction("WebsiteSettings_CookiesDialogOpened"));
509 chrome::ShowCollectedCookiesDialog(webContents_); 507 chrome::ShowCollectedCookiesDialog(webContents_);
510 } 508 }
511 509
512 // Handler for the link button to show certificate information. 510 // Handler for the link button to show certificate information.
513 - (void)showCertificateInfo:(id)sender { 511 - (void)showCertificateInfo:(id)sender {
514 DCHECK(certificateId_); 512 DCHECK(certificateId_);
515 ShowCertificateViewerByID(webContents_, [self parentWindow], certificateId_); 513 ShowCertificateViewerByID(webContents_, [self parentWindow], certificateId_);
516 } 514 }
517 515
518 // Handler for the link button to revoke user certificate decisions. 516 // Handler for the link button to revoke user certificate decisions.
(...skipping 700 matching lines...) Expand 10 before | Expand all | Expand 10 after
1219 [bubble_controller_ setPermissionInfo:permission_info_list]; 1217 [bubble_controller_ setPermissionInfo:permission_info_list];
1220 } 1218 }
1221 1219
1222 void WebsiteSettingsUIBridge::SetFirstVisit(const base::string16& first_visit) { 1220 void WebsiteSettingsUIBridge::SetFirstVisit(const base::string16& first_visit) {
1223 [bubble_controller_ setFirstVisit:first_visit]; 1221 [bubble_controller_ setFirstVisit:first_visit];
1224 } 1222 }
1225 1223
1226 void WebsiteSettingsUIBridge::SetSelectedTab(TabId tab_id) { 1224 void WebsiteSettingsUIBridge::SetSelectedTab(TabId tab_id) {
1227 [bubble_controller_ setSelectedTab:tab_id]; 1225 [bubble_controller_ setSelectedTab:tab_id];
1228 } 1226 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698