<?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>206913</bug_id>
          
          <creation_ts>2020-01-28 15:56:05 -0800</creation_ts>
          <short_desc>TestWebKitAPI: Conditionalize device orientation tests on ENABLE(DEVICE_ORIENTATION)</short_desc>
          <delta_ts>2020-01-28 16:55:30 -0800</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>1</classification_id>
          <classification>Unclassified</classification>
          <product>WebKit</product>
          <component>Tools / Tests</component>
          <version>WebKit 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>InRadar</keywords>
          <priority>P2</priority>
          <bug_severity>Normal</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Jonathan Bedard">jbedard</reporter>
          <assigned_to name="Jonathan Bedard">jbedard</assigned_to>
          <cc>cdumez</cc>
    
    <cc>commit-queue</cc>
    
    <cc>thorton</cc>
    
    <cc>webkit-bug-importer</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>1612230</commentid>
    <comment_count>0</comment_count>
    <who name="Jonathan Bedard">jbedard</who>
    <bug_when>2020-01-28 15:56:05 -0800</bug_when>
    <thetext>It doesn&apos;t make sense to attempt to run device orientation tests when on a platform that has DEVICE_ORIENTATION disabled.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1612231</commentid>
    <comment_count>1</comment_count>
    <who name="Radar WebKit Bug Importer">webkit-bug-importer</who>
    <bug_when>2020-01-28 15:56:52 -0800</bug_when>
    <thetext>&lt;rdar://problem/58973357&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1612233</commentid>
    <comment_count>2</comment_count>
      <attachid>389082</attachid>
    <who name="Jonathan Bedard">jbedard</who>
    <bug_when>2020-01-28 15:58:27 -0800</bug_when>
    <thetext>Created attachment 389082
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1612236</commentid>
    <comment_count>3</comment_count>
      <attachid>389082</attachid>
    <who name="Chris Dumez">cdumez</who>
    <bug_when>2020-01-28 16:06:16 -0800</bug_when>
    <thetext>Comment on attachment 389082
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=389082&amp;action=review

&gt; Tools/TestWebKitAPI/Tests/WebKitCocoa/DeviceOrientation.mm:28
&gt; +#if ENABLE(DEVICE_ORIENTATION) &amp;&amp; PLATFORM(IOS_FAMILY)

Please update this comment at the end of the file as well:
#endif // PLATFORM(IOS_FAMILY)</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1612240</commentid>
    <comment_count>4</comment_count>
      <attachid>389084</attachid>
    <who name="Jonathan Bedard">jbedard</who>
    <bug_when>2020-01-28 16:13:05 -0800</bug_when>
    <thetext>Created attachment 389084
Patch for landing</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1612263</commentid>
    <comment_count>5</comment_count>
      <attachid>389084</attachid>
    <who name="WebKit Commit Bot">commit-queue</who>
    <bug_when>2020-01-28 16:55:28 -0800</bug_when>
    <thetext>Comment on attachment 389084
Patch for landing

Clearing flags on attachment: 389084

Committed r255328: &lt;https://trac.webkit.org/changeset/255328&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1612264</commentid>
    <comment_count>6</comment_count>
    <who name="WebKit Commit Bot">commit-queue</who>
    <bug_when>2020-01-28 16:55:30 -0800</bug_when>
    <thetext>All reviewed patches have been landed.  Closing bug.</thetext>
  </long_desc>
      
          <attachment
              isobsolete="1"
              ispatch="1"
              isprivate="0"
          >
            <attachid>389082</attachid>
            <date>2020-01-28 15:58:27 -0800</date>
            <delta_ts>2020-01-28 16:13:03 -0800</delta_ts>
            <desc>Patch</desc>
            <filename>bug-206913-20200128155826.patch</filename>
            <type>text/plain</type>
            <size>1112</size>
            <attacher name="Jonathan Bedard">jbedard</attacher>
            
              <data encoding="base64">SW5kZXg6IFRvb2xzL0NoYW5nZUxvZwo9PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBUb29scy9DaGFuZ2VMb2cJKHJl
