<?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>97048</bug_id>
          
          <creation_ts>2012-09-18 15:01:25 -0700</creation_ts>
          <short_desc>Check settings before registering AVFoundation media engine.</short_desc>
          <delta_ts>2012-09-18 21:21:31 -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>Media</component>
          <version>528+ (Nightly build)</version>
          <rep_platform>Unspecified</rep_platform>
          <op_sys>Unspecified</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>Normal</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Eric Carlson">eric.carlson</reporter>
          <assigned_to name="Eric Carlson">eric.carlson</assigned_to>
          <cc>feature-media-reviews</cc>
    
    <cc>webkit.review.bot</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>723073</commentid>
    <comment_count>0</comment_count>
    <who name="Eric Carlson">eric.carlson</who>
    <bug_when>2012-09-18 15:01:25 -0700</bug_when>
    <thetext>r122676 mistakenly enabled the AVFoundation media engine without regard to the preference.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>723082</commentid>
    <comment_count>1</comment_count>
      <attachid>164626</attachid>
    <who name="Eric Carlson">eric.carlson</who>
    <bug_when>2012-09-18 15:16:07 -0700</bug_when>
    <thetext>Created attachment 164626
Proposed patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>723083</commentid>
    <comment_count>2</comment_count>
    <who name="Eric Carlson">eric.carlson</who>
    <bug_when>2012-09-18 15:17:05 -0700</bug_when>
    <thetext>&lt;rdar://problem/12313594&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>723086</commentid>
    <comment_count>3</comment_count>
      <attachid>164628</attachid>
    <who name="Eric Carlson">eric.carlson</who>
    <bug_when>2012-09-18 15:19:54 -0700</bug_when>
    <thetext>Created attachment 164628
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>723246</commentid>
    <comment_count>4</comment_count>
      <attachid>164628</attachid>
    <who name="WebKit Review Bot">webkit.review.bot</who>
    <bug_when>2012-09-18 21:21:28 -0700</bug_when>
    <thetext>Comment on attachment 164628
Patch

Clearing flags on attachment: 164628

Committed r128964: &lt;http://trac.webkit.org/changeset/128964&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>723247</commentid>
    <comment_count>5</comment_count>
    <who name="WebKit Review Bot">webkit.review.bot</who>
    <bug_when>2012-09-18 21:21:31 -0700</bug_when>
    <thetext>All reviewed patches have been landed.  Closing bug.</thetext>
  </long_desc>
      
          <attachment
              isobsolete="1"
              ispatch="1"
              isprivate="0"
          >
            <attachid>164626</attachid>
            <date>2012-09-18 15:16:07 -0700</date>
            <delta_ts>2012-09-18 15:19:54 -0700</delta_ts>
            <desc>Proposed patch</desc>
            <filename>check_preferences_patch_1.txt</filename>
            <type>text/plain</type>
            <size>1365</size>
            <attacher name="Eric Carlson">eric.carlson</attacher>
            
              <data encoding="base64">SW5kZXg6IFNvdXJjZS9XZWJDb3JlL0NoYW5nZUxvZwo9PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBTb3VyY2UvV2Vi
Q29yZS9DaGFuZ2VMb2cJKHJldmlzaW9uIDEyODkzNSkKKysrIFNvdXJjZS9XZWJDb3JlL0NoYW5n
ZUxvZwkod29ya2luZyBjb3B5KQpAQCAtMSwzICsxLDE1IEBACisyMDEyLTA5LTE4ICBFcmljIENh
cmxzb24gIDxlcmljLmNhcmxzb25AYXBwbGUuY29tPgorCisgICAgICAgIENoZWNrIHNldHRpbmdz
IGJlZm9yZSByZWdpc3RlcmluZyBBVkZvdW5kYXRpb24gbWVkaWEgZW5naW5lLgorICAgICAgICBo
dHRwczovL2J1Z3Mud2Via2l0Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9OTcwNDgKKworICAgICAgICBS
ZXZpZXdlZCBieSBOT0JPRFkgKE9PUFMhKS4KKworICAgICAgICBGaXggdGhlIGJ1ZyBpbnRyb2R1
Y2VkIGluIHIxMjI2NzYuCisKKyAgICAgICAgKiBwbGF0Zm9ybS9ncmFwaGljcy9NZWRpYVBsYXll
ci5jcHA6CisgICAgICAgIChXZWJDb3JlOjppbnN0YWxsZWRNZWRpYUVuZ2luZXMpOiBVbmNvbW1l
bnQgdGhlIGNhbGwgdG8gY2hlY2sgQVZGb3VuZGF0aW9uIHNldHRpbmdzLgorCiAyMDEyLTA5LTE4
ICBKb2huIE1lbGxvciAgPGpvaG5tZUBjaHJvbWl1bS5vcmc+CiAKICAgICAgICAgVGV4dCBBdXRv
c2l6aW5nOiBJZ25vcmUgY29uc3RyYWluZWQgaGVpZ2h0cyBpbiBjZXJ0YWluIGNpcmN1bXN0YW5j
ZXMuCkluZGV4OiBTb3VyY2UvV2ViQ29yZS9wbGF0Zm9ybS9ncmFwaGljcy9NZWRpYVBsYXllci5j
cHAKPT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PQotLS0gU291cmNlL1dlYkNvcmUvcGxhdGZvcm0vZ3JhcGhpY3MvTWVkaWFQ
bGF5ZXIuY3BwCShyZXZpc2lvbiAxMjg5MzQpCisrKyBTb3VyY2UvV2ViQ29yZS9wbGF0Zm9ybS9n
cmFwaGljcy9NZWRpYVBsYXllci5jcHAJKHdvcmtpbmcgY29weSkKQEAgLTIxMCw3ICsyMTAsNyBA
QCBzdGF0aWMgVmVjdG9yPE1lZGlhUGxheWVyRmFjdG9yeSo+JiBpbnN0CiAgICAgICAgIGVuZ2lu
ZXNRdWVyaWVkID0gdHJ1ZTsKIAogI2lmIFVTRShBVkZPVU5EQVRJT04pCi0gICAgICAgIGlmICgx
IC8qIEBAU2V0dGluZ3M6OmlzQVZGb3VuZGF0aW9uRW5hYmxlZCgpIEBAICovKSB7CisgICAgICAg
IGlmIChTZXR0aW5nczo6aXNBVkZvdW5kYXRpb25FbmFibGVkKCkpIHsKICNpZiBQTEFURk9STShN
QUMpCiAgICAgICAgICAgICBNZWRpYVBsYXllclByaXZhdGVBVkZvdW5kYXRpb25PYmpDOjpyZWdp
c3Rlck1lZGlhRW5naW5lKGFkZE1lZGlhRW5naW5lKTsKICNlbGlmIFBMQVRGT1JNKFdJTikK
</data>

          </attachment>
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>164628</attachid>
            <date>2012-09-18 15:19:54 -0700</date>
            <delta_ts>2012-09-18 21:21:28 -0700</delta_ts>
            <desc>Patch</desc>
            <filename>check_preferences_patch_1.txt</filename>
            <type>text/plain</type>
            <size>1400</size>
            <attacher name="Eric Carlson">eric.carlson</attacher>
            
              <data encoding="base64">SW5kZXg6IFNvdXJjZS9XZWJDb3JlL0NoYW5nZUxvZwo9PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBTb3VyY2UvV2Vi
