[Webkit-unassigned] [Bug 24846] HTMLMediaElement should implement 'autobuffer' attribute

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Sat Mar 28 17:36:28 PDT 2009


darin at apple.com changed:

           What    |Removed                     |Added
  Attachment #29038|review?                     |review-
               Flag|                            |

------- Comment #2 from darin at apple.com  2009-03-28 17:36 PDT -------
(From update of attachment 29038)
> +    setBooleanAttribute(autobufferAttr, b);
> +    if (m_player)
> +        m_player->setAutobuffer(b);

This is not the right way to bind an attribute to a back-end player property. I
haven't looked at the other attributes to see if you're doing it that way. The
correct place to do the mapping is in the parseMappedAttribute function. The
setter should only set the attribute and the back end effect should be
triggered by parseMappedAttribute.

Otherwise, setting the attribute via setAttribute or by setting the attribute
in the initial document won't work.

> +void MediaPlayer::setAutobuffer(bool b)
> +{
> +    m_autobuffer = b;
> +    m_private->setAutobuffer(b);
> +}

Since we already know the old value here, I suggest we call the private
function only if the value is changing. That way if the private version has to
do some real work it won't get done if there's no change.

review- because of the first comment above

Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.

More information about the webkit-unassigned mailing list