[ios] Adds GoogleLogoService to help with fetching doodles.

[ios] Adds GoogleLogoService to help with fetching doodles.

BUG=656617
Committed: https://crrev.com/3152599bcce134b11eb8b6c5229bc8d4fd772e70
Cr-Commit-Position: refs/heads/master@{#426795}

[rohitrao (ping after 24h), 2016-10-20 02:56:08]

The CQ bit was checked by rohitrao@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-10-20 02:56:50]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[rohitrao (ping after 24h), 2016-10-20 02:58:38]

rohitrao@chromium.org changed reviewers:
+ justincohen@chromium.org, sdefresne@chromium.org

[rohitrao (ping after 24h), 2016-10-20 02:58:38]

rohitrao@chromium.org changed required reviewers:
+ justincohen@chromium.org, sdefresne@chromium.org

[rohitrao (ping after 24h), 2016-10-20 02:58:39]

The implementation of this service felt Google-specific, so I named it
accordingly.  Renaming helps me cut a few steps out of an otherwise seven-sided
patch, but it's not a big deal if people don't like the new name.

[commit-bot: I haz the power, 2016-10-20 04:22:32]

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

[commit-bot: I haz the power, 2016-10-20 04:22:33]

Dry run: This issue passed the CQ dry run.

[justincohen, 2016-10-20 14:39:41]

name change rs lgtm

[sdefresne, 2016-10-20 18:53:45]

lgtm

[rohitrao (ping after 24h), 2016-10-20 22:06:01]

The CQ bit was checked by rohitrao@chromium.org

[commit-bot: I haz the power, 2016-10-20 22:06:45]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-10-20 22:30:49]

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

[commit-bot: I haz the power, 2016-10-20 22:30:51]

Try jobs failed on following builders:
  chromium_presubmit on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)

[rohitrao (ping after 24h), 2016-10-20 22:37:58]

rohitrao@chromium.org changed reviewers:
+ reed@chromium.org

[rohitrao (ping after 24h), 2016-10-20 22:37:58]

+reed for OWNERS approval to add SkBitmap.h in ios/chrome/browser/DEPS.  Thanks!

[rohitrao (ping after 24h), 2016-10-21 13:25:38]

rohitrao@chromium.org changed reviewers:
+ bsalomon@chromium.org

[rohitrao (ping after 24h), 2016-10-21 13:25:39]

Also +bsalomon, because I can't tell if reed is OOO today =)

[reed1, 2016-10-21 13:27:50]

reed@google.com changed reviewers:
+ reed@google.com

[reed1, 2016-10-21 13:27:51]

lgtm

boy do we need to update gfx to use SkImages ...

[rohitrao (ping after 24h), 2016-10-21 13:30:06]

rohitrao@chromium.org changed reviewers:
- bsalomon@chromium.org

[rohitrao (ping after 24h), 2016-10-21 13:30:07]

Thanks for the review!

Happy to chat about skia on iOS anytime =)

[rohitrao (ping after 24h), 2016-10-21 13:30:15]

The CQ bit was checked by rohitrao@chromium.org

[commit-bot: I haz the power, 2016-10-21 13:30:23]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-10-21 15:12:24]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-10-21 15:34:05]

Description was changed from

==========
[ios] Adds GoogleLogoService to help with fetching doodles.

BUG=656617
==========

to

==========
[ios] Adds GoogleLogoService to help with fetching doodles.

BUG=656617

Committed: https://crrev.com/3152599bcce134b11eb8b6c5229bc8d4fd772e70
Cr-Commit-Position: refs/heads/master@{#426795}
==========

[commit-bot: I haz the power, 2016-10-21 15:34:07]

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