<?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>194341</bug_id>
          
          <creation_ts>2019-02-06 05:23:51 -0800</creation_ts>
          <short_desc>Build failure after r240315</short_desc>
          <delta_ts>2019-02-06 07:41:28 -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>WebCore JavaScript</component>
          <version>WebKit Local Build</version>
          <rep_platform>All</rep_platform>
          <op_sys>All</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="Pablo Saavedra">psaavedra</reporter>
          <assigned_to name="Pablo Saavedra">psaavedra</assigned_to>
          <cc>commit-queue</cc>
    
    <cc>rniwa</cc>
    
    <cc>webkit-bug-importer</cc>
    
    <cc>wenson_hsieh</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>1502754</commentid>
    <comment_count>0</comment_count>
    <who name="Pablo Saavedra">psaavedra</who>
    <bug_when>2019-02-06 05:23:51 -0800</bug_when>
    <thetext>Description:

   Failure happens building WPE with no VIDEO enabled (proably happens as well in other ports):


  -DENABLE_VIDEO=OFF
  -DPORT=WPE 


Diagnosis:

  JSC namespace not added to the Webcore namespace  when video disabled




Build errors:

In file included from DerivedSources/WebCore/unified-sources/UnifiedSource-68aea4ac-14.cpp:6:
/home/igalia/psaavedra/yocto-wandboard-wpe/builds/wandboard-mesa-wpe-alternative/tmp/work/armv7at2hf-neon-imx-poky-linux-gnueabi/wpewebkit/nightly-AUTOINC+3646d86f28-r0/git/Source/WebCore/bindings/js/JSUndoItemCustom.cpp: At global scope:
/home/igalia/psaavedra/yocto-wandboard-wpe/builds/wandboard-mesa-wpe-alternative/tmp/work/armv7at2hf-neon-imx-poky-linux-gnueabi/wpewebkit/nightly-AUTOINC+3646d86f28-r0/git/Source/WebCore/bindings/js/JSUndoItemCustom.cpp:41:91: error: &apos;SlotVisitor&apos; has not been declared
 bool JSUndoItemOwner::isReachableFromOpaqueRoots(JSC::Handle&lt;JSC::Unknown&gt; handle, void*, SlotVisitor&amp; visitor, const char** reason)


In file included from DerivedSources/WebCore/unified-sources/UnifiedSource-68aea4ac-14.cpp:6:
/home/igalia/psaavedra/yocto-wandboard-wpe/builds/wandboard-mesa-wpe-alternative/tmp/work/armv7at2hf-neon-imx-poky-linux-gnueabi/wpewebkit/nightly-AUTOINC+3646d86f28-r0/git/Source/WebCore/bindings/js/JSUndoItemCustom.cpp: In member function &apos;virtual bool WebCore::JSUndoItemOwner::isReachableFromOpaqueRoots(JSC::Handle&lt;JSC::Unknown&gt;, void*, JSC::SlotVisitor&amp;, const char**)&apos;:
/home/igalia/psaavedra/yocto-wandboard-wpe/builds/wandboard-mesa-wpe-alternative/tmp/work/armv7at2hf-neon-imx-poky-linux-gnueabi/wpewebkit/nightly-AUTOINC+3646d86f28-r0/git/Source/WebCore/bindings/js/JSUndoItemCustom.cpp:46:33: error: &apos;jsCast&apos; was not declared in this scope
     auto* documentForUndoItem = jsCast&lt;JSUndoItem*&gt;(handle.slot()-&gt;asCell())-&gt;wrapped().document();
                                 ^~~~~~



Related to: https://bugs.webkit.org/show_bug.cgi?id=193111</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1502755</commentid>
    <comment_count>1</comment_count>
      <attachid>361292</attachid>
    <who name="Pablo Saavedra">psaavedra</who>
    <bug_when>2019-02-06 05:26:10 -0800</bug_when>
    <thetext>Created attachment 361292
patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1502762</commentid>
    <comment_count>2</comment_count>
      <attachid>361292</attachid>
    <who name="WebKit Commit Bot">commit-queue</who>
    <bug_when>2019-02-06 07:40:36 -0800</bug_when>
    <thetext>Comment on attachment 361292
patch

Clearing flags on attachment: 361292

Committed r241017: &lt;https://trac.webkit.org/changeset/241017&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1502763</commentid>
    <comment_count>3</comment_count>
    <who name="WebKit Commit Bot">commit-queue</who>
    <bug_when>2019-02-06 07:40:37 -0800</bug_when>
    <thetext>All reviewed patches have been landed.  Closing bug.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1502764</commentid>
    <comment_count>4</comment_count>
    <who name="Radar WebKit Bug Importer">webkit-bug-importer</who>
    <bug_when>2019-02-06 07:41:28 -0800</bug_when>
    <thetext>&lt;rdar://problem/47849690&gt;</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>361292</attachid>
            <date>2019-02-06 05:26:10 -0800</date>
            <delta_ts>2019-02-06 07:40:36 -0800</delta_ts>
            <desc>patch</desc>
            <filename>bug-194341-20190206142609.patch</filename>
            <type>text/plain</type>
            <size>1848</size>
            <attacher name="Pablo Saavedra">psaavedra</attacher>
            
              <data encoding="base64">U3VidmVyc2lvbiBSZXZpc2lvbjogMjQxMDE1CmRpZmYgLS1naXQgYS9Tb3VyY2UvV2ViQ29yZS9D
