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

Side by Side Diff: chrome/browser/prerender/prerender_origin.cc

Issue 1767243002: Update prerender policy for custom tabs (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Renaming nits Created 4 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 unified diff | Download patch
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 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 "chrome/browser/prerender/prerender_origin.h" 5 #include "chrome/browser/prerender/prerender_origin.h"
6 6
7 #include "base/macros.h" 7 #include "base/macros.h"
8 #include "base/metrics/histogram.h" 8 #include "base/metrics/histogram.h"
9 #include "chrome/browser/prerender/prerender_manager.h" 9 #include "chrome/browser/prerender/prerender_manager.h"
10 10
11 namespace prerender { 11 namespace prerender {
12 12
13 namespace { 13 namespace {
14 14
15 const char* kOriginNames[] = { 15 const char* kOriginNames[] = {
16 "[Deprecated] Link Rel Prerender (original)", 16 "[Deprecated] Link Rel Prerender (original)",
17 "[Deprecated] Omnibox (original)", 17 "[Deprecated] Omnibox (original)",
18 "GWS Prerender", 18 "GWS Prerender",
19 "[Deprecated] Omnibox (conservative)", 19 "[Deprecated] Omnibox (conservative)",
20 "[Deprecated] Omnibox (exact)", 20 "[Deprecated] Omnibox (exact)",
21 "Omnibox", 21 "Omnibox",
22 "None", 22 "None",
23 "Link Rel Prerender (same domain)", 23 "Link Rel Prerender (same domain)",
24 "Link Rel Prerender (cross domain)", 24 "Link Rel Prerender (cross domain)",
25 "Local Predictor", 25 "Local Predictor",
26 "External Request", 26 "External Request",
27 "Instant", 27 "Instant",
28 "Link Rel Next", 28 "Link Rel Next",
29 "External Request Forced Custom Tabs",
pasko 2016/03/10 19:19:24 nit: it would be more human readable this way: "Fo
Yusuf 2016/03/14 23:08:20 Acknowledged.
29 "Max", 30 "Max",
30 }; 31 };
31 static_assert(arraysize(kOriginNames) == ORIGIN_MAX + 1, 32 static_assert(arraysize(kOriginNames) == ORIGIN_MAX + 1,
32 "prerender origin name count mismatch"); 33 "prerender origin name count mismatch");
33 34
34 } // namespace 35 } // namespace
35 36
36 const char* NameFromOrigin(Origin origin) { 37 const char* NameFromOrigin(Origin origin) {
37 DCHECK(static_cast<int>(origin) >= 0 && 38 DCHECK(static_cast<int>(origin) >= 0 &&
38 origin <= ORIGIN_MAX); 39 origin <= ORIGIN_MAX);
39 return kOriginNames[origin]; 40 return kOriginNames[origin];
40 } 41 }
41 42
42 } // namespace prerender 43 } // namespace prerender
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698