[Download Notification] Show preview if downloaded file is image

chrome/browser/image_decoder.h

Index: chrome/browser/image_decoder.h
diff --git a/chrome/browser/image_decoder.h b/chrome/browser/image_decoder.h
index 9e0b3edeaff92cff17a04aac93b0cbc236e06c9e..10dbf0785cc1d5e20e4890f865a6691fdb64f15e 100644
--- a/chrome/browser/image_decoder.h
+++ b/chrome/browser/image_decoder.h
@@ -9,6 +9,7 @@
 #include <string>
 #include <vector>
 
+#include "base/files/file.h"
 #include "base/lazy_instance.h"
 #include "base/memory/ref_counted.h"
 #include "base/sequence_checker.h"
@@ -71,6 +72,9 @@ class ImageDecoder : public content::UtilityProcessHostClient {
 #endif  // defined(OS_CHROMEOS)
   };
 
+  static void Start(ImageRequest* image_request,
+                    const base::FilePath& image_file_path);
+
   // Calls StartWithOptions() with ImageCodec::DEFAULT_CODEC and
   // shrink_to_fit = false.
   static void Start(ImageRequest* image_request,
@@ -100,11 +104,13 @@ class ImageDecoder : public content::UtilityProcessHostClient {
   // batch mode if necessary. If the utility process fails to start,
   // an OnDecodeImageFailed task is posted to image_request's |task_runner_|.
   void DecodeImageInSandbox(int request_id,

[asanka, 2015/06/11 03:30:52]

I'll defer to the owners.

However, for both DecodeImageInSandbox() and StartWithOptionsImpl() different
subsets of parameters are valid. It's probably a good sign that these should be
split.

[yoshiki, 2015/06/11 08:22:09]

Acknowledged.

+                            base::File image_file,
                             const std::vector<unsigned char>& image_data,
                             ImageCodec image_codec,
                             bool shrink_to_fit);
 
   void StartWithOptionsImpl(ImageRequest* image_request,
+                            const base::FilePath& image_file_path,
                             const std::string& image_data,
                             ImageCodec image_codec,
                             bool shrink_to_fit);