cc/blink: Veto checker-imaging for multipart image resources.

third_party/WebKit/Source/core/loader/resource/ImageResource.cpp

 /*
     Copyright (C) 1998 Lars Knoll (knoll@mpi-hd.mpg.de)
     Copyright (C) 2001 Dirk Mueller (mueller@kde.org)
     Copyright (C) 2002 Waldo Bastian (bastian@kde.org)
     Copyright (C) 2006 Samuel Weinig (sam.weinig@gmail.com)
     Copyright (C) 2004, 2005, 2006, 2007 Apple Inc. All rights reserved.
 
     This library is free software; you can redistribute it and/or
     modify it under the terms of the GNU Library General Public
     License as published by the Free Software Foundation; either
@@ skipping 324 matching lines @@
 }
 
 void ImageResource::DecodeError(bool all_data_received) {
   size_t size = EncodedSize();
 
   ClearData();
   SetEncodedSize(0);
   if (!ErrorOccurred())
     SetStatus(ResourceStatus::kDecodeError);
 
+  bool is_multi_part = !!multipart_parser_;

[vmpstr, 2017/06/16 17:35:36]

everything else is called multipart, so maybe this should be is_multipart :)

[Khushal, 2017/06/16 18:26:33]

Done.

   // Finishes loading if needed, and notifies observers.
   if (!all_data_received && Loader()) {
     // Observers are notified via ImageResource::finish().
     // TODO(hiroshige): Do not call didFinishLoading() directly.
     Loader()->DidFinishLoading(MonotonicallyIncreasingTime(), size, size, size);
   } else {
     auto result = GetContent()->UpdateImage(
         nullptr, GetStatus(),
-        ImageResourceContent::kClearImageAndNotifyObservers, all_data_received);
+        ImageResourceContent::kClearImageAndNotifyObservers, all_data_received,
+        is_multi_part);
     DCHECK_EQ(result, ImageResourceContent::UpdateImageResult::kNoDecodeError);
   }
 
   GetMemoryCache()->Remove(this);
 }
 
 void ImageResource::UpdateImageAndClearBuffer() {
   UpdateImage(Data(), ImageResourceContent::kClearAndUpdateImage, true);
   ClearData();
 }
@@ skipping 239 matching lines @@
 }
 
 ResourcePriority ImageResource::PriorityFromObservers() {
   return GetContent()->PriorityFromObservers();
 }
 
 void ImageResource::UpdateImage(
     PassRefPtr<SharedBuffer> shared_buffer,
     ImageResourceContent::UpdateImageOption update_image_option,
     bool all_data_received) {
-  auto result =
-      GetContent()->UpdateImage(std::move(shared_buffer), GetStatus(),
-                                update_image_option, all_data_received);
+  bool is_multi_part = !!multipart_parser_;
+  auto result = GetContent()->UpdateImage(std::move(shared_buffer), GetStatus(),
+                                          update_image_option,
+                                          all_data_received, is_multi_part);
   if (result == ImageResourceContent::UpdateImageResult::kShouldDecodeError) {
     // In case of decode error, we call imageNotifyFinished() iff we don't
     // initiate reloading:
     // [(a): when this is in the middle of loading, or (b): otherwise]
     // 1. The updateImage() call above doesn't call notifyObservers().
     // 2. notifyObservers(ShouldNotifyFinish) is called
     //    (a) via updateImage() called in ImageResource::finish()
     //        called via didFinishLoading() called in decodeError(), or
     //    (b) via updateImage() called in decodeError().
     //    imageNotifyFinished() is called here iff we will not initiate
     //    reloading in Step 3 due to notifyObservers()'s
     //    schedulingReloadOrShouldReloadBrokenPlaceholder() check.
     // 3. reloadIfLoFiOrPlaceholderImage() is called via ResourceFetcher
     //    (a) via didFinishLoading() called in decodeError(), or
     //    (b) after returning ImageResource::updateImage().
     DecodeError(all_data_received);
   }
 }
 
 }  // namespace blink