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

Unified Diff: components/previews/core/previews_black_list.cc

Issue 2760063002: Add support to previews/ for Server LoFi and LitePages (Closed)
Patch Set: build fix Created 3 years, 9 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 side-by-side diff with in-line comments
Download patch
Index: components/previews/core/previews_black_list.cc
diff --git a/components/previews/core/previews_black_list.cc b/components/previews/core/previews_black_list.cc
index ea5de7d822f6ebb8cf9d5504a500ad896895303a..f74cb45fa00d9e9797e994a3577c00c93e4ca35d 100644
--- a/components/previews/core/previews_black_list.cc
+++ b/components/previews/core/previews_black_list.cc
@@ -75,10 +75,17 @@ void PreviewsBlackList::AddPreviewNavigation(const GURL& url,
DCHECK(thread_checker_.CalledOnValidThread());
DCHECK(url.has_host());
switch (type) {
+ case PreviewsType::LITE_PAGE:
+ UMA_HISTOGRAM_BOOLEAN("Previews.OptOut.UserOptedOut.LitePage", opt_out);
+ break;
+ case PreviewsType::SERVER_LOFI:
+ UMA_HISTOGRAM_BOOLEAN("Previews.OptOut.UserOptedOut.ServerLoFi", opt_out);
+ break;
case PreviewsType::OFFLINE:
UMA_HISTOGRAM_BOOLEAN("Previews.OptOut.UserOptedOut.Offline", opt_out);
break;
- default:
+ case PreviewsType::NONE:
+ case PreviewsType::LAST:
NOTREACHED();
}
if (opt_out) {

Powered by Google App Engine
This is Rietveld 408576698