📰 Add default empty state strings for remote sections

[NTP Client] Add default empty state strings for remote sections

Preview: https://goo.gl/photos/aK3GPFL5Z7MSzcwn8

BUG=656008
Committed: https://crrev.com/b48353878e2fcf4d3223daed5d5c13e5bbcf2e04
Cr-Commit-Position: refs/heads/master@{#425932}

[dgn, 2016-10-17 15:33:25]

Description was changed from

==========
[NTP Client] Add default empty state strings for remote sections

BUG=656008
==========

to

==========
[NTP Client] Add default empty state strings for remote sections

BUG=656008
==========

[dgn, 2016-10-17 15:33:58]

The CQ bit was checked by dgn@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-10-17 15:34:21]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[dgn, 2016-10-17 15:45:57]

Description was changed from

==========
[NTP Client] Add default empty state strings for remote sections

BUG=656008
==========

to

==========
[NTP Client] Add default empty state strings for remote sections

Preview: https://goo.gl/photos/aK3GPFL5Z7MSzcwn8

BUG=656008
==========

[commit-bot: I haz the power, 2016-10-17 16:02:06]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-10-17 16:02:07]

Dry run: This issue passed the CQ dry run.

[dgn, 2016-10-17 16:31:21]

dgn@chromium.org changed reviewers:
+ treib@chromium.org

[dgn, 2016-10-17 16:31:21]

PTAL

[dgn, 2016-10-17 16:31:22]

PTAL

[Marc Treib, 2016-10-18 08:00:08]

https://codereview.chromium.org/2426663002/diff/1/components/ntp_snippets_str...
File components/ntp_snippets_strings.grdp (right):

https://codereview.chromium.org/2426663002/diff/1/components/ntp_snippets_str...
components/ntp_snippets_strings.grdp:15: Your suggestions appear here
The other two "empty" strings have a dot at the end, probably this one should
too?

Also, the other two say "will appear", but this one is in present tense. Is this
what everyone agreed on? It's not quite clear to me from the bug.

[dgn, 2016-10-18 09:05:44]

https://codereview.chromium.org/2426663002/diff/1/components/ntp_snippets_str...
File components/ntp_snippets_strings.grdp (right):

https://codereview.chromium.org/2426663002/diff/1/components/ntp_snippets_str...
components/ntp_snippets_strings.grdp:15: Your suggestions appear here
On 2016/10/18 08:00:08, Marc Treib wrote:
> The other two "empty" strings have a dot at the end, probably this one should
> too?
> 
> Also, the other two say "will appear", but this one is in present tense. Is
this
> what everyone agreed on? It's not quite clear to me from the bug.

I had a chat with Rachel about this. The current wording is to be consistent
with  https://codereview.chromium.org/2424033002/ and with the approved UI:
https://docs.google.com/presentation/d/1IT0NPdbZqV_uumideOjv9IuEjyqpDHfEWYZkX...

[Marc Treib, 2016-10-18 09:09:21]

Ah I see - didn't know we were updating the other strings too. LGTM then! :)

[dgn, 2016-10-18 09:19:11]

The CQ bit was checked by dgn@chromium.org

[commit-bot: I haz the power, 2016-10-18 09:19:38]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-10-18 09:24:32]

Description was changed from

==========
[NTP Client] Add default empty state strings for remote sections

Preview: https://goo.gl/photos/aK3GPFL5Z7MSzcwn8

BUG=656008
==========

to

==========
[NTP Client] Add default empty state strings for remote sections

Preview: https://goo.gl/photos/aK3GPFL5Z7MSzcwn8

BUG=656008
==========

[commit-bot: I haz the power, 2016-10-18 09:24:33]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-10-18 09:26:13]

Description was changed from

==========
[NTP Client] Add default empty state strings for remote sections

Preview: https://goo.gl/photos/aK3GPFL5Z7MSzcwn8

BUG=656008
==========

to

==========
[NTP Client] Add default empty state strings for remote sections

Preview: https://goo.gl/photos/aK3GPFL5Z7MSzcwn8

BUG=656008

Committed: https://crrev.com/b48353878e2fcf4d3223daed5d5c13e5bbcf2e04
Cr-Commit-Position: refs/heads/master@{#425932}
==========

[commit-bot: I haz the power, 2016-10-18 09:26:14]

Patchset 1 (id:??) landed as
https://crrev.com/b48353878e2fcf4d3223daed5d5c13e5bbcf2e04
Cr-Commit-Position: refs/heads/master@{#425932}