Is this happening for 65?

sfiera, should we WontFix this? IIRC we decided not to go with this approach after all.

I was mistaken in the original description, in that interactive doodles still include a CTA. We still anticipate that (one day) it would make sense to not include it, though. I think open/P3 matches that, but it might be better to rephrase the bug to match the goal (“Don’t download interactive CTA if it won’t be shown” or something).

(not working on client-side Chrome any more)

Need to evaluate whether this applies to doodles on the Local NTP.

It looks like everything here works. We're just doing some extra work by downloading an image that's never used, which would take a bit of refactoring to eliminate?

Yes, I believe that's correct: For interactive Doodles, we download, decode etc. an image that's never used. Since that's not particularly bad, and interactive Doodles are somewhat rare anyway, this is not very urgent. Would still be nice to fix eventually.
(Video Doodles are also implemented as interactives, so they're not *that* rare.)

While debugging today's Doodle we discovered that the Doodle team's testing sandboxes don't include the image (data_uri field) for interactive doodles. 

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/bf7b0cb7ff8897c1e1160c48dc2dcfca0c37329e

commit bf7b0cb7ff8897c1e1160c48dc2dcfca0c37329e
Author: Kyle Milka <kmilka@chromium.org>
Date: Thu Jan 17 19:07:41 2019

Allow ddljson response to not contain an image

Interactive doodles don't use the image (data_uri field) included
in the ddljson response. Don't require that responses (on Desktop)
contain an image. Allow logo_cache to cache a doodle that doesn't
include an image.

Bug:  800170 
Change-Id: I637ab2130db29efadecebd787c244e59008d3e63
Reviewed-on: https://chromium-review.googlesource.com/c/1405431
Commit-Queue: Kyle Milka <kmilka@chromium.org>
Reviewed-by: Marc Treib <treib@chromium.org>
Cr-Commit-Position: refs/heads/master@{#623783}
[modify] https://crrev.com/bf7b0cb7ff8897c1e1160c48dc2dcfca0c37329e/chrome/browser/resources/local_ntp/doodles.js
[modify] https://crrev.com/bf7b0cb7ff8897c1e1160c48dc2dcfca0c37329e/chrome/browser/search/local_ntp_source.cc
[modify] https://crrev.com/bf7b0cb7ff8897c1e1160c48dc2dcfca0c37329e/chrome/browser/ui/search/local_ntp_doodle_browsertest.cc
[modify] https://crrev.com/bf7b0cb7ff8897c1e1160c48dc2dcfca0c37329e/components/search_provider_logos/google_logo_api_unittest.cc
[modify] https://crrev.com/bf7b0cb7ff8897c1e1160c48dc2dcfca0c37329e/components/search_provider_logos/logo_cache.cc
[modify] https://crrev.com/bf7b0cb7ff8897c1e1160c48dc2dcfca0c37329e/components/search_provider_logos/logo_cache_unittest.cc
[modify] https://crrev.com/bf7b0cb7ff8897c1e1160c48dc2dcfca0c37329e/components/search_provider_logos/logo_service_impl.cc
[modify] https://crrev.com/bf7b0cb7ff8897c1e1160c48dc2dcfca0c37329e/components/search_provider_logos/logo_service_impl.h
[modify] https://crrev.com/bf7b0cb7ff8897c1e1160c48dc2dcfca0c37329e/tools/metrics/histograms/enums.xml

The NextAction date has arrived: 2019-01-22

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/f630b0d3526a18552fe41cf72bfec465b68ed6b8

commit f630b0d3526a18552fe41cf72bfec465b68ed6b8
Author: Kyle Milka <kmilka@chromium.org>
Date: Tue Jan 22 17:31:11 2019

Revert "Allow ddljson response to not contain an image"

This reverts commit bf7b0cb7ff8897c1e1160c48dc2dcfca0c37329e.

Reason for revert: <INSERT REASONING HERE>

Original change's description:
> Allow ddljson response to not contain an image
> 
> Interactive doodles don't use the image (data_uri field) included
> in the ddljson response. Don't require that responses (on Desktop)
> contain an image. Allow logo_cache to cache a doodle that doesn't
> include an image.
> 
> Bug:  800170 
> Change-Id: I637ab2130db29efadecebd787c244e59008d3e63
> Reviewed-on: https://chromium-review.googlesource.com/c/1405431
> Commit-Queue: Kyle Milka <kmilka@chromium.org>
> Reviewed-by: Marc Treib <treib@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#623783}

TBR=treib@chromium.org,kmilka@chromium.org

# Not skipping CQ checks because original CL landed > 1 day ago.

Bug:  800170 
Change-Id: Ided4301b88586d21cbb653aefaf75206aa6d2c5d
Reviewed-on: https://chromium-review.googlesource.com/c/1426879
Reviewed-by: Kyle Milka <kmilka@chromium.org>
Commit-Queue: Kyle Milka <kmilka@chromium.org>
Cr-Commit-Position: refs/heads/master@{#624813}
[modify] https://crrev.com/f630b0d3526a18552fe41cf72bfec465b68ed6b8/chrome/browser/resources/local_ntp/doodles.js
[modify] https://crrev.com/f630b0d3526a18552fe41cf72bfec465b68ed6b8/chrome/browser/search/local_ntp_source.cc
[modify] https://crrev.com/f630b0d3526a18552fe41cf72bfec465b68ed6b8/chrome/browser/ui/search/local_ntp_doodle_browsertest.cc
[modify] https://crrev.com/f630b0d3526a18552fe41cf72bfec465b68ed6b8/components/search_provider_logos/google_logo_api_unittest.cc
[modify] https://crrev.com/f630b0d3526a18552fe41cf72bfec465b68ed6b8/components/search_provider_logos/logo_cache.cc
[modify] https://crrev.com/f630b0d3526a18552fe41cf72bfec465b68ed6b8/components/search_provider_logos/logo_cache_unittest.cc
[modify] https://crrev.com/f630b0d3526a18552fe41cf72bfec465b68ed6b8/components/search_provider_logos/logo_service_impl.cc
[modify] https://crrev.com/f630b0d3526a18552fe41cf72bfec465b68ed6b8/components/search_provider_logos/logo_service_impl.h
[modify] https://crrev.com/f630b0d3526a18552fe41cf72bfec465b68ed6b8/tools/metrics/histograms/enums.xml