<?xml version="1.0" encoding="UTF-8" standalone="yes" ?>
<!DOCTYPE bugzilla SYSTEM "https://bugs.webkit.org/page.cgi?id=bugzilla.dtd">

<bugzilla version="5.0.4.1"
          urlbase="https://bugs.webkit.org/"
          
          maintainer="admin@webkit.org"
>

    <bug>
          <bug_id>28475</bug_id>
          
          <creation_ts>2009-08-19 17:09:40 -0700</creation_ts>
          <short_desc>Chromium: render disabled mute button if audio is not present</short_desc>
          <delta_ts>2009-08-21 14:46:46 -0700</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>1</classification_id>
          <classification>Unclassified</classification>
          <product>WebKit</product>
          <component>WebCore Misc.</component>
          <version>528+ (Nightly build)</version>
          <rep_platform>PC</rep_platform>
          <op_sys>Windows XP</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>FIXED</resolution>
          
          
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords></keywords>
          <priority>P2</priority>
          <bug_severity>Trivial</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>0</everconfirmed>
          <reporter name="Andrew Scherkus">scherkus</reporter>
          <assigned_to name="Nobody">webkit-unassigned</assigned_to>
          <cc>eric</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>141647</commentid>
    <comment_count>0</comment_count>
    <who name="Andrew Scherkus">scherkus</who>
    <bug_when>2009-08-19 17:09:40 -0700</bug_when>
    <thetext>If hasAudio() returns false or the media is in an error state, display a disabled mute button.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>141660</commentid>
    <comment_count>1</comment_count>
      <attachid>35164</attachid>
    <who name="Andrew Scherkus">scherkus</who>
    <bug_when>2009-08-19 17:26:23 -0700</bug_when>
    <thetext>Created attachment 35164
Check for audio</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>141732</commentid>
    <comment_count>2</comment_count>
      <attachid>35164</attachid>
    <who name="David Levin">levin</who>
    <bug_when>2009-08-19 21:38:57 -0700</bug_when>
    <thetext>Comment on attachment 35164
Check for audio

Made commit queue- pending Andrew&apos;s confirmation that the mediaSoundDisabled resource has been added on the Chromium side.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>141733</commentid>
    <comment_count>3</comment_count>
      <attachid>35164</attachid>
    <who name="David Levin">levin</who>
    <bug_when>2009-08-19 21:43:17 -0700</bug_when>
    <thetext>Comment on attachment 35164
Check for audio

I just found out that the resource is in.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>141734</commentid>
    <comment_count>4</comment_count>
      <attachid>35164</attachid>
    <who name="Eric Seidel (no email)">eric</who>
    <bug_when>2009-08-19 21:56:50 -0700</bug_when>
    <thetext>Comment on attachment 35164
Check for audio

Rejecting patch 35164 from commit-queue.  This patch will require manual commit.

Found no modified ChangeLogs, cannot create a commit message.
All changes require a ChangeLog.  See:
http://webkit.org/coding/contributing.html</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>141985</commentid>
    <comment_count>5</comment_count>
    <who name="Eric Seidel (no email)">eric</who>
    <bug_when>2009-08-21 14:46:46 -0700</bug_when>
    <thetext>Looks like this was already landed.</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>35164</attachid>
            <date>2009-08-19 17:26:23 -0700</date>
            <delta_ts>2009-08-19 21:56:50 -0700</delta_ts>
            <desc>Check for audio</desc>
            <filename>has_audio.patch</filename>
            <type>text/plain</type>
            <size>1649</size>
            <attacher name="Andrew Scherkus">scherkus</attacher>
            
              <data encoding="base64">ZGlmZiAtLWdpdCBhL1dlYkNvcmUvQ2hhbmdlTG9nIGIvV2ViQ29yZS9DaGFuZ2VMb2cKaW5kZXgg
