views: remove the last function abstraction from browser_dialogs.h

views: remove the last function abstraction from browser_dialogs.h

After commit faed95410a75: ("views: remove unnecessary EditSearchEngine()
function abstraction"), only CreateFindBar() function abstraction
remained in browser_dialogs.h, but like EditSearchEngine(), it is
unlikely this function will be used to be called by cross-platform
code, so it is not necessary keep it wrapping FindBarHost.

BrowserView can just depend on it like it does with many other
chrome/browser/ui/views/ header files already. This is also what
BrowserWindowCocoa does as well.

This in effect get rid of browser_dialogs.h header file finally.

BUG=None
R=sky@chromium.org

Committed: https://crrev.com/ba8376b3933223aa787273744c1bfd83f5574a15
Cr-Commit-Position: refs/heads/master@{#383282}

[sky, 2016-03-25 15:17:50]

LGTM

[tfarina, 2016-03-25 16:04:54]

The CQ bit was checked by tfarina@chromium.org

[commit-bot: I haz the power, 2016-03-25 16:05:02]

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

[commit-bot: I haz the power, 2016-03-25 16:18:31]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-03-25 16:20:22]

Description was changed from

==========
views: remove the last function abstraction from browser_dialogs.h

After commit faed95410a75: ("views: remove unnecessary EditSearchEngine()
function abstraction"), only CreateFindBar() function abstraction
remained in browser_dialogs.h, but like EditSearchEngine(), it is
unlikely this function will be used to be called by cross-platform
code, so it is not necessary keep it wrapping FindBarHost.

BrowserView can just depend on it like it does with many other
chrome/browser/ui/views/ header files already. This is also what
BrowserWindowCocoa does as well.

This in effect get rid of browser_dialogs.h header file finally.

BUG=None
R=sky@chromium.org
==========

to

==========
views: remove the last function abstraction from browser_dialogs.h

After commit faed95410a75: ("views: remove unnecessary EditSearchEngine()
function abstraction"), only CreateFindBar() function abstraction
remained in browser_dialogs.h, but like EditSearchEngine(), it is
unlikely this function will be used to be called by cross-platform
code, so it is not necessary keep it wrapping FindBarHost.

BrowserView can just depend on it like it does with many other
chrome/browser/ui/views/ header files already. This is also what
BrowserWindowCocoa does as well.

This in effect get rid of browser_dialogs.h header file finally.

BUG=None
R=sky@chromium.org

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

[commit-bot: I haz the power, 2016-03-25 16:20:23]

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