Schedule bitmap animation timers on the compositor task runner.

third_party/WebKit/Source/core/css/CSSImageSetValue.cpp

 /*
  * Copyright (C) 2012 Apple Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
  * are met:
  * 1. Redistributions of source code must retain the above copyright
  *    notice, this list of conditions and the following disclaimer.
  * 2. Redistributions in binary form must reproduce the above copyright
  *    notice, this list of conditions and the following disclaimer in the
@@ skipping 11 matching lines @@
  * ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF
  * THE POSSIBILITY OF SUCH DAMAGE.
  */
 
 #include "core/css/CSSImageSetValue.h"
 
 #include <algorithm>
 #include "core/css/CSSImageValue.h"
 #include "core/css/CSSPrimitiveValue.h"
 #include "core/dom/Document.h"
+#include "core/dom/TaskRunnerHelper.h"
 #include "core/frame/Settings.h"
 #include "core/loader/resource/ImageResourceContent.h"
 #include "core/style/StyleFetchedImageSet.h"
 #include "core/style/StyleInvalidImage.h"
 #include "platform/loader/fetch/FetchInitiatorTypeNames.h"
 #include "platform/loader/fetch/FetchParameters.h"
 #include "platform/loader/fetch/ResourceFetcher.h"
 #include "platform/loader/fetch/ResourceLoaderOptions.h"
 #include "platform/weborigin/KURL.h"
 #include "platform/weborigin/SecurityPolicy.h"
@@ skipping 74 matching lines @@
     FetchParameters params(resource_request, FetchInitiatorTypeNames::css);
 
     if (cross_origin != kCrossOriginAttributeNotSet) {
       params.SetCrossOriginAccessControl(document.GetSecurityOrigin(),
                                          cross_origin);
     }
     if (document.GetSettings() &&
         document.GetSettings()->GetFetchImagePlaceholders())
       params.SetAllowImagePlaceholder();
 
-    if (ImageResourceContent* cached_image =
-            ImageResourceContent::Fetch(params, document.Fetcher())) {
+    // TODO(delphick): const_cast
+    RefPtr<WebTaskRunner> task_runner = TaskRunnerHelper::Get(

[Sami, 2017/04/27 17:38:17]

Interesting, can TaskRunnerHelper be changed to take a const instead?

[Dan Elphick, 2017/05/03 09:41:06]

not required now we're not using getting the scheduler via the frame

+        TaskType::kAnimationTimer, const_cast<Document*>(&document));
+    if (ImageResourceContent* cached_image = ImageResourceContent::Fetch(
+            task_runner, params, document.Fetcher())) {
       cached_image_ = StyleFetchedImageSet::Create(
           cached_image, image.scale_factor, this, params.Url());
     } else {
       cached_image_ = StyleInvalidImage::Create(image.image_url);
     }
     cached_scale_factor_ = device_scale_factor;
   }
 
   return cached_image_.Get();
 }
@@ skipping 44 matching lines @@
 CSSImageSetValue* CSSImageSetValue::ValueWithURLsMadeAbsolute() {
   CSSImageSetValue* value = CSSImageSetValue::Create();
   for (auto& item : *this)
     item->IsImageValue()
         ? value->Append(*ToCSSImageValue(*item).ValueWithURLMadeAbsolute())
         : value->Append(*item);
   return value;
 }
 
 }  // namespace blink