MD Settings: Close action menu when search engine is removed.

MD Settings: Close action menu when search engine is removed.

BUG=657254, 633898
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

Committed: https://crrev.com/9d8c18b0b6e89c707733bc6fbac6a4195f8aedd9
Cr-Commit-Position: refs/heads/master@{#426297}

[dpapad, 2016-10-19 17:03:53]

Description was changed from

==========
MD Settings: Close action menu when search engine is removed.

BUG=657254,633898
==========

to

==========
MD Settings: Close action menu when search engine is removed.

BUG=657254,633898
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

[dpapad, 2016-10-19 17:15:35]

Patchset #1 (id:1) has been deleted

[dpapad, 2016-10-19 17:15:42]

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

[commit-bot: I haz the power, 2016-10-19 17:16:15]

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

[dpapad, 2016-10-19 18:05:45]

dpapad@chromium.org changed reviewers:
+ hcarmona@chromium.org

[hcarmona, 2016-10-19 18:10:31]

lgtm

https://codereview.chromium.org/2425423005/diff/40001/chrome/test/data/webui/...
File chrome/test/data/webui/settings/search_engines_page_test.js (right):

https://codereview.chromium.org/2425423005/diff/40001/chrome/test/data/webui/...
chrome/test/data/webui/settings/search_engines_page_test.js:183: 
extra space (sorry for nittiest of nits!)

[dpapad, 2016-10-19 18:53:22]

https://codereview.chromium.org/2425423005/diff/40001/chrome/test/data/webui/...
File chrome/test/data/webui/settings/search_engines_page_test.js (right):

https://codereview.chromium.org/2425423005/diff/40001/chrome/test/data/webui/...
chrome/test/data/webui/settings/search_engines_page_test.js:183: 
On 2016/10/19 at 18:10:31, Hector Carmona wrote:
> extra space (sorry for nittiest of nits!)

Done, no problem!

[dpapad, 2016-10-19 18:55:06]

The CQ bit was checked by dpapad@chromium.org

[dpapad, 2016-10-19 18:55:06]

The patchset sent to the CQ was uploaded after l-g-t-m from
hcarmona@chromium.org
Link to the patchset: https://codereview.chromium.org/2425423005/#ps60001
(title: "Nit")

[commit-bot: I haz the power, 2016-10-19 18:55:57]

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, 2016-10-19 21:57:43]

Committed patchset #3 (id:60001)

[commit-bot: I haz the power, 2016-10-21 13:11:54]

Description was changed from

==========
MD Settings: Close action menu when search engine is removed.

BUG=657254,633898
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

to

==========
MD Settings: Close action menu when search engine is removed.

BUG=657254,633898
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

Committed: https://crrev.com/9d8c18b0b6e89c707733bc6fbac6a4195f8aedd9
Cr-Commit-Position: refs/heads/master@{#426297}
==========

[commit-bot: I haz the power, 2016-10-21 13:11:55]

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