Media Session: use dictionary instead of an interface for MediaImage.

third_party/WebKit/Source/modules/mediasession/MediaMetadataSanitizer.cpp

 // Copyright 2016 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "modules/mediasession/MediaMetadataSanitizer.h"
 
 #include "core/dom/ExecutionContext.h"
 #include "core/inspector/ConsoleMessage.h"
 #include "modules/mediasession/MediaImage.h"
 #include "modules/mediasession/MediaMetadata.h"
@@ skipping 41 matching lines @@
         JSMessageSource, WarningMessageLevel,
         "MediaImage src exceeds maximum URL length: " + src.getString()));
     return false;
   }
   return true;
 }
 
 // Sanitize MediaImage and do mojo serialization. Returns null when
 // |image.src()| is bad.
 blink::mojom::blink::MediaImagePtr sanitizeMediaImageAndConvertToMojo(
-    const MediaImage* image,
+    const MediaImage& image,
     ExecutionContext* context) {
-  DCHECK(image);
-
   blink::mojom::blink::MediaImagePtr mojoImage;
 
-  KURL url = KURL(ParsedURLString, image->src());
+  KURL url = KURL(ParsedURLString, image.src());

[Zhiqiang Zhang (Slow), 2017/01/04 16:05:24]

Resolve to absolute URL if we decide to do it here.

[mlamouri (slow - plz ping), 2017/01/05 15:40:20]

I did it when we set. Though, this code assumes it is an absolute URL already.

   if (!checkMediaImageSrcSanity(url, context))
     return mojoImage;
 
   mojoImage = blink::mojom::blink::MediaImage::New();
   mojoImage->src = url;
-  mojoImage->type = image->type().left(kMaxImageTypeLength);
+  mojoImage->type = image.type().left(kMaxImageTypeLength);
   for (const auto& webSize :
-       WebIconSizesParser::parseIconSizes(image->sizes())) {
+       WebIconSizesParser::parseIconSizes(image.sizes())) {
     mojoImage->sizes.append(webSize);
     if (mojoImage->sizes.size() == kMaxNumberOfImageSizes) {
       context->addConsoleMessage(ConsoleMessage::create(
           JSMessageSource, WarningMessageLevel,
           "The number of MediaImage sizes exceeds the upper limit. "
           "All remaining MediaImage will be ignored"));
       break;
     }
   }
   return mojoImage;
 }
 
 }  // anonymous namespace
 
 blink::mojom::blink::MediaMetadataPtr
 MediaMetadataSanitizer::sanitizeAndConvertToMojo(const MediaMetadata* metadata,
                                                  ExecutionContext* context) {
   blink::mojom::blink::MediaMetadataPtr mojoMetadata;
   if (!metadata)
     return mojoMetadata;
 
   mojoMetadata = blink::mojom::blink::MediaMetadata::New();
 
   mojoMetadata->title = metadata->title().left(kMaxStringLength);
   mojoMetadata->artist = metadata->artist().left(kMaxStringLength);
   mojoMetadata->album = metadata->album().left(kMaxStringLength);
 
-  for (const auto image : metadata->artwork()) {
+  for (const MediaImage& image : metadata->artwork()) {
     blink::mojom::blink::MediaImagePtr mojoImage =
-        sanitizeMediaImageAndConvertToMojo(image.get(), context);
+        sanitizeMediaImageAndConvertToMojo(image, context);
     if (!mojoImage.is_null())
       mojoMetadata->artwork.append(std::move(mojoImage));
     if (mojoMetadata->artwork.size() == kMaxNumberOfMediaImages) {
       context->addConsoleMessage(ConsoleMessage::create(
           JSMessageSource, WarningMessageLevel,
           "The number of MediaImage sizes exceeds the upper limit. "
           "All remaining MediaImage will be ignored"));
       break;
     }
   }
   return mojoMetadata;
 }
 
 }  // namespace blink