[NTP::Push] Add feature

chrome/browser/flag_descriptions.cc

Index: chrome/browser/flag_descriptions.cc
diff --git a/chrome/browser/flag_descriptions.cc b/chrome/browser/flag_descriptions.cc
index af8b38ede4704bcf3ab0f057ce3e4340a090d51a..4d5302d4e5c593493290fbc59df5cff6eb2d43b2 100644
--- a/chrome/browser/flag_descriptions.cc
+++ b/chrome/browser/flag_descriptions.cc
@@ -2443,6 +2443,13 @@ const char kEnableNtpSuggestionsNotificationsDescription[] =
     "If enabled, notifications will inform about new content suggestions on "
     "the New Tab page (see #enable-ntp-snippets).";
 
+const char kEnableNtpPushUpdatesName[] =
+    "Receive GCM push updates for timely content at the New Tab Page";

[sfiera, 2017/05/24 14:11:46]

It's not really tied to the New Tab Page. You could just say:
"Receive timely content suggestions via GCM push"

(applies to all places NTP appears in this CL)

[mamir, 2017/05/24 15:07:27]

Done.

+
+const char kEnableNtpPushUpdatesDescription[] =
+    "If enabled, the client will listen for push messages sent from the "
+    "server about timely content such as breaking news.";

[sfiera, 2017/05/24 14:11:46]

Describing this flag in terms of "client" and "server" makes sense to the
implementer but not to others. Better to describe the observable behavior and
not the implementation.
"If enabled, Chrome will register with Google services to receive content
suggestions via push (in addition to normal background polling). Push will be
used for timely content, such as breaking news."

[mamir, 2017/05/24 15:07:27]

Done.

+
 const char kNtpCondensedLayoutName[] = "Condensed NTP layout";
 
 const char kNtpCondensedLayoutDescription[] =