Call buffer.SetDuration in InitializeNewFrame

This requires the new libavif function avifDecoderNthImageTiming, which
provides each frame's duration before the frames are decoded.

Bug: 960620
Change-Id: Id00c760e094b8663b1a8781201fdd4a6c88beddd
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2149915
Commit-Queue: Wan-Teh Chang <wtc@google.com>
Reviewed-by: Peter Kasting <pkasting@chromium.org>
Cr-Commit-Position: refs/heads/master@{#759347}

DEPS

@@ -322,7 +322,7 @@ vars = {
   # Three lines of non-changing comments so that
   # the commit queue can handle CLs rolling libavif
   # and whatever else without interference from each other.
-  'libavif_revision': '78e3c9d2fbfa935638145d017dda6950cc559ad3',
+  'libavif_revision': '167e406f1e2b86f6699224b5c492dc9e862ab539',
 
   # TODO(crbug.com/941824): The values below need to be kept in sync
   # between //DEPS and //buildtools/DEPS, so if you're updating one,

third_party/blink/renderer/platform/image-decoders/avif/avif_image_decoder.cc

@@ -257,11 +257,10 @@ void AVIFImageDecoder::InitializeNewFrame(size_t index) {
 
   buffer.SetOriginalFrameRect(IntRect(IntPoint(), Size()));
 
-  // TODO(wtc): libavif provides only the current frame's duration. But when
-  // this method is called, we have not decoded any frames. Change libavif to
-  // provide all frames' durations after avifDecoderParse(). Then call
-  // buffer.SetDuration() here and remove the buffer.SetDuration() call in
-  // Decode().
+  avifImageTiming timing;
+  auto ret = avifDecoderNthImageTiming(decoder_.get(), index, &timing);
+  DCHECK_EQ(ret, AVIF_RESULT_OK);
+  buffer.SetDuration(base::TimeDelta::FromSecondsD(timing.duration));
 
   // TODO(wtc): Find out if disposal method and alpha blend source are specified
   // in the file format.
@@ -334,8 +333,6 @@ void AVIFImageDecoder::Decode(size_t index) {
       return;
     }
 
-    buffer.SetDuration(
-        base::TimeDelta::FromSecondsD(decoder_->imageTiming.duration));
     buffer.SetPixelsChanged(true);
     buffer.SetStatus(ImageFrame::kFrameComplete);
   }