<?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>195550</bug_id>
          
          <creation_ts>2019-03-11 03:32:20 -0700</creation_ts>
          <short_desc>[CoordinatedGraphics] ASSERTION FAILED: !m_state.isSuspended</short_desc>
          <delta_ts>2019-03-24 19:31:25 -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>Platform</component>
          <version>WebKit Nightly Build</version>
          <rep_platform>Unspecified</rep_platform>
          <op_sys>Unspecified</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>FIXED</resolution>
          
          <see_also>https://bugs.webkit.org/show_bug.cgi?id=196190</see_also>
          <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="Fujii Hironori">fujii</reporter>
          <assigned_to name="Fujii Hironori">fujii</assigned_to>
          <cc>cgarcia</cc>
    
    <cc>webkit-bug-importer</cc>
    
    <cc>zan</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>1514731</commentid>
    <comment_count>0</comment_count>
    <who name="Fujii Hironori">fujii</who>
    <bug_when>2019-03-11 03:32:20 -0700</bug_when>
    <thetext>[CoordinatedGraphics] ASSERTION FAILED: !m_state.isSuspended

I&apos;m working on enabling Coordinated Graphics for WinCairo port (Bug 186364).
Some compositing LayoutTests are failing due to the following assertion failures randomly.

&gt; python ./Tools/Scripts/run-webkit-tests --debug  --wincairo --no-new-test-results compositing

