Fixing display issue in bookmark manager.

Fixing display issue in bookmark manager.

Splitting bookmark strings into two sets, the original strings and the new strings, because the original strings are still needed for the bookmark manager until how to get the pluralization to work is figured out.

BUG=708815, 712995
Review-Url: https://codereview.chromium.org/2828873003
Cr-Commit-Position: refs/heads/master@{#466038}
Committed: https://chromium.googlesource.com/chromium/src/+/c32cfc7c2ca9fbc0c289dba04d7878ea8a65d7e9

[Paezagon, 2017-04-19 22:19:02]

paezagon@chromium.org changed reviewers:
+ pkasting@chromium.org

[Paezagon, 2017-04-19 22:19:03]

Hello,

Small patch fixing visual bug related to Issue 708815.

Thanks,
Anna

[Peter Kasting, 2017-04-20 00:01:59]

This is a temporary hack, right?

LGTM

https://codereview.chromium.org/2828873003/diff/1/chrome/app/bookmarks_string...
File chrome/app/bookmarks_strings.grdp (right):

https://codereview.chromium.org/2828873003/diff/1/chrome/app/bookmarks_string...
chrome/app/bookmarks_strings.grdp:67: &Open all bookmark
I think this should be "bookmarks"

[Paezagon, 2017-04-20 15:54:02]

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

[commit-bot: I haz the power, 2017-04-20 15:54:26]

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

[Paezagon, 2017-04-20 15:56:54]

Yes, this is a temporary hack.

https://codereview.chromium.org/2828873003/diff/1/chrome/app/bookmarks_string...
File chrome/app/bookmarks_strings.grdp (right):

https://codereview.chromium.org/2828873003/diff/1/chrome/app/bookmarks_string...
chrome/app/bookmarks_strings.grdp:67: &Open all bookmark
On 2017/04/20 00:01:59, Peter Kasting (OOO until 4-24) wrote:
> I think this should be "bookmarks"

Done.

[commit-bot: I haz the power, 2017-04-20 16:41:23]

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

[commit-bot: I haz the power, 2017-04-20 16:41:24]

Dry run: This issue passed the CQ dry run.

[Paezagon, 2017-04-20 16:45:22]

The CQ bit was checked by paezagon@chromium.org

[Paezagon, 2017-04-20 16:45:22]

The patchset sent to the CQ was uploaded after l-g-t-m from
pkasting@chromium.org
Link to the patchset: https://codereview.chromium.org/2828873003/#ps20001
(title: "Fixing grammer mistake")

[commit-bot: I haz the power, 2017-04-20 16:45:51]

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-04-20 17:16:15]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1492706722505000,
"parent_rev": "3194254dc652c6ca45f7f7b11685ccd03252ea6b", "commit_rev":
"c32cfc7c2ca9fbc0c289dba04d7878ea8a65d7e9"}

[commit-bot: I haz the power, 2017-04-20 17:16:25]

Description was changed from

==========
Fixing display issue in bookmark manager.

Splitting bookmark strings into two sets, the original strings and the new
strings, because the original strings are still needed for the bookmark manager
until how to get the pluralization to work is figured out.

BUG=708815, 712995
==========

to

==========
Fixing display issue in bookmark manager.

Splitting bookmark strings into two sets, the original strings and the new
strings, because the original strings are still needed for the bookmark manager
until how to get the pluralization to work is figured out.

BUG=708815, 712995

Review-Url: https://codereview.chromium.org/2828873003
Cr-Commit-Position: refs/heads/master@{#466038}
Committed:
https://chromium.googlesource.com/chromium/src/+/c32cfc7c2ca9fbc0c289dba04d78...
==========

[commit-bot: I haz the power, 2017-04-20 17:16:27]

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