Remove SearchButton.

Remove SearchButton.

As our old friend OriginChip, so the SearchButton too has to bid adieu.
She served well, and we shall miss her.

BUG=424751
Committed: https://crrev.com/dcf300ae062ae29c02bba5b0559d006b24814867
Cr-Commit-Position: refs/heads/master@{#316302}

[groby-ooo-7-16, 2014-12-02 21:18:10]

groby@chromium.org changed reviewers:
+ jdonnelly@chromium.org, pkasting@chromium.org

[groby-ooo-7-16, 2014-12-02 21:18:10]

PTAL at SearchButton removal as well.

Note: Did most of the digging for references via ag - here's the cmd:

ag --ignore-dir=chrome/test --ignore-dir=tools/perf --ignore-dir=ui/file_manager
--ignore-dir=ui/views/focus --ignore-dir=chrome/app/theme
--ignore-dir=bookmark_manager_private --ignore-dir=history/
--ignore-dir=downloads/ -i '(?<!MIC_)(?<!HISTORY_)(?<!DOWNLOAD_)search_?button'

[groby-ooo-7-16, 2014-12-02 22:41:57]

If you'd rather review a single CL for OriginChip and SearchButton:
https://codereview.chromium.org/768413003/

[Justin Donnelly, 2014-12-03 16:08:33]

LGTM

Made some comments on the merged CL, but in general lgtm.

[groby-ooo-7-16, 2015-01-10 02:18:19]

groby@chromium.org changed reviewers:
+ kmadhusu@chromium.org

[groby-ooo-7-16, 2015-01-10 02:18:32]

kmadhusu: Please review c/b/search.

[kmadhusu, 2015-01-12 17:49:27]

On 2015/01/10 02:18:32, groby wrote:
> kmadhusu: Please review c/b/search.

lgm

[groby-ooo-7-16, 2015-02-12 03:11:53]

kmadhusu: Please LG (previous one was not parsed due to misspelling, no new
changes)

pkasting: Part 2 of the removal CLs

[Peter Kasting, 2015-02-12 07:07:14]

Rubber-stamp LGTM (I assume you'll land the origin chip change first and then
run tryjobs on this or something)

[groby-ooo-7-16, 2015-02-13 02:58:18]

Yep, that's the plan.

On Wed, Feb 11, 2015 at 11:07 PM, <pkasting@chromium.org> wrote:

> Rubber-stamp LGTM (I assume you'll land the origin chip change first and
> then
> run tryjobs on this or something)
>
> https://codereview.chromium.org/749693005/
>

To unsubscribe from this group and stop receiving emails from it, send an email
to chromium-reviews+unsubscribe@chromium.org.

[groby-ooo-7-16, 2015-02-13 20:10:58]

New patchsets have been uploaded after l-g-t-m from pkasting@chromium.org

[groby-ooo-7-16, 2015-02-13 20:58:46]

kmadhusu: Mind LG'ing again so this can land?

[kmadhusu, 2015-02-13 21:41:18]

On 2015/02/13 20:58:46, groby wrote:
> kmadhusu: Mind LG'ing again so this can land?

LGTM.

[groby-ooo-7-16, 2015-02-13 21:56:41]

The CQ bit was checked by groby@chromium.org

[commit-bot: I haz the power, 2015-02-13 21:57:43]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/749693005/100001

[commit-bot: I haz the power, 2015-02-13 22:02:35]

Committed patchset #6 (id:100001)

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

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