Set desired_image_size when decoding images for NTP Tile icons.

components/ntp_snippets/remote/remote_suggestions_provider_impl.cc

Index: components/ntp_snippets/remote/remote_suggestions_provider_impl.cc
diff --git a/components/ntp_snippets/remote/remote_suggestions_provider_impl.cc b/components/ntp_snippets/remote/remote_suggestions_provider_impl.cc
index 81dcbf38c7c6922cbf17bb1860aa0c353fa2964e..363b2b0851bf7dc3536c0980676396ae27948b9f 100644
--- a/components/ntp_snippets/remote/remote_suggestions_provider_impl.cc
+++ b/components/ntp_snippets/remote/remote_suggestions_provider_impl.cc
@@ -30,6 +30,7 @@
 #include "components/prefs/pref_registry_simple.h"
 #include "components/prefs/pref_service.h"
 #include "components/strings/grit/components_strings.h"
+#include "ui/gfx/geometry/size.h"
 #include "ui/gfx/image/image.h"
 
 namespace ntp_snippets {
@@ -215,8 +216,11 @@ void CachedImageFetcher::OnImageFetchedFromDatabase(
   // |image_decoder_| is null in tests.
   if (image_decoder_ && !data.empty()) {
     image_decoder_->DecodeImage(
-        data, base::Bind(&CachedImageFetcher::OnImageDecodedFromDatabase,
-                         base::Unretained(this), callback, suggestion_id, url));
+        data,
+        // We're not dealing with multi-frame images.
+        /*desired_image_frame_size=*/gfx::Size(),

[Mathieu, 2017/03/01 14:36:23]

it's a nit, but I would put a space between "/" and "gfx"

[tschumann, 2017/03/01 19:40:08]

We adopted this format because there's a clang-tidy check
(https://g3doc.corp.google.com/devtools/cymbal/clang_tidy/g3doc/checks/misc-ar...)
which can verify this. We chose the format without space to minimize space (and
it's how the clang-tidy example are).

+        base::Bind(&CachedImageFetcher::OnImageDecodedFromDatabase,
+                   base::Unretained(this), callback, suggestion_id, url));
     return;
   }
   // Fetching from the DB failed; start a network fetch.