Change strings for update permissions infobar.

Change strings for update permissions infobar.

The strings were brief but confusing.

BUG=710362
Review-Url: https://codereview.chromium.org/2869763002
Cr-Commit-Position: refs/heads/master@{#473488}
Committed: https://chromium.googlesource.com/chromium/src/+/36ecf1abe91cd116d144aeb82b821bd201587916

[benwells, 2017-05-08 12:36:10]

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

[commit-bot: I haz the power, 2017-05-08 12:36:25]

Dry run: CQ is trying da patch.

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

[benwells, 2017-05-08 12:55:00]

benwells@chromium.org changed reviewers:
+ raymes@chromium.org

[benwells, 2017-05-08 12:55:01]

I'll be tackling the multiple permissions string (the worst) in a separate CL,
as we'll be changing the way it works.

[commit-bot: I haz the power, 2017-05-08 13:23:42]

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

[commit-bot: I haz the power, 2017-05-08 13:23:43]

Dry run: This issue passed the CQ dry run.

[raymes, 2017-05-08 22:19:44]

lgtm

https://codereview.chromium.org/2869763002/diff/1/chrome/android/java/strings...
File chrome/android/java/strings/android_chrome_strings.grd (right):

https://codereview.chromium.org/2869763002/diff/1/chrome/android/java/strings...
chrome/android/java/strings/android_chrome_strings.grd:1640: Chrome needs access
to to your location to share your location with this site.
I don't quite understand where access is more appropriate than permission, what
was the reason? I see you've already talked to UX about it though so don't want
to block :)

[benwells, 2017-05-09 07:41:10]

https://codereview.chromium.org/2869763002/diff/1/chrome/android/java/strings...
File chrome/android/java/strings/android_chrome_strings.grd (right):

https://codereview.chromium.org/2869763002/diff/1/chrome/android/java/strings...
chrome/android/java/strings/android_chrome_strings.grd:1640: Chrome needs access
to to your location to share your location with this site.
On 2017/05/08 22:19:44, raymes wrote:
> I don't quite understand where access is more appropriate than permission,
what
> was the reason? I see you've already talked to UX about it though so don't
want
> to block :)

Yeah I was wondering that too. It is kind of strange, and might lead to
wierdness in translations. I'll ask the UI string person...

[benwells, 2017-05-17 03:57:56]

The mic and camera strings, as well as the button, have all been revised. Please
have another look before I send to the android string owner.

https://codereview.chromium.org/2869763002/diff/1/chrome/android/java/strings...
File chrome/android/java/strings/android_chrome_strings.grd (right):

https://codereview.chromium.org/2869763002/diff/1/chrome/android/java/strings...
chrome/android/java/strings/android_chrome_strings.grd:1640: Chrome needs access
to to your location to share your location with this site.
On 2017/05/09 07:41:10, benwells wrote:
> On 2017/05/08 22:19:44, raymes wrote:
> > I don't quite understand where access is more appropriate than permission,
> what
> > was the reason? I see you've already talked to UX about it though so don't
> want
> > to block :)
> 
> Yeah I was wondering that too. It is kind of strange, and might lead to
> wierdness in translations. I'll ask the UI string person...

Apparently it was deliberate, to quote srahim@: "Yes, it was deliberate, thanks
for checking. I was trying to keep the word count manageable. Unlike the other
permissions, the concept of "sharing location" is important here."

[raymes, 2017-05-17 04:46:43]

lgtm

[benwells, 2017-05-17 04:47:21]

benwells@chromium.org changed reviewers:
+ tedchoc@chromium.org

[benwells, 2017-05-17 04:47:21]

+tedchoc for android_chrome_strings owner

[Ted C, 2017-05-17 13:59:20]

lgtm

[benwells, 2017-05-17 23:19:09]

The CQ bit was checked by benwells@chromium.org

[commit-bot: I haz the power, 2017-05-17 23:20:45]

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, 2017-05-18 05:28:15]

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

[commit-bot: I haz the power, 2017-05-18 05:28:16]

Try jobs failed on following builders:
  win_chromium_x64_rel_ng on master.tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_x64_...)

[benwells, 2017-05-19 20:26:37]

The CQ bit was checked by benwells@chromium.org

[commit-bot: I haz the power, 2017-05-19 20:27:41]

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, 2017-05-19 23:30:02]

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

[commit-bot: I haz the power, 2017-05-19 23:30:04]

Try jobs failed on following builders:
  linux_android_rel_ng on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/linux_androi...)

[benwells, 2017-05-21 22:19:49]

The CQ bit was checked by benwells@chromium.org

[commit-bot: I haz the power, 2017-05-21 22:20:03]

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, 2017-05-21 23:03:48]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1495405189617770,
"parent_rev": "be827b4b23c8e28143f175ebb4d219065e72a607", "commit_rev":
"36ecf1abe91cd116d144aeb82b821bd201587916"}

[commit-bot: I haz the power, 2017-05-21 23:03:59]

Description was changed from

==========
Change strings for update permissions infobar.

The strings were brief but confusing.

BUG=710362
==========

to

==========
Change strings for update permissions infobar.

The strings were brief but confusing.

BUG=710362

Review-Url: https://codereview.chromium.org/2869763002
Cr-Commit-Position: refs/heads/master@{#473488}
Committed:
https://chromium.googlesource.com/chromium/src/+/36ecf1abe91cd116d144aeb82b82...
==========

[commit-bot: I haz the power, 2017-05-21 23:04:02]

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/chromium/src/+/36ecf1abe91cd116d144aeb82b82...