[Material][Mac] Remove the colon in the selected keyword decoration

[Material][Mac] Remove the colon in the selected keyword decoration

BUG=615251
Committed: https://crrev.com/dc97fcbf3accecbb6898dca34586c1be33dea7f5
Cr-Commit-Position: refs/heads/master@{#396565}

[spqchan, 2016-05-26 21:15:46]

spqchan@chromium.org changed reviewers:
+ avi@chromium.org, shrike@chromium.org

[spqchan, 2016-05-26 21:15:47]

PTAL

[shrike, 2016-05-26 23:00:59]

LGTM.

Do you want to give it its own separate bug? If not, we have to remember to
cherry pick both of the cls on 611113.

https://codereview.chromium.org/2016893002/diff/1/chrome/browser/ui/cocoa/loc...
File chrome/browser/ui/cocoa/location_bar/selected_keyword_decoration.mm (left):

https://codereview.chromium.org/2016893002/diff/1/chrome/browser/ui/cocoa/loc...
chrome/browser/ui/cocoa/location_bar/selected_keyword_decoration.mm:73: // The
text will be like "Search <name>:".  "<name>" is a parameter
Please update the comment (because in the MD case there is no color).

[spqchan, 2016-05-26 23:04:58]

Description was changed from

==========
[Material][Mac] Remove the colon in the selected keyword decoration

BUG=611113
==========

to

==========
[Material][Mac] Remove the colon in the selected keyword decoration

BUG=615251
==========

[spqchan, 2016-05-27 00:29:51]

Created and updated the new bug

https://codereview.chromium.org/2016893002/diff/1/chrome/browser/ui/cocoa/loc...
File chrome/browser/ui/cocoa/location_bar/selected_keyword_decoration.mm (left):

https://codereview.chromium.org/2016893002/diff/1/chrome/browser/ui/cocoa/loc...
chrome/browser/ui/cocoa/location_bar/selected_keyword_decoration.mm:73: // The
text will be like "Search <name>:".  "<name>" is a parameter
On 2016/05/26 23:00:59, shrike wrote:
> Please update the comment (because in the MD case there is no color).

Done.

[Avi (use Gerrit), 2016-05-27 01:52:51]

lgtm

[spqchan, 2016-05-27 17:01:22]

The CQ bit was checked by spqchan@chromium.org

[spqchan, 2016-05-27 17:01:22]

The patchset sent to the CQ was uploaded after l-g-t-m from shrike@chromium.org
Link to the patchset: https://codereview.chromium.org/2016893002/#ps20001
(title: "fix for shrike")

[commit-bot: I haz the power, 2016-05-27 17:01:50]

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

[commit-bot: I haz the power, 2016-05-27 17:54:07]

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

[commit-bot: I haz the power, 2016-05-27 17:54:08]

Try jobs failed on following builders:
  mac_chromium_gn_rel on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_gn_r...)

[spqchan, 2016-05-27 19:34:43]

The CQ bit was checked by spqchan@chromium.org

[spqchan, 2016-05-27 19:34:44]

The patchset sent to the CQ was uploaded after l-g-t-m from avi@chromium.org,
shrike@chromium.org
Link to the patchset: https://codereview.chromium.org/2016893002/#ps40001
(title: "fixed test")

[commit-bot: I haz the power, 2016-05-27 19:35:06]

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

[commit-bot: I haz the power, 2016-05-27 21:26:13]

Description was changed from

==========
[Material][Mac] Remove the colon in the selected keyword decoration

BUG=615251
==========

to

==========
[Material][Mac] Remove the colon in the selected keyword decoration

BUG=615251
==========

[commit-bot: I haz the power, 2016-05-27 21:26:15]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-05-27 21:27:42]

Description was changed from

==========
[Material][Mac] Remove the colon in the selected keyword decoration

BUG=615251
==========

to

==========
[Material][Mac] Remove the colon in the selected keyword decoration

BUG=615251

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

[commit-bot: I haz the power, 2016-05-27 21:27:43]

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