Change preload's as values audio/video to media

third_party/WebKit/Source/core/loader/LinkLoader.cpp

 /*
  * Copyright (C) 2011 Google 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:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Redistributions in binary form must reproduce the above
@@ skipping 153 matching lines @@
                 document.addConsoleMessage(ConsoleMessage::create(OtherMessageSource, DebugMessageLevel,
                     String("Preconnect CORS setting is ") + String((crossOrigin == CrossOriginAttributeAnonymous) ? "anonymous" : "use-credentials")));
             }
         }
         networkHintsInterface.preconnectHost(href, crossOrigin);
     }
 }
 
 bool LinkLoader::getResourceTypeFromAsAttribute(const String& as, Resource::Type& type)
 {
-    if (equalIgnoringCase(as, "image")) {
+    ASSERT(as.lower() == as);

[Nate Chapin, 2016/03/10 22:49:05]

Was this always true?

[Yoav Weiss, 2016/03/10 23:12:12]

Yes, it was (embarrassingly)

+    if (as == "image") {
         type = Resource::Image;
-    } else if (equalIgnoringCase(as, "script")) {
+    } else if (as == "script") {
         type = Resource::Script;
-    } else if (equalIgnoringCase(as, "style")) {
+    } else if (as == "style") {
         type = Resource::CSSStyleSheet;
-    } else if (equalIgnoringCase(as, "audio") || equalIgnoringCase(as, "video")) {
+    } else if (as == "media") {
         type = Resource::Media;
-    } else if (equalIgnoringCase(as, "font")) {
+    } else if (as == "font") {
         type = Resource::Font;
-    } else if (equalIgnoringCase(as, "track")) {
+    } else if (as == "track") {
         type = Resource::TextTrack;
     } else {
         type = Resource::LinkPreload;
         if (!as.isEmpty())
             return false;
     }
     return true;
 }
 
 void LinkLoader::createLinkPreloadResourceClient(Resource* resource)
@@ skipping 179 matching lines @@
 
 DEFINE_TRACE(LinkLoader)
 {
     visitor->trace(m_client);
     visitor->trace(m_prerender);
     visitor->trace(m_linkPreloadResourceClient);
     ResourceOwner<Resource, ResourceClient>::trace(visitor);
 }
 
 } // namespace blink