Refactor resource load and resource selection algorithms as per spec

third_party/WebKit/Source/core/testing/Internals.cpp

 /*
  * Copyright (C) 2012 Google Inc. All rights reserved.
  * Copyright (C) 2013 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.
@@ skipping 1849 matching lines @@
     }
     document->page()->setIsCursorVisible(isVisible);
 }
 
 double Internals::effectiveMediaVolume(HTMLMediaElement* mediaElement)
 {
     ASSERT(mediaElement);
     return mediaElement->effectiveMediaVolume();
 }
 
+unsigned short Internals::effectivePreloadType(HTMLMediaElement* mediaElement)
+{
+    ASSERT(mediaElement);
+    return static_cast<unsigned short>(mediaElement->effectivePreloadType());

[philipj_slow, 2016/03/14 13:03:15]

Can you call if just effectivePreload and return a string in a fashion similar
to HTMLMediaElement::preload()? It could perhaps be moved into HTMLMediaElement
itself at a later point if a effectivePreload attribute is added to
HTMLMediaElement, which think is worth considering at least.

[Srirama, 2016/03/14 15:12:54]

Done.

+}
+
 void Internals::mediaPlayerRemoteRouteAvailabilityChanged(HTMLMediaElement* mediaElement, bool available)
 {
     ASSERT(mediaElement);
     mediaElement->remoteRouteAvailabilityChanged(available);
 }
 
 void Internals::mediaPlayerPlayingRemotelyChanged(HTMLMediaElement* mediaElement, bool remote)
 {
     ASSERT(mediaElement);
     if (remote)
@@ skipping 686 matching lines @@
 }
 
 int Internals::getScrollAnimationState(Node* node) const
 {
     if (ScrollableArea* scrollableArea = scrollableAreaForNode(node))
         return static_cast<int>(scrollableArea->scrollAnimator().m_runState);
     return -1;
 }
 
 } // namespace blink