aGFuZ2VMb2cgYi9Tb3VyY2UvV2ViQ29yZS9DaGFuZ2VMb2cKaW5kZXggNzZkYTc1Y2I5YWRlOTI5
NGMyNjYyYjA1ZDZiNDMzYjNjZTUzMDdjMi4uZjEwYWUxMmRiNzc3ZDUxYmI5ZjAyODExZDRhYTg5
MjdhZjIwYjU3YSAxMDA2NDQKLS0tIGEvU291cmNlL1dlYkNvcmUvQ2hhbmdlTG9nCisrKyBiL1Nv
dXJjZS9XZWJDb3JlL0NoYW5nZUxvZwpAQCAtMSwzICsxLDEzIEBACisyMDE5LTAyLTA2ICBQYWJs
byBTYWF2ZWRyYSAgPHBzYWF2ZWRyYUBpZ2FsaWEuY29tPgorCisgICAgICAgIEJ1aWxkIGZhaWx1
cmUgYWZ0ZXIgcjI0MDMxNQorICAgICAgICBodHRwczovL2J1Z3Mud2Via2l0Lm9yZy9zaG93X2J1
Zy5jZ2k/aWQ9MTk0MzQxCisKKyAgICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BTISkuCisK
KyAgICAgICAgKiBiaW5kaW5ncy9qcy9KU1VuZG9JdGVtQ3VzdG9tLmNwcDoKKyAgICAgICAgKFdl
YkNvcmU6OkpTVW5kb0l0ZW1Pd25lcjo6aXNSZWFjaGFibGVGcm9tT3BhcXVlUm9vdHMpOgorCiAy
MDE5LTAyLTA1ICBSeW9zdWtlIE5pd2EgIDxybml3YUB3ZWJraXQub3JnPgogCiAgICAgICAgIFJF
R1JFU1NJT04gKHIyNDA5MDkpOiBSZWxlYXNlIGFzc2VydCBpbiBGcmFtZUxvYWRlcjo6bG9hZFVS
TCB3aGVuIG5hdmlnYXRpbmcgd2l0aCBhIG5vbi1leGlzdGVudCB0YXJnZXQgbmFtZQpkaWZmIC0t
Z2l0IGEvU291cmNlL1dlYkNvcmUvYmluZGluZ3MvanMvSlNVbmRvSXRlbUN1c3RvbS5jcHAgYi9T
b3VyY2UvV2ViQ29yZS9iaW5kaW5ncy9qcy9KU1VuZG9JdGVtQ3VzdG9tLmNwcAppbmRleCBjZmFh
MDI4YmJmZjVmZDViNGEzZjRhMGQ2MTAxMGZjYmU2NTNmYzc1Li4zZjZkZDlmZDAwZDg0NDIxMDVm
YTQxMWFjNjU4MzhjMTVjMWMxNGQ5IDEwMDY0NAotLS0gYS9Tb3VyY2UvV2ViQ29yZS9iaW5kaW5n
cy9qcy9KU1VuZG9JdGVtQ3VzdG9tLmNwcAorKysgYi9Tb3VyY2UvV2ViQ29yZS9iaW5kaW5ncy9q
cy9KU1VuZG9JdGVtQ3VzdG9tLmNwcApAQCAtMzQsMTIgKzM0LDEyIEBAIHZvaWQgSlNVbmRvSXRl
bTo6dmlzaXRBZGRpdGlvbmFsQ2hpbGRyZW4oSlNDOjpTbG90VmlzaXRvciYgdmlzaXRvcikKICAg
ICB3cmFwcGVkKCkucmVkb0hhbmRsZXIoKS52aXNpdEpTRnVuY3Rpb24odmlzaXRvcik7CiB9CiAK
LWJvb2wgSlNVbmRvSXRlbU93bmVyOjppc1JlYWNoYWJsZUZyb21PcGFxdWVSb290cyhKU0M6Okhh
bmRsZTxKU0M6OlVua25vd24+IGhhbmRsZSwgdm9pZCosIFNsb3RWaXNpdG9yJiB2aXNpdG9yLCBj
b25zdCBjaGFyKiogcmVhc29uKQorYm9vbCBKU1VuZG9JdGVtT3duZXI6OmlzUmVhY2hhYmxlRnJv
bU9wYXF1ZVJvb3RzKEpTQzo6SGFuZGxlPEpTQzo6VW5rbm93bj4gaGFuZGxlLCB2b2lkKiwgSlND
OjpTbG90VmlzaXRvciYgdmlzaXRvciwgY29uc3QgY2hhcioqIHJlYXNvbikKIHsKICAgICBpZiAo
VU5MSUtFTFkocmVhc29uKSkKICAgICAgICAgKnJlYXNvbiA9ICJEb2N1bWVudCBpcyBhbiBvcGFx
dWUgcm9vdC4iOwogCi0gICAgYXV0byogZG9jdW1lbnRGb3JVbmRvSXRlbSA9IGpzQ2FzdDxKU1Vu
ZG9JdGVtKj4oaGFuZGxlLnNsb3QoKS0+YXNDZWxsKCkpLT53cmFwcGVkKCkuZG9jdW1lbnQoKTsK
KyAgICBhdXRvKiBkb2N1bWVudEZvclVuZG9JdGVtID0gSlNDOjpqc0Nhc3Q8SlNVbmRvSXRlbSo+
KGhhbmRsZS5zbG90KCktPmFzQ2VsbCgpKS0+d3JhcHBlZCgpLmRvY3VtZW50KCk7CiAgICAgcmV0
dXJuIGRvY3VtZW50Rm9yVW5kb0l0ZW0gJiYgdmlzaXRvci5jb250YWluc09wYXF1ZVJvb3QoZG9j
dW1lbnRGb3JVbmRvSXRlbSk7CiB9CiAK
</data>

          </attachment>
      

    </bug>

</bugzilla>