Eliminate unnecessary forward declarations from components/password_manager

Eliminate unnecessary forward declarations from components/password_manager

Some of the forward declrations are not necessary, hence, removed them.
This also makes sure that we forward declare wherever there is use
instead of using that declaration through another #include.

BUG=432826
Committed: https://crrev.com/e101909637ac60db4ac0b44a0c722a621354c97c
Cr-Commit-Position: refs/heads/master@{#304130}

[Sunil Ratnu, 2014-11-13 07:14:07]

sunil.ratnu@samsung.com changed reviewers:
+ isherman@chromium.org, msw@chromium.org, vabr@chromium.org

[Sunil Ratnu, 2014-11-13 07:14:07]

msw@chromium.org: Please review changes in chrome/browser/ui/login/

vabr@chromium.org/isherman@chromium.org: Please review changes in 
components/password_manager/

[vabr (Chromium), 2014-11-13 08:54:11]

Very nice, thanks for the cleanup!

Please add a tracking bug on http://crbug.com and a reference to that bug in the
CL description. It is good to always have a tracking bug, to increase the
visibility of the change in case people get concerned, and to track possible
follow-ups together.

Other than that, LGTM.

Cheers,
Vaclav

[Sunil Ratnu, 2014-11-13 09:01:19]

On 2014/11/13 08:54:11, vabr (Chromium) wrote:
> Very nice, thanks for the cleanup!
> 
> Please add a tracking bug on http://crbug.com and a reference to that bug in
the
> CL description. It is good to always have a tracking bug, to increase the
> visibility of the change in case people get concerned, and to track possible
> follow-ups together.
> 
> Other than that, LGTM.
> 
> Cheers,
> Vaclav

Thanks for the review Vaclav. Referenced the associated bug.

Thanks!
Sunil

[vabr (Chromium), 2014-11-13 09:14:09]

Thanks!
components/password_manager/* LGTM.

Cheers,
Vaclav

[msw, 2014-11-13 19:47:22]

lgtm

[Ilya Sherman, 2014-11-13 22:39:01]

The CQ bit was checked by isherman@chromium.org

[commit-bot: I haz the power, 2014-11-13 22:39:46]

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

[commit-bot: I haz the power, 2014-11-14 00:59:40]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2014-11-14 00:59:57]

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