Update content desciption string for bookmark button in NTP toolbar

Update content desciption string for bookmark button in NTP toolbar

BUG=557900
Committed: https://crrev.com/44d232c33e46199f9247a68bd6cc368374367c16
Cr-Commit-Position: refs/heads/master@{#360972}

[jianli, 2015-11-20 23:58:06]

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

[newt (away), 2015-11-21 00:03:33]

lgtm with suggestion

https://codereview.chromium.org/1470593003/diff/1/chrome/android/java/src/org...
File
chrome/android/java/src/org/chromium/chrome/browser/ntp/NewTabPageToolbar.java
(right):

https://codereview.chromium.org/1470593003/diff/1/chrome/android/java/src/org...
chrome/android/java/src/org/chromium/chrome/browser/ntp/NewTabPageToolbar.java:46:
((TextView)
mBookmarksButton.getChildAt(0)).setText(OfflinePageBridge.isEnabled()
nit: I think it'd be clearer to rewrite this with an if statement

  if (OfflinePageBridge.isEnabled()) {
      ((TextView) mBookmarksButton.getChildAt(0)).setText(
              R.string.offline_pages_ntp_button_name);
      ((TextView) mBookmarksButton.getChildAt(0)).setContentDescription(
              R.string.offline_pages_ntp_button_accessibility);
  }

[jianli, 2015-11-21 00:48:08]

The CQ bit was checked by jianli@chromium.org

[jianli, 2015-11-21 00:48:09]

The patchset sent to the CQ was uploaded after l-g-t-m from newt@chromium.org
Link to the patchset: https://codereview.chromium.org/1470593003/#ps20001
(title: "Address feedback")

[commit-bot: I haz the power, 2015-11-21 00:49:21]

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

[commit-bot: I haz the power, 2015-11-21 01:05:27]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-11-21 01:06:21]

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