[TTS] Add command to open Touch to search preferences.

[TTS] Add command to open Touch to search preferences.

BUG=503585
Committed: https://crrev.com/95e2926b29b8d8c7390ea64b73e98f978b6ea0ff
Cr-Commit-Position: refs/heads/master@{#362969}

[Olivier, 2015-12-03 12:37:05]

olivierrobin@chromium.org changed reviewers:
+ droger@chromium.org, marq@chromium.org

[Olivier, 2015-12-03 12:37:05]

Add command to open TTS settings.
Please note that format is not what git cl format does, but this is to conform
with surrounding format.

[droger, 2015-12-03 13:05:00]

lgtm

I think there is a typo in the CL title.

https://codereview.chromium.org/1498813002/diff/1/ios/chrome/browser/ui/comma...
File ios/chrome/browser/ui/commands/ios_command_ids.h (right):

https://codereview.chromium.org/1498813002/diff/1/ios/chrome/browser/ui/comma...
ios/chrome/browser/ui/commands/ios_command_ids.h:60: #define
IDC_SHOW_CONTEXTUAL_SEARCH_SETTINGS            40949
Could you use 40919?

[marq (ping after 24h), 2015-12-03 13:07:54]

lgtm

[Olivier, 2015-12-03 13:07:56]

Description was changed from

==========
[TTS] Add command to open Touth to search preferences.

BUG=503585
==========

to

==========
[TTS] Add command to open Touch to search preferences.

BUG=503585
==========

[Olivier, 2015-12-03 13:09:59]

Thanks. Done

https://codereview.chromium.org/1498813002/diff/1/ios/chrome/browser/ui/comma...
File ios/chrome/browser/ui/commands/ios_command_ids.h (right):

https://codereview.chromium.org/1498813002/diff/1/ios/chrome/browser/ui/comma...
ios/chrome/browser/ui/commands/ios_command_ids.h:60: #define
IDC_SHOW_CONTEXTUAL_SEARCH_SETTINGS            40949
On 2015/12/03 13:05:00, droger wrote:
> Could you use 40919?

Done.

[Olivier, 2015-12-03 13:10:04]

The CQ bit was checked by olivierrobin@chromium.org

[Olivier, 2015-12-03 13:10:04]

The patchset sent to the CQ was uploaded after l-g-t-m from droger@chromium.org,
marq@chromium.org
Link to the patchset: https://codereview.chromium.org/1498813002/#ps20001
(title: "40919")

[commit-bot: I haz the power, 2015-12-03 13:10:22]

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

[commit-bot: I haz the power, 2015-12-03 13:21:20]

Description was changed from

==========
[TTS] Add command to open Touch to search preferences.

BUG=503585
==========

to

==========
[TTS] Add command to open Touch to search preferences.

BUG=503585
==========

[commit-bot: I haz the power, 2015-12-03 13:21:20]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-12-03 13:22:19]

Description was changed from

==========
[TTS] Add command to open Touch to search preferences.

BUG=503585
==========

to

==========
[TTS] Add command to open Touch to search preferences.

BUG=503585

Committed: https://crrev.com/95e2926b29b8d8c7390ea64b73e98f978b6ea0ff
Cr-Commit-Position: refs/heads/master@{#362969}
==========

[commit-bot: I haz the power, 2015-12-03 13:22:20]

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