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

Unified Diff: components/ntp_snippets/content_suggestion.h

Issue 2593573003: Ntp: use AMP urls for content suggestions when available. (Closed)
Patch Set: Address review comments by vitalii. 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
« no previous file with comments | « chrome/browser/ui/webui/snippets_internals_message_handler.cc ('k') | components/ntp_snippets/features.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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..33c84c0b66e53204f6af8937f8e34d46a806fc94 100644
--- a/components/ntp_snippets/content_suggestion.h
+++ b/components/ntp_snippets/content_suggestion.h
@@ -82,15 +82,10 @@ class ContentSuggestion {
// An ID for identifying the suggestion. The ID is unique application-wide.
const ID& id() const { return id_; }
- // The normal content URL where the content referenced by the suggestion can
- // be accessed.
+ // The URL where the content referenced by the suggestion can be accessed.
+ // This may be an AMP URL.
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_;
« no previous file with comments | « chrome/browser/ui/webui/snippets_internals_message_handler.cc ('k') | components/ntp_snippets/features.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698