&gt; ASSERTION FAILED: !m_state.isSuspended
&gt; ..\..\Source\WebKit\Shared\CoordinatedGraphics\threadedcompositor\CompositingRunLoop.cpp(225) : updateTimerFired
&gt; 1   00007FF94B331429 WTFCrash
&gt; 2   00007FF90EE90680 WTFCrashWithInfo
&gt; 3   00007FF90EF69EAC WebKit::CompositingRunLoop::updateTimerFired
&gt; 4   00007FF90EF6C6E2 WTF::RunLoop::Timer&lt;WebKit::CompositingRunLoop&gt;::fired
&gt; 5   00007FF94B45A610 WTF::RunLoop::TimerBase::timerFired
&gt; 6   00007FF94B45A495 WTF::RunLoop::wndProc
&gt; 7   00007FF94B45A389 WTF::RunLoop::RunLoopWndProc
&gt; 8   00007FF976F06D41 CallWindowProcW
&gt; 9   00007FF976F06713 DispatchMessageW
&gt; 10  00007FF94B45A693 WTF::RunLoop::run
&gt; 11  00007FF90EF6AC24 WebKit::createRunLoop::&lt;unnamed-tag&gt;::operator()
&gt; 12  00007FF90EF6ABC7 WTF::Function&lt;void ()&gt;::CallableWrapper&lt;`lambda at ..\..\Source\WebKit\Shared\CoordinatedGraphics\threadedcompositor\CompositingRunLoop.cpp:46:53&apos;&gt;::call
&gt; 13  00007FF94B345040 WTF::Function&lt;void ()&gt;::operator()
&gt; 14  00007FF94B3994A3 WTF::Thread::entryPoint
&gt; 15  00007FF94B45BBB3 WTF::wtfThreadEntryPoint
&gt; 16  00007FF97527C4CE o_ceil
&gt; 17  00007FF977DD3DC4 BaseThreadInitThunk
&gt; 18  00007FF9780E3691 RtlUserThreadStart
&gt; #CRASHED - WebProcess (pid 18052)</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1514732</commentid>
    <comment_count>1</comment_count>
      <attachid>364244</attachid>
    <who name="Fujii Hironori">fujii</who>
    <bug_when>2019-03-11 03:42:38 -0700</bug_when>
    <thetext>Created attachment 364244
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1514735</commentid>
    <comment_count>2</comment_count>
    <who name="Carlos Garcia Campos">cgarcia</who>
    <bug_when>2019-03-11 03:58:56 -0700</bug_when>
    <thetext>Could you show the bt of the other threads too, please?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1514737</commentid>
    <comment_count>3</comment_count>
    <who name="Fujii Hironori">fujii</who>
    <bug_when>2019-03-11 04:36:40 -0700</bug_when>
    <thetext>Here is the callstack of main thread.

&gt; [External Code]	
&gt; WTF.dll!WTF::ThreadCondition::timedWait(WTF::Mutex &amp; mutex, WTF::WallTime absoluteTime) Line 414	C++
&gt; WTF.dll!WTF::ParkingLot::parkConditionallyImpl(const void * address, const WTF::ScopedLambda&lt;bool ()&gt; &amp; validation, const WTF::ScopedLambda&lt;void ()&gt; &amp; beforeSleep, const WTF::TimeWithDynamicClockType &amp; timeout) Line 603	C++
&gt; WebKit2.dll!WTF::ParkingLot::parkConditionally&lt;`lambda at DerivedSources\ForwardingHeaders\wtf/Condition.h:77:17&apos;,`lambda at DerivedSources\ForwardingHeaders\wtf/Condition.h:83:17&apos;&gt;(const void * address, const WTF::Condition::waitUntil&lt;WTF::Lock&gt;::&lt;unnamed-tag&gt; &amp; validation, const WTF::Condition::waitUntil&lt;WTF::Lock&gt;::&lt;unnamed-tag&gt; &amp; beforeSleep, const WTF::TimeWithDynamicClockType &amp; timeout) Line 80	C++
&gt; WebKit2.dll!WTF::Condition::waitUntil&lt;WTF::Lock&gt;(WTF::Lock &amp; lock, const WTF::TimeWithDynamicClockType &amp; timeout) Line 75	C++
&gt; WebKit2.dll!WTF::Condition::wait&lt;WTF::Lock&gt;(WTF::Lock &amp; lock) Line 121	C++
&gt; WebKit2.dll!WebKit::CompositingRunLoop::performTaskSync(WTF::Function&lt;void ()&gt; &amp;&amp; function) Line 95	C++
&gt; WebKit2.dll!WebKit::ThreadedCompositor::suspend() Line 125	C++
&gt; WebKit2.dll!WebKit::LayerTreeHost::pauseRendering() Line 254	C++
&gt; WebKit2.dll!WebKit::DrawingAreaCoordinatedGraphics::exitAcceleratedCompositingMode() Line 594	C++
&gt; WebKit2.dll!WTF::RunLoop::Timer&lt;WebKit::DrawingAreaCoordinatedGraphics&gt;::fired() Line 152	C++
&gt; WTF.dll!WTF::RunLoop::TimerBase::timerFired() Line 141	C++
&gt; WTF.dll!WTF::RunLoop::wndProc(HWND__ * hWnd, unsigned int message, unsigned __int64 wParam, __int64 lParam) Line 60	C++
&gt; WTF.dll!WTF::RunLoop::RunLoopWndProc(HWND__ * hWnd, unsigned int message, unsigned __int64 wParam, __int64 lParam) Line 39	C++
&gt; [External Code]	
&gt; WTF.dll!WTF::RunLoop::run() Line 69	C++
&gt; WebKit2.dll!WebKit::AuxiliaryProcessMain&lt;WebKit::WebProcess,WebKit::WebProcessMain&gt;(int argc, char * * argv) Line 67	C++
&gt; WebKit2.dll!WebKit::WebProcessMainWin(int argc, char * * argv) Line 46	C++
&gt; WebKitWebProcess.exe!main(int argc, char * * argv) Line 33	C++
&gt; [External Code]	

ThreadedCompositor::suspend is calling the next function.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1515253</commentid>
    <comment_count>4</comment_count>
      <attachid>364244</attachid>
    <who name="Fujii Hironori">fujii</who>
    <bug_when>2019-03-11 21:12:57 -0700</bug_when>
    <thetext>Comment on attachment 364244
Patch

Clearing flags on attachment: 364244

Committed r242771: &lt;https://trac.webkit.org/changeset/242771&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1515254</commentid>
    <comment_count>5</comment_count>
    <who name="Fujii Hironori">fujii</who>
    <bug_when>2019-03-11 21:13:00 -0700</bug_when>
    <thetext>All reviewed patches have been landed.  Closing bug.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1515259</commentid>
    <comment_count>6</comment_count>
    <who name="Radar WebKit Bug Importer">webkit-bug-importer</who>
    <bug_when>2019-03-11 21:17:53 -0700</bug_when>
    <thetext>&lt;rdar://problem/48796633&gt;</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>364244</attachid>
            <date>2019-03-11 03:42:38 -0700</date>
            <delta_ts>2019-03-11 21:12:57 -0700</delta_ts>
            <desc>Patch</desc>
            <filename>bug-195550-20190311194237.patch</filename>
            <type>text/plain</type>
            <size>1835</size>
            <attacher name="Fujii Hironori">fujii</attacher>
            
              <data encoding="base64">U3VidmVyc2lvbiBSZXZpc2lvbjogMjQyNjk3CmRpZmYgLS1naXQgYS9Tb3VyY2UvV2ViS2l0L0No
YW5nZUxvZyBiL1NvdXJjZS9XZWJLaXQvQ2hhbmdlTG9nCmluZGV4IDVhYmY1ODY5YTY4MTJjMzYy
ODIzZWJhMGM1MzZlMDMxMTM2NTA5MTEuLjY5ZjZkNDQyM2ZkM2M1OWFhOWQ3NmI2NTIyN2VhY2Uz
ODY5ZjRkOTggMTAwNjQ0Ci0tLSBhL1NvdXJjZS9XZWJLaXQvQ2hhbmdlTG9nCisrKyBiL1NvdXJj
ZS9XZWJLaXQvQ2hhbmdlTG9nCkBAIC0xLDMgKzEsMTggQEAKKzIwMTktMDMtMTEgIEZ1amlpIEhp
cm9ub3JpICA8SGlyb25vcmkuRnVqaWlAc29ueS5jb20+CisKKyAgICAgICAgW0Nvb3JkaW5hdGVk
R3JhcGhpY3NdIEFTU0VSVElPTiBGQUlMRUQ6ICFtX3N0YXRlLmlzU3VzcGVuZGVkCisgICAgICAg
IGh0dHBzOi8vYnVncy53ZWJraXQub3JnL3Nob3dfYnVnLmNnaT9pZD0xOTU1NTAKKworICAgICAg
ICBSZXZpZXdlZCBieSBOT0JPRFkgKE9PUFMhKS4KKworICAgICAgICBDb21wb3NpdGluZ1J1bkxv
b3A6OnN1c3BlbmQoKSBsb2NrcyBhIG11dGV4IGFuZCBzdG9wcyB0aGUgdXBkYXRlCisgICAgICAg
IHRpbWVyLiBCdXQsIHRoZSB0aW1lciBjYW4gYmUgZmlyZWQgYWZ0ZXIgdGhlIGxvY2sgd2FzIGFj
cXVpcmVkIGFuZAorICAgICAgICBiZWZvcmUgdGhlIHRpbWVyIGlzIHN0b3BwZWQuCisKKyAgICAg
ICAgKiBTaGFyZWQvQ29vcmRpbmF0ZWRHcmFwaGljcy90aHJlYWRlZGNvbXBvc2l0b3IvQ29tcG9z
aXRpbmdSdW5Mb29wLmNwcDoKKyAgICAgICAgKFdlYktpdDo6Q29tcG9zaXRpbmdSdW5Mb29wOjp1
cGRhdGVUaW1lckZpcmVkKTogUmVtb3ZlZCB0aGUKKyAgICAgICAgYXNzZXJ0aW9uLiBSZXR1cm4g
ZWFybHkgaWYgbV9zdGF0ZS5pc1N1c3BlbmRlZC4KKwogMjAxOS0wMy0xMCAgSmlld2VuIFRhbiAg
PGppZXdlbl90YW5AYXBwbGUuY29tPgogCiAgICAgICAgIE9wdGltaXppbmcgbG9hZHMgd2hlbiBj
cmVhdGluZyBuZXcgcGFnZXMKZGlmZiAtLWdpdCBhL1NvdXJjZS9XZWJLaXQvU2hhcmVkL0Nvb3Jk
aW5hdGVkR3JhcGhpY3MvdGhyZWFkZWRjb21wb3NpdG9yL0NvbXBvc2l0aW5nUnVuTG9vcC5jcHAg
Yi9Tb3VyY2UvV2ViS2l0L1NoYXJlZC9Db29yZGluYXRlZEdyYXBoaWNzL3RocmVhZGVkY29tcG9z
aXRvci9Db21wb3NpdGluZ1J1bkxvb3AuY3BwCmluZGV4IDg3YzZlZTg2MjdkZDZhMjg1NGQ4OTA3
ZGYwMmZkN2I5ZTM0YzJhZDQuLmM4MDg0YmRhZGIyYThlMjcwY2I5OGIyODNkNDY4ZDU2Y2M4MjVl
MzUgMTAwNjQ0Ci0tLSBhL1NvdXJjZS9XZWJLaXQvU2hhcmVkL0Nvb3JkaW5hdGVkR3JhcGhpY3Mv
dGhyZWFkZWRjb21wb3NpdG9yL0NvbXBvc2l0aW5nUnVuTG9vcC5jcHAKKysrIGIvU291cmNlL1dl
YktpdC9TaGFyZWQvQ29vcmRpbmF0ZWRHcmFwaGljcy90aHJlYWRlZGNvbXBvc2l0b3IvQ29tcG9z
aXRpbmdSdW5Mb29wLmNwcApAQCAtMjIyLDcgKzIyMiw4IEBAIHZvaWQgQ29tcG9zaXRpbmdSdW5M
b29wOjp1cGRhdGVUaW1lckZpcmVkKCkKICAgICB7CiAgICAgICAgIC8vIEJvdGggY29tcG9zaXRp
b24gYW5kIHNjZW5lIHVwZGF0ZSBhcmUgbm93IGluIHByb2dyZXNzLgogICAgICAgICBMb2NrSG9s
ZGVyIGxvY2tlcihtX3N0YXRlLmxvY2spOwotICAgICAgICBBU1NFUlQoIW1fc3RhdGUuaXNTdXNw
ZW5kZWQpOworICAgICAgICBpZiAobV9zdGF0ZS5pc1N1c3BlbmRlZCkKKyAgICAgICAgICAgIHJl
dHVybjsKICAgICAgICAgbV9zdGF0ZS5jb21wb3NpdGlvbiA9IENvbXBvc2l0aW9uU3RhdGU6Oklu
UHJvZ3Jlc3M7CiAgICAgICAgIG1fc3RhdGUudXBkYXRlID0gVXBkYXRlU3RhdGU6OkluUHJvZ3Jl
c3M7CiAgICAgfQo=
</data>

          </attachment>
      

    </bug>

</bugzilla>