Fix metadata loss when revalidating search provider logo.

Fix metadata loss when revalidating search provider logo.

Previously, when revaliding a cached search provider's logo, the
mime_type of the image and the URL of the animated image to show (if
any) were dropped. This fixes those bugs and adds tests.

BUG=480090
Committed: https://crrev.com/3a171a20534963af96e28129c1019882afe83c97
Cr-Commit-Position: refs/heads/master@{#326664}

[newt (away), 2015-04-23 03:46:47]

newt@chromium.org changed reviewers:
+ justincohen@chromium.org

[newt (away), 2015-04-23 03:46:47]

PTAL. It's similar to your CL, but treats animated_url differently, and beefs up
the tests more.

[justincohen, 2015-04-23 13:03:38]

LGTM, Thanks!  This should be merged to branch.

[newt (away), 2015-04-23 18:00:29]

The CQ bit was checked by newt@chromium.org

[newt (away), 2015-04-23 18:02:37]

The CQ bit was unchecked by newt@chromium.org

[newt (away), 2015-04-23 18:03:12]

newt@chromium.org changed reviewers:
+ tedchoc@chromium.org

[newt (away), 2015-04-23 18:03:13]

+tedchoc for committers approval

[commit-bot: I haz the power, 2015-04-23 18:03:53]

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

[commit-bot: I haz the power, 2015-04-23 18:03:54]

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

[commit-bot: I haz the power, 2015-04-23 18:03:57]

No LGTM from a valid reviewer yet. Only full committers are accepted.
Even if an LGTM may have been provided, it was from a non-committer,
_not_ a full super star committer.
See http://www.chromium.org/getting-involved/become-a-committer
Note that this has nothing to do with OWNERS files.

[Ted C, 2015-04-23 20:47:37]

On 2015/04/23 18:03:57, I haz the power (commit-bot) wrote:
> No LGTM from a valid reviewer yet. Only full committers are accepted.
> Even if an LGTM may have been provided, it was from a non-committer,
> _not_ a full super star committer.
> See http://www.chromium.org/getting-involved/become-a-committer
> Note that this has nothing to do with OWNERS files.

lgtm

[newt (away), 2015-04-23 21:13:25]

The CQ bit was checked by newt@chromium.org

[commit-bot: I haz the power, 2015-04-23 21:14:26]

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

[commit-bot: I haz the power, 2015-04-23 22:02:28]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2015-04-23 22:03:38]

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