Q29yZS9DaGFuZ2VMb2cJKHJldmlzaW9uIDEyODkzNSkKKysrIFNvdXJjZS9XZWJDb3JlL0NoYW5n
ZUxvZwkod29ya2luZyBjb3B5KQpAQCAtMSwzICsxLDE2IEBACisyMDEyLTA5LTE4ICBFcmljIENh
cmxzb24gIDxlcmljLmNhcmxzb25AYXBwbGUuY29tPgorCisgICAgICAgIENoZWNrIHNldHRpbmdz
IGJlZm9yZSByZWdpc3RlcmluZyBBVkZvdW5kYXRpb24gbWVkaWEgZW5naW5lLgorICAgICAgICBo
dHRwczovL2J1Z3Mud2Via2l0Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9OTcwNDgKKyAgICAgICAgPHJk
YXI6Ly9wcm9ibGVtLzEyMzEzNTk0PgorCisgICAgICAgIFJldmlld2VkIGJ5IE5PQk9EWSAoT09Q
UyEpLgorCisgICAgICAgIEZpeCB0aGUgYnVnIGludHJvZHVjZWQgaW4gcjEyMjY3Ni4KKworICAg
ICAgICAqIHBsYXRmb3JtL2dyYXBoaWNzL01lZGlhUGxheWVyLmNwcDoKKyAgICAgICAgKFdlYkNv
cmU6Omluc3RhbGxlZE1lZGlhRW5naW5lcyk6IFVuY29tbWVudCB0aGUgY2FsbCB0byBjaGVjayBB
VkZvdW5kYXRpb24gc2V0dGluZ3MuCisKIDIwMTItMDktMTggIEpvaG4gTWVsbG9yICA8am9obm1l
QGNocm9taXVtLm9yZz4KIAogICAgICAgICBUZXh0IEF1dG9zaXppbmc6IElnbm9yZSBjb25zdHJh
aW5lZCBoZWlnaHRzIGluIGNlcnRhaW4gY2lyY3Vtc3RhbmNlcy4KSW5kZXg6IFNvdXJjZS9XZWJD
b3JlL3BsYXRmb3JtL2dyYXBoaWNzL01lZGlhUGxheWVyLmNwcAo9PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBTb3Vy
Y2UvV2ViQ29yZS9wbGF0Zm9ybS9ncmFwaGljcy9NZWRpYVBsYXllci5jcHAJKHJldmlzaW9uIDEy
ODkzNCkKKysrIFNvdXJjZS9XZWJDb3JlL3BsYXRmb3JtL2dyYXBoaWNzL01lZGlhUGxheWVyLmNw
cAkod29ya2luZyBjb3B5KQpAQCAtMjEwLDcgKzIxMCw3IEBAIHN0YXRpYyBWZWN0b3I8TWVkaWFQ
bGF5ZXJGYWN0b3J5Kj4mIGluc3QKICAgICAgICAgZW5naW5lc1F1ZXJpZWQgPSB0cnVlOwogCiAj
aWYgVVNFKEFWRk9VTkRBVElPTikKLSAgICAgICAgaWYgKDEgLyogQEBTZXR0aW5nczo6aXNBVkZv
dW5kYXRpb25FbmFibGVkKCkgQEAgKi8pIHsKKyAgICAgICAgaWYgKFNldHRpbmdzOjppc0FWRm91
bmRhdGlvbkVuYWJsZWQoKSkgewogI2lmIFBMQVRGT1JNKE1BQykKICAgICAgICAgICAgIE1lZGlh
UGxheWVyUHJpdmF0ZUFWRm91bmRhdGlvbk9iakM6OnJlZ2lzdGVyTWVkaWFFbmdpbmUoYWRkTWVk
aWFFbmdpbmUpOwogI2VsaWYgUExBVEZPUk0oV0lOKQo=
</data>

          </attachment>
      

    </bug>

</bugzilla>