Small cleanup in SearchTabHelper

Small cleanup in SearchTabHelper

Remove SearchTabHelper::SupportsInstant - it's only used by tests, so move it into the test.

BUG=627747
Committed: https://crrev.com/89075d4cbd441f51ff1c38884be566fba1822ca9
Cr-Commit-Position: refs/heads/master@{#437238}

[Marc Treib, 2016-09-27 08:59:51]

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

[commit-bot: I haz the power, 2016-09-27 09:00:08]

Dry run: 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-09-27 09:22:17]

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

[commit-bot: I haz the power, 2016-09-27 09:22:17]

Dry run: Try jobs failed on following builders:
  android_arm64_dbg_recipe on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_arm6...)

[Marc Treib, 2016-12-08 13:02:47]

Description was changed from

==========
Small cleanup in SearchTabHelper

BUG=627747
==========

to

==========
Small cleanup in SearchTabHelper

Remove SearchTabHelper::SupportsInstant - it's only used by tests, so move it
into the test.

BUG=627747
==========

[Marc Treib, 2016-12-08 13:03:28]

treib@chromium.org changed reviewers:
+ sfiera@chromium.org

[Marc Treib, 2016-12-08 13:03:29]

PTAL!

[sfiera, 2016-12-08 13:12:38]

LGTM

https://codereview.chromium.org/2376633002/diff/20001/chrome/browser/ui/searc...
File chrome/browser/ui/search/search_tab_helper.h (left):

https://codereview.chromium.org/2376633002/diff/20001/chrome/browser/ui/searc...
chrome/browser/ui/search/search_tab_helper.h:10: #include
"base/compiler_specific.h"
wut

[Marc Treib, 2016-12-08 13:15:28]

https://codereview.chromium.org/2376633002/diff/20001/chrome/browser/ui/searc...
File chrome/browser/ui/search/search_tab_helper.h (left):

https://codereview.chromium.org/2376633002/diff/20001/chrome/browser/ui/searc...
chrome/browser/ui/search/search_tab_helper.h:10: #include
"base/compiler_specific.h"
On 2016/12/08 13:12:37, sfiera wrote:
> wut

I have no idea why this was here. :)

[Marc Treib, 2016-12-08 13:15:32]

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

[commit-bot: I haz the power, 2016-12-08 13:15:45]

Dry run: 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-12-08 13:59:38]

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

[commit-bot: I haz the power, 2016-12-08 13:59:39]

Dry run: This issue passed the CQ dry run.

[Marc Treib, 2016-12-08 13:59:51]

The CQ bit was checked by treib@chromium.org

[commit-bot: I haz the power, 2016-12-08 14:00:12]

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-12-08 14:04:10]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1481205591288820,
"parent_rev": "dcc34984a039f05913677dfbb685ce55b7c116e0", "commit_rev":
"6b9357df4dafcc52dafbd450a1ce6e31a7bef076"}

[commit-bot: I haz the power, 2016-12-08 14:04:34]

Description was changed from

==========
Small cleanup in SearchTabHelper

Remove SearchTabHelper::SupportsInstant - it's only used by tests, so move it
into the test.

BUG=627747
==========

to

==========
Small cleanup in SearchTabHelper

Remove SearchTabHelper::SupportsInstant - it's only used by tests, so move it
into the test.

BUG=627747
==========

[commit-bot: I haz the power, 2016-12-08 14:04:35]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-12-08 14:06:33]

Description was changed from

==========
Small cleanup in SearchTabHelper

Remove SearchTabHelper::SupportsInstant - it's only used by tests, so move it
into the test.

BUG=627747
==========

to

==========
Small cleanup in SearchTabHelper

Remove SearchTabHelper::SupportsInstant - it's only used by tests, so move it
into the test.

BUG=627747

Committed: https://crrev.com/89075d4cbd441f51ff1c38884be566fba1822ca9
Cr-Commit-Position: refs/heads/master@{#437238}
==========

[commit-bot: I haz the power, 2016-12-08 14:06:34]

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