Componentize FaviconService and FaviconHandler

Componentize FaviconService and FaviconHandler

Move FaviconService and FaviconHandler into //components/favicon/core/browser
as they have no problematic dependencies anymore.

Update DEPS and dependencies of favicon component.

BUG=359514, 359513
Committed: https://crrev.com/fee69b4b384e94f03cd7d4dadf3140dd91bb9029
Cr-Commit-Position: refs/heads/master@{#320738}

[sdefresne, 2015-03-06 17:21:25]

sdefresne@chromium.org changed reviewers:
+ sky@chromium.org

[sdefresne, 2015-03-06 17:21:26]

Please take a look.

[sky, 2015-03-09 15:35:49]

LGTM

[sdefresne, 2015-03-13 21:05:26]

The CQ bit was checked by sdefresne@chromium.org

[sdefresne, 2015-03-13 21:05:27]

The patchset sent to the CQ was uploaded after l-g-t-m from sky@chromium.org
Link to the patchset: https://codereview.chromium.org/983043003/#ps20001 (title:
"Rebase")

[commit-bot: I haz the power, 2015-03-13 21:05:56]

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

[sdefresne, 2015-03-13 21:13:43]

The CQ bit was unchecked by sdefresne@chromium.org

[sdefresne, 2015-03-13 21:16:43]

sdefresne@chromium.org changed reviewers:
+ bsalomon@google.com, reed@google.com

[sdefresne, 2015-03-13 21:19:44]

reed, bsalomon: for whoever would be fastest, I need OWNERS approval for adding
to favicon component a dependency on
- skia
- third_party/skia

** Presubmit ERRORS **
Missing LGTM from OWNERS of dependencies added to DEPS:
    '+skia',
    '+third_party/skia',

I'm just moving code from chrome/browser/favicon to components/favicon, so the
dependency already existed, not adding a new one.

If you l-g-t-m, can you also CQ?

[reed1, 2015-03-16 14:19:23]

skia lgtm

[reed1, 2015-03-16 14:19:32]

The CQ bit was checked by reed@google.com

[reed1, 2015-03-16 14:19:33]

The patchset sent to the CQ was uploaded after l-g-t-m from sky@chromium.org
Link to the patchset: https://codereview.chromium.org/983043003/#ps40001 (title:
"Fix android_aosp build")

[commit-bot: I haz the power, 2015-03-16 14:19:50]

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

[sdefresne, 2015-03-16 15:10:30]

The CQ bit was unchecked by sdefresne@chromium.org

[sdefresne, 2015-03-16 16:51:39]

The CQ bit was checked by sdefresne@chromium.org

[sdefresne, 2015-03-16 16:51:39]

The patchset sent to the CQ was uploaded after l-g-t-m from sky@chromium.org,
reed@google.com
Link to the patchset: https://codereview.chromium.org/983043003/#ps80001 (title:
"Fix android_aosp (reverted the change by mistake by switching computer)")

[commit-bot: I haz the power, 2015-03-16 16:51:59]

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

[commit-bot: I haz the power, 2015-03-16 16:56:13]

Committed patchset #5 (id:80001)

[commit-bot: I haz the power, 2015-03-16 16:56:52]

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