Add warning message when disabling ARC.

Add warning message when disabling ARC.

BUG=610941
TEST=cros autotest to be submitted in
https://chrome-internal-review.googlesource.com/#/c/262336
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:closure_compilation

Committed: https://crrev.com/2b1b13921cd0291635484827eba0d879926365ec
Cr-Commit-Position: refs/heads/master@{#399824}

[dspaid, 2016-05-25 03:50:49]

Description was changed from

==========
Add warning message when disabling ARC.

BUG=610941
TEST=Manually verified
==========

to

==========
Add warning message when disabling ARC.

BUG=610941
TEST=Manually verified
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:closure_compilation
==========

[dspaid, 2016-06-03 06:24:00]

Description was changed from

==========
Add warning message when disabling ARC.

BUG=610941
TEST=Manually verified
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:closure_compilation
==========

to

==========
Add warning message when disabling ARC.

BUG=610941
TEST=cros autotest to be submitted in
https://chrome-internal-review.googlesource.com/#/c/262336
==========

[dspaid, 2016-06-03 06:58:13]

dspaid@chromium.org changed reviewers:
+ stevenjb@chromium.org

[dspaid, 2016-06-03 07:02:32]

Description was changed from

==========
Add warning message when disabling ARC.

BUG=610941
TEST=cros autotest to be submitted in
https://chrome-internal-review.googlesource.com/#/c/262336
==========

to

==========
Add warning message when disabling ARC.

BUG=610941
TEST=cros autotest to be submitted in
https://chrome-internal-review.googlesource.com/#/c/262336
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:closure_compilation
==========

[stevenjb, 2016-06-08 00:41:42]

lgtm w/ text change suggestion

https://codereview.chromium.org/2002253004/diff/60001/chrome/app/chromeos_str...
File chrome/app/chromeos_strings.grdp (right):

https://codereview.chromium.org/2002253004/diff/60001/chrome/app/chromeos_str...
chrome/app/chromeos_strings.grdp:6613: Apps you've downloaded from the Play
Store, along with all app data, will be deleted from this Chromebook. This
cannot be undone. This doesn't affect apps from the Play Store on your Android
devices and other Chromebooks.
You might want to run this by UX, but I believe we generally avoid the use of
"you". I would just remove it, i.e. "Apps downloaded from the Play Store"

[dspaid, 2016-06-08 02:31:55]

dspaid@chromium.org changed reviewers:
+ mitsuji@google.com

[dspaid, 2016-06-08 02:31:56]

Hiro,
Any comments on the suggested text change?  This is based on your mocks.

[dspaid, 2016-06-08 22:51:25]

On 2016/06/08 00:41:42, stevenjb wrote:
> lgtm w/ text change suggestion
> 
>
https://codereview.chromium.org/2002253004/diff/60001/chrome/app/chromeos_str...
> File chrome/app/chromeos_strings.grdp (right):
> 
>
https://codereview.chromium.org/2002253004/diff/60001/chrome/app/chromeos_str...
> chrome/app/chromeos_strings.grdp:6613: Apps you've downloaded from the Play
> Store, along with all app data, will be deleted from this Chromebook. This
> cannot be undone. This doesn't affect apps from the Play Store on your Android
> devices and other Chromebooks.
> You might want to run this by UX, but I believe we generally avoid the use of
> "you". I would just remove it, i.e. "Apps downloaded from the Play Store"

Updated

[mitsuji, 2016-06-09 21:02:27]

On 2016/06/08 22:51:25, dspaid wrote:
> On 2016/06/08 00:41:42, stevenjb wrote:
> > lgtm w/ text change suggestion
> > 
> >
>
https://codereview.chromium.org/2002253004/diff/60001/chrome/app/chromeos_str...
> > File chrome/app/chromeos_strings.grdp (right):
> > 
> >
>
https://codereview.chromium.org/2002253004/diff/60001/chrome/app/chromeos_str...
> > chrome/app/chromeos_strings.grdp:6613: Apps you've downloaded from the Play
> > Store, along with all app data, will be deleted from this Chromebook. This
> > cannot be undone. This doesn't affect apps from the Play Store on your
Android
> > devices and other Chromebooks.
> > You might want to run this by UX, but I believe we generally avoid the use
of
> > "you". I would just remove it, i.e. "Apps downloaded from the Play Store"
> 
> Updated

Thanks for checking. I've verified the changes with our copy team. Good to go.

[mitsuji, 2016-06-09 21:02:30]

On 2016/06/08 22:51:25, dspaid wrote:
> On 2016/06/08 00:41:42, stevenjb wrote:
> > lgtm w/ text change suggestion
> > 
> >
>
https://codereview.chromium.org/2002253004/diff/60001/chrome/app/chromeos_str...
> > File chrome/app/chromeos_strings.grdp (right):
> > 
> >
>
https://codereview.chromium.org/2002253004/diff/60001/chrome/app/chromeos_str...
> > chrome/app/chromeos_strings.grdp:6613: Apps you've downloaded from the Play
> > Store, along with all app data, will be deleted from this Chromebook. This
> > cannot be undone. This doesn't affect apps from the Play Store on your
Android
> > devices and other Chromebooks.
> > You might want to run this by UX, but I believe we generally avoid the use
of
> > "you". I would just remove it, i.e. "Apps downloaded from the Play Store"
> 
> Updated

Thanks for checking. I've verified the changes with our copy team. Good to go.

[dspaid, 2016-06-14 22:52:52]

The CQ bit was checked by dspaid@chromium.org

[dspaid, 2016-06-14 22:52:55]

The patchset sent to the CQ was uploaded after l-g-t-m from
stevenjb@chromium.org
Link to the patchset: https://codereview.chromium.org/2002253004/#ps160001
(title: "fix css copy detection")

[commit-bot: I haz the power, 2016-06-14 22:53:20]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/2002253004/160001

[commit-bot: I haz the power, 2016-06-15 00:54:57]

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

[commit-bot: I haz the power, 2016-06-15 00:54:59]

Try jobs failed on following builders:
  ios-device on tryserver.chromium.mac (JOB_TIMED_OUT, no build URL)
  ios-device-gn on tryserver.chromium.mac (JOB_TIMED_OUT, no build URL)

[dspaid, 2016-06-15 00:55:59]

The CQ bit was checked by dspaid@chromium.org

[commit-bot: I haz the power, 2016-06-15 00:56:30]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/2002253004/160001

[commit-bot: I haz the power, 2016-06-15 02:53:44]

Committed patchset #9 (id:160001)

[commit-bot: I haz the power, 2016-06-15 02:56:49]

Description was changed from

==========
Add warning message when disabling ARC.

BUG=610941
TEST=cros autotest to be submitted in
https://chrome-internal-review.googlesource.com/#/c/262336
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:closure_compilation
==========

to

==========
Add warning message when disabling ARC.

BUG=610941
TEST=cros autotest to be submitted in
https://chrome-internal-review.googlesource.com/#/c/262336
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:closure_compilation

Committed: https://crrev.com/2b1b13921cd0291635484827eba0d879926365ec
Cr-Commit-Position: refs/heads/master@{#399824}
==========

[commit-bot: I haz the power, 2016-06-15 02:56:50]

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