Yzc3MmIyMC4uY2QwNjY1MCAxMDA2NDQKLS0tIGEvV2ViQ29yZS9DaGFuZ2VMb2cKKysrIGIvV2Vi
Q29yZS9DaGFuZ2VMb2cKQEAgLTEsMyArMSwxOCBAQAorMjAwOS0wOC0xOSAgQW5kcmV3IFNjaGVy
a3VzICA8c2NoZXJrdXNAY2hyb21pdW0ub3JnPgorCisgICAgICAgIFJldmlld2VkIGJ5IE5PQk9E
WSAoT09QUyEpLgorCisgICAgICAgIFJlbmRlciBkaXNhYmxlZCBtdXRlIGJ1dHRvbiBkdXJpbmcg
YW4gZXJyb3Igb3IgaWYgbm8gYXVkaW8gaXMgcHJlc2VudC4KKworICAgICAgICBodHRwczovL2J1
Z3Mud2Via2l0Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9Mjg0NzUKKworICAgICAgICBDb3ZlcmVkIGJ5
IGV4aXN0aW5nIGxheW91dCB0ZXN0czoKKyAgICAgICAgbWVkaWEvdmlkZW8tY29udHJvbHMtdmlz
aWJsZS1hdWRpby1vbmx5Lmh0bWwKKyAgICAgICAgbWVkaWEvdmlkZW8tbm8tYXVkaW8uaHRtbAor
CisgICAgICAgICogcmVuZGVyaW5nL1JlbmRlclRoZW1lQ2hyb21pdW1Ta2lhLmNwcDoKKyAgICAg
ICAgKFdlYkNvcmU6OlJlbmRlclRoZW1lQ2hyb21pdW1Ta2lhOjpwYWludE1lZGlhTXV0ZUJ1dHRv
bik6CisKIDIwMDktMDgtMTkgIERpcmsgU2NodWx6ZSAgPGtyaXRAd2Via2l0Lm9yZz4KIAogICAg
ICAgICBSZXZpZXdlZCBieSBPbGl2ZXIgSHVudC4KZGlmZiAtLWdpdCBhL1dlYkNvcmUvcmVuZGVy
aW5nL1JlbmRlclRoZW1lQ2hyb21pdW1Ta2lhLmNwcCBiL1dlYkNvcmUvcmVuZGVyaW5nL1JlbmRl
clRoZW1lQ2hyb21pdW1Ta2lhLmNwcAppbmRleCBjODg3NWRjLi5hMWM1YzM1IDEwMDY0NAotLS0g
YS9XZWJDb3JlL3JlbmRlcmluZy9SZW5kZXJUaGVtZUNocm9taXVtU2tpYS5jcHAKKysrIGIvV2Vi
Q29yZS9yZW5kZXJpbmcvUmVuZGVyVGhlbWVDaHJvbWl1bVNraWEuY3BwCkBAIC02MzMsNiArNjMz
LDEwIEBAIGJvb2wgUmVuZGVyVGhlbWVDaHJvbWl1bVNraWE6OnBhaW50TWVkaWFNdXRlQnV0dG9u
KFJlbmRlck9iamVjdCogb2JqZWN0LCBjb25zdCBSCiAKICAgICBzdGF0aWMgSW1hZ2UqIHNvdW5k
RnVsbCA9IEltYWdlOjpsb2FkUGxhdGZvcm1SZXNvdXJjZSgibWVkaWFTb3VuZEZ1bGwiKS5yZWxl
YXNlUmVmKCk7CiAgICAgc3RhdGljIEltYWdlKiBzb3VuZE5vbmUgPSBJbWFnZTo6bG9hZFBsYXRm
b3JtUmVzb3VyY2UoIm1lZGlhU291bmROb25lIikucmVsZWFzZVJlZigpOworICAgIHN0YXRpYyBJ
bWFnZSogc291bmREaXNhYmxlZCA9IEltYWdlOjpsb2FkUGxhdGZvcm1SZXNvdXJjZSgibWVkaWFT
b3VuZERpc2FibGVkIikucmVsZWFzZVJlZigpOworCisgICAgaWYgKG1lZGlhRWxlbWVudC0+bmV0
d29ya1N0YXRlKCkgPT0gSFRNTE1lZGlhRWxlbWVudDo6TkVUV09SS19OT19TT1VSQ0UgfHwgIW1l
ZGlhRWxlbWVudC0+aGFzQXVkaW8oKSkKKyAgICAgICAgcmV0dXJuIHBhaW50TWVkaWFCdXR0b25J
bnRlcm5hbChwYWludEluZm8uY29udGV4dCwgcmVjdCwgc291bmREaXNhYmxlZCk7CiAKICAgICBy
ZXR1cm4gcGFpbnRNZWRpYUJ1dHRvbkludGVybmFsKHBhaW50SW5mby5jb250ZXh0LCByZWN0LCBt
ZWRpYUVsZW1lbnQtPm11dGVkKCkgPyBzb3VuZE5vbmU6IHNvdW5kRnVsbCk7CiAjZWxzZQo=
</data>
<flag name="review"
          id="19355"
          type_id="1"
          status="+"
          setter="levin"
    />
    <flag name="commit-queue"
          id="19378"
          type_id="3"
          status="-"
          setter="eric"
    />
          </attachment>
      

    </bug>

</bugzilla>