Update settings string for enabling network predictions.

Update settings string for enabling network predictions.

BUG=585297
Committed: https://crrev.com/e40cf6154c893214099ac7ad0b8b4d4babbd3cd5
Cr-Commit-Position: refs/heads/master@{#377336}

[newt (away), 2016-02-23 22:24:19]

newt@chromium.org changed reviewers:
+ igrigorik@chromium.org

[newt (away), 2016-02-23 22:24:19]

PTAL

[newt (away), 2016-02-23 22:25:15]

newt@chromium.org changed reviewers:
+ blundell@chromium.org

[newt (away), 2016-02-23 22:25:16]

+blundell for components OWNERS

[newt (away), 2016-02-23 22:26:45]

newt@chromium.org changed reviewers:
+ dschuyler@chromium.org

[newt (away), 2016-02-23 22:26:45]

+dschuyler for settings_strings.grdp

[newt (away), 2016-02-23 22:28:17]

igrigorik: Is there a bug I can link to specifically for the string change?

[dschuyler, 2016-02-23 22:35:04]

lgtm

[igrigorik, 2016-02-23 22:48:19]

lgtm

lgtm. One minor suggestion, feel free to ignore.

https://codereview.chromium.org/1724193002/diff/1/chrome/app/settings_strings...
File chrome/app/settings_strings.grdp (right):

https://codereview.chromium.org/1724193002/diff/1/chrome/app/settings_strings...
chrome/app/settings_strings.grdp:407: <message
name="IDS_SETTINGS_NETWORK_PREDICTION_ENABLED_DESCRIPTION" desc="In the advanced
options tab, the text next to the checkbox that enables prediction of network
actions.  Actions include DNS prefetching, TCP and SSL preconnection, and
prerendering of webpages.">
For posterity, is it worth clarifying.. "Actions include browser-initiated DNS
prefetching, ...."?

https://codereview.chromium.org/1724193002/diff/1/components/components_setti...
File components/components_settings_strings.grdp (right):

https://codereview.chromium.org/1724193002/diff/1/components/components_setti...
components/components_settings_strings.grdp:12: <message
name="IDS_NETWORK_PREDICTION_ENABLED_DESCRIPTION" desc="In the advanced options
tab, the text next to the checkbox that enables prediction of network actions. 
Actions include DNS prefetching, TCP and SSL preconnection, and prerendering of
webpages.">
Ditto.

[igrigorik, 2016-02-23 22:48:58]

On 2016/02/23 22:28:17, newt wrote:
> igrigorik: Is there a bug I can link to specifically for the string change?

No, we don't have  meta-bug for this.

[newt (away), 2016-02-23 23:02:05]

https://codereview.chromium.org/1724193002/diff/1/chrome/app/settings_strings...
File chrome/app/settings_strings.grdp (right):

https://codereview.chromium.org/1724193002/diff/1/chrome/app/settings_strings...
chrome/app/settings_strings.grdp:407: <message
name="IDS_SETTINGS_NETWORK_PREDICTION_ENABLED_DESCRIPTION" desc="In the advanced
options tab, the text next to the checkbox that enables prediction of network
actions.  Actions include DNS prefetching, TCP and SSL preconnection, and
prerendering of webpages.">
On 2016/02/23 22:48:18, igrigorik wrote:
> For posterity, is it worth clarifying.. "Actions include browser-initiated DNS
> prefetching, ...."?

Sure.

https://codereview.chromium.org/1724193002/diff/1/components/components_setti...
File components/components_settings_strings.grdp (right):

https://codereview.chromium.org/1724193002/diff/1/components/components_setti...
components/components_settings_strings.grdp:12: <message
name="IDS_NETWORK_PREDICTION_ENABLED_DESCRIPTION" desc="In the advanced options
tab, the text next to the checkbox that enables prediction of network actions. 
Actions include DNS prefetching, TCP and SSL preconnection, and prerendering of
webpages.">
On 2016/02/23 22:48:19, igrigorik wrote:
> Ditto.

Done.

[blundell, 2016-02-24 12:25:27]

lgtm

[newt (away), 2016-02-24 17:39:50]

The CQ bit was checked by newt@chromium.org

[newt (away), 2016-02-24 17:39:50]

The patchset sent to the CQ was uploaded after l-g-t-m from
igrigorik@chromium.org, dschuyler@chromium.org
Link to the patchset: https://codereview.chromium.org/1724193002/#ps20001
(title: "igrigorik's comments")

[commit-bot: I haz the power, 2016-02-24 17:42:32]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1724193002/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1724193002/20001

[commit-bot: I haz the power, 2016-02-24 18:59:46]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-02-24 19:01:46]

Description was changed from

==========
Update settings string for enabling network predictions.

BUG=585297
==========

to

==========
Update settings string for enabling network predictions.

BUG=585297

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

[commit-bot: I haz the power, 2016-02-24 19:01:47]

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