Fix favicon updating in document mode.

Fix favicon updating in document mode.

This change modified how we get notified of favicon updates.
https://codereview.chromium.org/1464743005/

In particular for native pages, we send a null favicon out
and the call sites do not support that.

BUG=561996
Committed: https://crrev.com/ac441bfd7987f6ade76a7eea77d67be47de6b33c
Cr-Commit-Position: refs/heads/master@{#363672}

[Ted C, 2015-12-08 00:01:06]

tedchoc@chromium.org changed reviewers:
+ dfalcantara@chromium.org, yusufo@chromium.org

[Ted C, 2015-12-08 00:01:07]

PTAL

[Yusuf, 2015-12-08 00:14:22]

lgtm

[Ted C, 2015-12-08 00:16:20]

The CQ bit was checked by tedchoc@chromium.org

[commit-bot: I haz the power, 2015-12-08 00:16:57]

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

[commit-bot: I haz the power, 2015-12-08 01:16:52]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2015-12-08 01:18:02]

Description was changed from

==========
Fix favicon updating in document mode.

This change modified how we get notified of favicon updates.
https://codereview.chromium.org/1464743005/

In particular for native pages, we send a null favicon out
and the call sites do not support that.

BUG=561996
==========

to

==========
Fix favicon updating in document mode.

This change modified how we get notified of favicon updates.
https://codereview.chromium.org/1464743005/

In particular for native pages, we send a null favicon out
and the call sites do not support that.

BUG=561996

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

[commit-bot: I haz the power, 2015-12-08 01:18:02]

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