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

Unified Diff: components/ntp_snippets/content_suggestion.h

Issue 2593573003: Ntp: use AMP urls for content suggestions when available. (Closed)
Patch Set: Fix tests. 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: components/ntp_snippets/content_suggestion.h
diff --git a/components/ntp_snippets/content_suggestion.h b/components/ntp_snippets/content_suggestion.h
index 992dd02d035e0fa0b6a89184df00d5416fb57ccd..50044172ff4ed0fc29c45c8c90fad009ad6638c0 100644
--- a/components/ntp_snippets/content_suggestion.h
+++ b/components/ntp_snippets/content_suggestion.h
@@ -86,11 +86,6 @@ class ContentSuggestion {
// be accessed.
vitaliii 2016/12/20 15:51:46 I would say that this may be an AMP URL. Also "nor
Michael van Ouwerkerk 2016/12/20 16:12:48 Done.
const GURL& url() const { return url_; }
- // If available, this contains an URL to an AMP version of the same content.
- // Otherwise, this is an empty GURL().
- const GURL& amp_url() const { return amp_url_; }
- void set_amp_url(const GURL& amp_url) { amp_url_ = amp_url; }
-
// Title of the suggestion.
const base::string16& title() const { return title_; }
void set_title(const base::string16& title) { title_ = title; }
@@ -141,7 +136,6 @@ class ContentSuggestion {
private:
ID id_;
GURL url_;
- GURL amp_url_;
base::string16 title_;
base::string16 snippet_text_;
base::Time publish_date_;

Powered by Google App Engine
This is Rietveld 408576698