Introduce Element::AttributeModificationParams

third_party/WebKit/Source/core/html/HTMLMediaElement.cpp

 /*
  * Copyright (C) 2007, 2008, 2009, 2010, 2011, 2012, 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.
  * 2. Redistributions in binary form must reproduce the above copyright
@@ skipping 530 matching lines @@
 
   // If no controls specified, we should still be able to focus the element if
   // it has tabIndex.
   return shouldShowControls() || HTMLElement::supportsFocus();
 }
 
 bool HTMLMediaElement::isMouseFocusable() const {
   return false;
 }
 
-void HTMLMediaElement::parseAttribute(const QualifiedName& name,
-                                      const AtomicString& oldValue,
-                                      const AtomicString& value) {
+void HTMLMediaElement::parseAttribute(
+    const AttributeModificationParams& params) {
+  const QualifiedName& name = params.name;
   if (name == srcAttr) {
     BLINK_MEDIA_LOG << "parseAttribute(" << (void*)this
-                    << ", srcAttr, old=" << oldValue << ", new=" << value
-                    << ")";
+                    << ", srcAttr, old=" << params.oldValue
+                    << ", new=" << params.newValue << ")";
     // Trigger a reload, as long as the 'src' attribute is present.
-    if (!value.isNull()) {
+    if (!params.newValue.isNull()) {
       m_ignorePreloadNone = false;
       invokeLoadAlgorithm();
     }
   } else if (name == controlsAttr) {
     UseCounter::count(document(),
                       UseCounter::HTMLMediaElementControlsAttribute);
     updateControlsVisibility();
   } else if (name == preloadAttr) {
     setPlayerPreload();
   } else if (name == disableremoteplaybackAttr) {
     // This attribute is an extension described in the Remote Playback API spec.
     // Please see: https://w3c.github.io/remote-playback
     UseCounter::count(document(), UseCounter::DisableRemotePlaybackAttribute);
-    if (oldValue != value) {
-      if (m_webMediaPlayer)
-        m_webMediaPlayer->requestRemotePlaybackDisabled(!value.isNull());
+    if (params.oldValue != params.newValue) {
+      if (m_webMediaPlayer) {
+        m_webMediaPlayer->requestRemotePlaybackDisabled(
+            !params.newValue.isNull());
+      }
       // TODO(mlamouri): there is no direct API to expose if
       // disableRemotePLayback attribute has changed. It will require a direct
       // access to MediaControls for the moment.
       if (mediaControls())
         mediaControls()->onDisableRemotePlaybackAttributeChanged();
     }
   } else {
-    HTMLElement::parseAttribute(name, oldValue, value);
+    HTMLElement::parseAttribute(params);
   }
 }
 
 void HTMLMediaElement::finishParsingChildren() {
   HTMLElement::finishParsingChildren();
 
   if (Traversal<HTMLTrackElement>::firstChild(*this))
     scheduleTextTrackResourceLoad();
 }
 
@@ skipping 3537 matching lines @@
       kMostlyFillViewportBecomeStableSeconds, BLINK_FROM_HERE);
 }
 
 void HTMLMediaElement::viewportFillDebouncerTimerFired(TimerBase*) {
   m_mostlyFillingViewport = true;
   if (m_webMediaPlayer)
     m_webMediaPlayer->becameDominantVisibleContent(m_mostlyFillingViewport);
 }
 
 }  // namespace blink