dmlzaW9uIDI1NTMxOSkKKysrIFRvb2xzL0NoYW5nZUxvZwkod29ya2luZyBjb3B5KQpAQCAtMSwz
ICsxLDEzIEBACisyMDIwLTAxLTI4ICBKb25hdGhhbiBCZWRhcmQgIDxqYmVkYXJkQGFwcGxlLmNv
bT4KKworICAgICAgICBUZXN0V2ViS2l0QVBJOiBDb25kaXRpb25hbGl6ZSBkZXZpY2Ugb3JpZW50
YXRpb24gdGVzdHMgb24gRU5BQkxFKERFVklDRV9PUklFTlRBVElPTikKKyAgICAgICAgaHR0cHM6
Ly9idWdzLndlYmtpdC5vcmcvc2hvd19idWcuY2dpP2lkPTIwNjkxMworICAgICAgICA8cmRhcjov
L3Byb2JsZW0vNTg5NzMzNTc+CisKKyAgICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BTISku
CisKKyAgICAgICAgKiBUZXN0V2ViS2l0QVBJL1Rlc3RzL1dlYktpdENvY29hL0RldmljZU9yaWVu
dGF0aW9uLm1tOgorCiAyMDIwLTAxLTI4ICBTaWh1aSBMaXUgIDxzaWh1aV9saXVAYXBwbGUuY29t
PgogCiAgICAgICAgIEluZGV4ZWREQjogc3BlZWQgdXAgaW5kZXggcmVjb3JkcyBkZWxldGlvbgpJ
bmRleDogVG9vbHMvVGVzdFdlYktpdEFQSS9UZXN0cy9XZWJLaXRDb2NvYS9EZXZpY2VPcmllbnRh
dGlvbi5tbQo9PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09Ci0tLSBUb29scy9UZXN0V2ViS2l0QVBJL1Rlc3RzL1dlYktpdENv
Y29hL0RldmljZU9yaWVudGF0aW9uLm1tCShyZXZpc2lvbiAyNTUzMTQpCisrKyBUb29scy9UZXN0
V2ViS2l0QVBJL1Rlc3RzL1dlYktpdENvY29hL0RldmljZU9yaWVudGF0aW9uLm1tCSh3b3JraW5n
IGNvcHkpCkBAIC0yNSw3ICsyNSw3IEBACiAKICNpbXBvcnQgImNvbmZpZy5oIgogCi0jaWYgUExB
VEZPUk0oSU9TX0ZBTUlMWSkKKyNpZiBFTkFCTEUoREVWSUNFX09SSUVOVEFUSU9OKSAmJiBQTEFU
Rk9STShJT1NfRkFNSUxZKQogCiAjaW1wb3J0ICJQbGF0Zm9ybVV0aWxpdGllcy5oIgogI2ltcG9y
dCAiVGVzdE5hdmlnYXRpb25EZWxlZ2F0ZS5oIgo=
</data>

          </attachment>
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>389084</attachid>
            <date>2020-01-28 16:13:05 -0800</date>
            <delta_ts>2020-01-28 16:55:28 -0800</delta_ts>
            <desc>Patch for landing</desc>
            <filename>bug-206913-20200128161305.patch</filename>
            <type>text/plain</type>
            <size>1374</size>
            <attacher name="Jonathan Bedard">jbedard</attacher>
            
              <data encoding="base64">SW5kZXg6IFRvb2xzL0NoYW5nZUxvZwo9PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBUb29scy9DaGFuZ2VMb2cJKHJl
dmlzaW9uIDI1NTMyMCkKKysrIFRvb2xzL0NoYW5nZUxvZwkod29ya2luZyBjb3B5KQpAQCAtMSwz
ICsxLDEzIEBACisyMDIwLTAxLTI4ICBKb25hdGhhbiBCZWRhcmQgIDxqYmVkYXJkQGFwcGxlLmNv
bT4KKworICAgICAgICBUZXN0V2ViS2l0QVBJOiBDb25kaXRpb25hbGl6ZSBkZXZpY2Ugb3JpZW50
YXRpb24gdGVzdHMgb24gRU5BQkxFKERFVklDRV9PUklFTlRBVElPTikKKyAgICAgICAgaHR0cHM6
Ly9idWdzLndlYmtpdC5vcmcvc2hvd19idWcuY2dpP2lkPTIwNjkxMworICAgICAgICA8cmRhcjov
L3Byb2JsZW0vNTg5NzMzNTc+CisKKyAgICAgICAgUmV2aWV3ZWQgYnkgQ2hyaXMgRHVtZXouCisK
KyAgICAgICAgKiBUZXN0V2ViS2l0QVBJL1Rlc3RzL1dlYktpdENvY29hL0RldmljZU9yaWVudGF0
aW9uLm1tOgorCiAyMDIwLTAxLTI4ICBKb25hdGhhbiBCZWRhcmQgIDxqYmVkYXJkQGFwcGxlLmNv
bT4KIAogICAgICAgICBUZXN0V2ViS2l0QVBJOiBDb25kaXRpb25hbGl6ZSBRdWlja0xvb2sgdGVz
dHMgb24gVVNFKFFVSUNLX0xPT0spCkluZGV4OiBUb29scy9UZXN0V2ViS2l0QVBJL1Rlc3RzL1dl
YktpdENvY29hL0RldmljZU9yaWVudGF0aW9uLm1tCj09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIFRvb2xzL1Rlc3RX
ZWJLaXRBUEkvVGVzdHMvV2ViS2l0Q29jb2EvRGV2aWNlT3JpZW50YXRpb24ubW0JKHJldmlzaW9u
IDI1NTMyMCkKKysrIFRvb2xzL1Rlc3RXZWJLaXRBUEkvVGVzdHMvV2ViS2l0Q29jb2EvRGV2aWNl
T3JpZW50YXRpb24ubW0JKHdvcmtpbmcgY29weSkKQEAgLTI1LDcgKzI1LDcgQEAKIAogI2ltcG9y
dCAiY29uZmlnLmgiCiAKLSNpZiBQTEFURk9STShJT1NfRkFNSUxZKQorI2lmIEVOQUJMRShERVZJ
Q0VfT1JJRU5UQVRJT04pICYmIFBMQVRGT1JNKElPU19GQU1JTFkpCiAKICNpbXBvcnQgIlBsYXRm
b3JtVXRpbGl0aWVzLmgiCiAjaW1wb3J0ICJUZXN0TmF2aWdhdGlvbkRlbGVnYXRlLmgiCkBAIC00
MDIsNCArNDAyLDQgQEAgVEVTVChEZXZpY2VPcmllbnRhdGlvbiwgUGVybWlzc2lvblNlY3VyZQog
ICAgIHJ1blBlcm1pc3Npb25TZWN1cmVDb250ZXh0Q2hlY2tUZXN0KFNob3VsZEVuYWJsZVNlY3Vy
ZUNvbnRleHRDaGVja3M6Ok5vKTsKIH0KIAotI2VuZGlmIC8vIFBMQVRGT1JNKElPU19GQU1JTFkp
CisjZW5kaWYgLy8gRU5BQkxFKERFVklDRV9PUklFTlRBVElPTikgJiYgUExBVEZPUk0oSU9TX0ZB
TUlMWSkK
</data>

          </attachment>
      

    </bug>

</bugzilla>