<?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>209467</bug_id>
          
          <creation_ts>2020-03-24 02:09:45 -0700</creation_ts>
          <short_desc>REGRESSION(r258871): [GTK] test bot exiting early due to too many crashes</short_desc>
          <delta_ts>2020-03-24 10:54:16 -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>New Bugs</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="Diego Pino">dpino</reporter>
          <assigned_to name="Diego Pino">dpino</assigned_to>
          <cc>darin</cc>
    
    <cc>webkit-bug-importer</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>1633257</commentid>
    <comment_count>0</comment_count>
    <who name="Diego Pino">dpino</who>
    <bug_when>2020-03-24 02:09:45 -0700</bug_when>
    <thetext>GTK test bot is exiting early due to too many crashes after r258871. See (https://build.webkit.org/builders/GTK%20Linux%2064-bit%20Release%20(Tests)?numbuilds=50):

Build #13070 (r258870-r258875): https://build.webkit.org/builders/GTK%20Linux%2064-bit%20Release%20%28Tests%29/builds/13070 Too many crashes.
Build #13069 (r258866-r258869): https://build.webkit.org/builders/GTK%20Linux%2064-bit%20Release%20%28Tests%29/builds/13069 OK

I bisected r258870-r258875, and already in r258871 the following tests are crashing:

Regressions: Unexpected crashes (3)
  accessibility/aria-invalid.html [ Crash ]
  accessibility/aria-labelledby-on-input.html [ Crash ]
  accessibility/aria-labelledby-overrides-label.html [ Crash ]

r258870 corresponds to https://trac.webkit.org/changeset/258870/webkit.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1633299</commentid>
    <comment_count>1</comment_count>
    <who name="Diego Pino">dpino</who>
    <bug_when>2020-03-24 08:07:49 -0700</bug_when>
    <thetext>Dump:

https://build.webkit.org/results/GTK%20Linux%2064-bit%20Release%20(Tests)/r258908%20(13076)/accessibility/aria-labelledby-on-input-crash-log.txt</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1633300</commentid>
    <comment_count>2</comment_count>
    <who name="Diego Pino">dpino</who>
    <bug_when>2020-03-24 08:08:24 -0700</bug_when>
    <thetext>Thread 1 (Thread 0x7f55ff57b9c0 (LWP 78041)):
#0  0x00007f560ac657ac in _ZN7WebCore11SimpleRangeC2ERKNS_5RangeE () from /home/slave/webkitgtk/gtk-linux-64-release/build/WebKitBuild/Release/lib/libwebkit2gtk-4.0.so.37
#1  0x00007f560a1bc868 in _ZNK6WebKit7WebPage19platformEditorStateERN7WebCore5FrameERNS_11EditorStateENS0_25IncludePostLayoutDataHintE () from /home/slave/webkitgtk/gtk-linux-64-release/build/WebKitBuild/Release/lib/libwebkit2gtk-4.0.so.37
#2  0x00007f560a196f11 in _ZNK6WebKit7WebPage11editorStateENS0_25IncludePostLayoutDataHintE () from /home/slave/webkitgtk/gtk-linux-64-release/build/WebKitBuild/Release/lib/libwebkit2gtk-4.0.so.37
#3  0x00007f560a19ab60 in _ZN6WebKit7WebPage21sendEditorStateUpdateEv () from /home/slave/webkitgtk/gtk-linux-64-release/build/WebKitBuild/Release/lib/libwebkit2gtk-4.0.so.37
#4  0x00007f560a1bd306 in _ZN6WebKit30DrawingAreaCoordinatedGraphics7displayERNS_10UpdateInfoE () from /home/slave/webkitgtk/gtk-linux-64-release/build/WebKitBuild/Release/lib/libwebkit2gtk-4.0.so.37
#5  0x00007f560a1bdde7 in _ZN6WebKit30DrawingAreaCoordinatedGraphics7displayEv () from /home/slave/webkitgtk/gtk-linux-64-release/build/WebKitBuild/Release/lib/libwebkit2gtk-4.0.so.37
#6  0x00007f560a1bdff1 in _ZN6WebKit30DrawingAreaCoordinatedGraphics12forceRepaintEv () from /home/slave/webkitgtk/gtk-linux-64-release/build/WebKitBuild/Release/lib/libwebkit2gtk-4.0.so.37
#7  0x00007f55a2aa6c52 in _ZN3WTR18InjectedBundlePage4dumpEv () from /home/slave/webkitgtk/gtk-linux-64-release-tests/build/WebKitBuild/Release/lib/libTestRunnerInjectedBundle.so
#8  0x00007f560a04221f in _ZN6WebKit30InjectedBundlePageLoaderClient21didFinishLoadForFrameERNS_7WebPageERNS_8WebFrameERN3WTF6RefPtrIN3API6ObjectENS5_13DumbPtrTraitsIS8_EEEE () from /home/slave/webkitgtk/gtk-linux-64-release/build/WebKitBuild/Release/lib/libwebkit2gtk-4.0.so.37
#9  0x00007f560a171f3a in _ZN6WebKit20WebFrameLoaderClient21dispatchDidFinishLoadEv () from /home/slave/webkitgtk/gtk-linux-64-release/build/WebKitBuild/Release/lib/libwebkit2gtk-4.0.so.37
#10 0x00007f560b001936 in _ZN7WebCore11FrameLoader29checkLoadCompleteForThisFrameEv () from /home/slave/webkitgtk/gtk-linux-64-release/build/WebKitBuild/Release/lib/libwebkit2gtk-4.0.so.37
#11 0x00007f560b001a53 in _ZN7WebCore11FrameLoader17checkLoadCompleteEv () from /home/slave/webkitgtk/gtk-linux-64-release/build/WebKitBuild/Release/lib/libwebkit2gtk-4.0.so.37
#12 0x00007f560afda990 in _ZN7WebCore14DocumentLoader15finishedLoadingEv () from /home/slave/webkitgtk/gtk-linux-64-release/build/WebKitBuild/Release/lib/libwebkit2gtk-4.0.so.37
#13 0x00007f560b055974 in _ZN7WebCore14CachedResource11checkNotifyEv () from /home/slave/webkitgtk/gtk-linux-64-release/build/WebKitBuild/Release/lib/libwebkit2gtk-4.0.so.37
#14 0x00007f560b05c6dc in _ZN7WebCore17CachedRawResource13finishLoadingEPNS_12SharedBufferE () from /home/slave/webkitgtk/gtk-linux-64-release/build/WebKitBuild/Release/lib/libwebkit2gtk-4.0.so.37
#15 0x00007f560b03c333 in _ZN7WebCore17SubresourceLoader16didFinishLoadingERKNS_18NetworkLoadMetricsE () from /home/slave/webkitgtk/gtk-linux-64-release/build/WebKitBuild/Release/lib/libwebkit2gtk-4.0.so.37
#16 0x00007f560a117442 in _ZN6WebKit17WebResourceLoader21didFinishResourceLoadERKN7WebCore18NetworkLoadMetricsE () from /home/slave/webkitgtk/gtk-linux-64-release/build/WebKitBuild/Release/lib/libwebkit2gtk-4.0.so.37
#17 0x00007f5609cb629e in _ZN3IPC13handleMessageIN8Messages17WebResourceLoader21DidFinishResourceLoadEN6WebKit17WebResourceLoaderEMS5_FvRKN7WebCore18NetworkLoadMetricsEEEEvRNS_7DecoderEPT0_T1_ () from /home/slave/webkitgtk/gtk-linux-64-release/build/WebKitBuild/Release/lib/libwebkit2gtk-4.0.so.37
#18 0x00007f5609cb4c07 in _ZN6WebKit17WebResourceLoader34didReceiveWebResourceLoaderMessageERN3IPC10ConnectionERNS1_7DecoderE () from /home/slave/webkitgtk/gtk-linux-64-release/build/WebKitBuild/Release/lib/libwebkit2gtk-4.0.so.37
#19 0x00007f560a103b3f in _ZN6WebKit24NetworkProcessConnection17didReceiveMessageERN3IPC10ConnectionERNS1_7DecoderE () from /home/slave/webkitgtk/gtk-linux-64-release/build/WebKitBuild/Release/lib/libwebkit2gtk-4.0.so.37
#20 0x00007f5609dc9418 in _ZN3IPC10Connection15dispatchMessageERNS_7DecoderE () from /home/slave/webkitgtk/gtk-linux-64-release/build/WebKitBuild/Release/lib/libwebkit2gtk-4.0.so.37
#21 0x00007f5609dca735 in _ZN3IPC10Connection15dispatchMessageESt10unique_ptrINS_7DecoderESt14default_deleteIS2_EE () from /home/slave/webkitgtk/gtk-linux-64-release/build/WebKitBuild/Release/lib/libwebkit2gtk-4.0.so.37
#22 0x00007f5609dcae4b in _ZN3IPC10Connection26dispatchOneIncomingMessageEv () from /home/slave/webkitgtk/gtk-linux-64-release/build/WebKitBuild/Release/lib/libwebkit2gtk-4.0.so.37
#23 0x00007f560794178d in _ZN3WTF7RunLoop11performWorkEv () from /home/slave/webkitgtk/gtk-linux-64-release/build/WebKitBuild/Release/lib/libjavascriptcoregtk-4.0.so.18
#24 0x00007f560799f369 in _ZZN3WTF7RunLoopC4EvENUlPvE_4_FUNES1_ () from /home/slave/webkitgtk/gtk-linux-64-release/build/WebKitBuild/Release/lib/libjavascriptcoregtk-4.0.so.18
#25 0x00007f560272b6b8 in g_main_dispatch () at ../../Source/glib-2.58.1/glib/gmain.c:3182
#26 g_main_context_dispatch () at ../../Source/glib-2.58.1/glib/gmain.c:3847
#27 0x00007f560272ba78 in g_main_context_iterate () at ../../Source/glib-2.58.1/glib/gmain.c:3920
#28 0x00007f560272bd62 in g_main_loop_run () at ../../Source/glib-2.58.1/glib/gmain.c:4116
#29 0x00007f560799fe80 in _ZN3WTF7RunLoop3runEv () from /home/slave/webkitgtk/gtk-linux-64-release/build/WebKitBuild/Release/lib/libjavascriptcoregtk-4.0.so.18
#30 0x00007f560a1ca14d in _ZN6WebKit20AuxiliaryProcessMainINS_10WebProcessENS_17WebProcessMainGtkEEEiiPPc () from /home/slave/webkitgtk/gtk-linux-64-release/build/WebKitBuild/Release/lib/libwebkit2gtk-4.0.so.37
#31 0x00007f56010e409b in __libc_start_main (main=0x5559a964dc00 &lt;main&gt;, argc=4, argv=0x7ffcdf62bcd8, init=&lt;optimized out&gt;, fini=&lt;optimized out&gt;, rtld_fini=&lt;optimized out&gt;, stack_end=0x7ffcdf62bcc8) at ../csu/libc-start.c:308
#32 0x00005559a964dc7a in _start ()</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1633303</commentid>
    <comment_count>3</comment_count>
    <who name="Diego Pino">dpino</who>
    <bug_when>2020-03-24 08:10:46 -0700</bug_when>
    <thetext>Sorry for the noise, here is backtrace unmangled:

Thread 1 (Thread 0x7f55ff57b9c0 (LWP 78041)):
#0  0x00007f560ac657ac in WebCore::SimpleRange::SimpleRange(WebCore::Range const&amp;) () from /home/slave/webkitgtk/gtk-linux-64-release/build/WebKitBuild/Release/lib/libwebkit2gtk-4.0.so.37
#1  0x00007f560a1bc868 in WebKit::WebPage::platformEditorState(WebCore::Frame&amp;, WebKit::EditorState&amp;, WebKit::WebPage::IncludePostLayoutDataHint) const () from /home/slave/webkitgtk/gtk-linux-64-release/build/WebKitBuild/Release/lib/libwebkit2gtk-4.0.so.37
#2  0x00007f560a196f11 in WebKit::WebPage::editorState(WebKit::WebPage::IncludePostLayoutDataHint) const () from /home/slave/webkitgtk/gtk-linux-64-release/build/WebKitBuild/Release/lib/libwebkit2gtk-4.0.so.37
#3  0x00007f560a19ab60 in WebKit::WebPage::sendEditorStateUpdate() () from /home/slave/webkitgtk/gtk-linux-64-release/build/WebKitBuild/Release/lib/libwebkit2gtk-4.0.so.37
#4  0x00007f560a1bd306 in WebKit::DrawingAreaCoordinatedGraphics::display(WebKit::UpdateInfo&amp;) () from /home/slave/webkitgtk/gtk-linux-64-release/build/WebKitBuild/Release/lib/libwebkit2gtk-4.0.so.37
#5  0x00007f560a1bdde7 in WebKit::DrawingAreaCoordinatedGraphics::display() () from /home/slave/webkitgtk/gtk-linux-64-release/build/WebKitBuild/Release/lib/libwebkit2gtk-4.0.so.37
#6  0x00007f560a1bdff1 in WebKit::DrawingAreaCoordinatedGraphics::forceRepaint() () from /home/slave/webkitgtk/gtk-linux-64-release/build/WebKitBuild/Release/lib/libwebkit2gtk-4.0.so.37
#7  0x00007f55a2aa6c52 in WTR::InjectedBundlePage::dump() () from /home/slave/webkitgtk/gtk-linux-64-release-tests/build/WebKitBuild/Release/lib/libTestRunnerInjectedBundle.so
#8  0x00007f560a04221f in WebKit::InjectedBundlePageLoaderClient::didFinishLoadForFrame(WebKit::WebPage&amp;, WebKit::WebFrame&amp;, WTF::RefPtr&lt;API::Object, WTF::DumbPtrTraits&lt;API::Object&gt; &gt;&amp;) () from /home/slave/webkitgtk/gtk-linux-64-release/build/WebKitBuild/Release/lib/libwebkit2gtk-4.0.so.37
#9  0x00007f560a171f3a in WebKit::WebFrameLoaderClient::dispatchDidFinishLoad() () from /home/slave/webkitgtk/gtk-linux-64-release/build/WebKitBuild/Release/lib/libwebkit2gtk-4.0.so.37
#10 0x00007f560b001936 in WebCore::FrameLoader::checkLoadCompleteForThisFrame() () from /home/slave/webkitgtk/gtk-linux-64-release/build/WebKitBuild/Release/lib/libwebkit2gtk-4.0.so.37
#11 0x00007f560b001a53 in WebCore::FrameLoader::checkLoadComplete() () from /home/slave/webkitgtk/gtk-linux-64-release/build/WebKitBuild/Release/lib/libwebkit2gtk-4.0.so.37
#12 0x00007f560afda990 in WebCore::DocumentLoader::finishedLoading() () from /home/slave/webkitgtk/gtk-linux-64-release/build/WebKitBuild/Release/lib/libwebkit2gtk-4.0.so.37
#13 0x00007f560b055974 in WebCore::CachedResource::checkNotify() () from /home/slave/webkitgtk/gtk-linux-64-release/build/WebKitBuild/Release/lib/libwebkit2gtk-4.0.so.37
#14 0x00007f560b05c6dc in WebCore::CachedRawResource::finishLoading(WebCore::SharedBuffer*) () from /home/slave/webkitgtk/gtk-linux-64-release/build/WebKitBuild/Release/lib/libwebkit2gtk-4.0.so.37
#15 0x00007f560b03c333 in WebCore::SubresourceLoader::didFinishLoading(WebCore::NetworkLoadMetrics const&amp;) () from /home/slave/webkitgtk/gtk-linux-64-release/build/WebKitBuild/Release/lib/libwebkit2gtk-4.0.so.37
#16 0x00007f560a117442 in WebKit::WebResourceLoader::didFinishResourceLoad(WebCore::NetworkLoadMetrics const&amp;) () from /home/slave/webkitgtk/gtk-linux-64-release/build/WebKitBuild/Release/lib/libwebkit2gtk-4.0.so.37
#17 0x00007f5609cb629e in void IPC::handleMessage&lt;Messages::WebResourceLoader::DidFinishResourceLoad, WebKit::WebResourceLoader, void (WebKit::WebResourceLoader::*)(WebCore::NetworkLoadMetrics const&amp;)&gt;(IPC::Decoder&amp;, WebKit::WebResourceLoader*, void (WebKit::WebResourceLoader::*)(WebCore::NetworkLoadMetrics const&amp;)) () from /home/slave/webkitgtk/gtk-linux-64-release/build/WebKitBuild/Release/lib/libwebkit2gtk-4.0.so.37
#18 0x00007f5609cb4c07 in WebKit::WebResourceLoader::didReceiveWebResourceLoaderMessage(IPC::Connection&amp;, IPC::Decoder&amp;) () from /home/slave/webkitgtk/gtk-linux-64-release/build/WebKitBuild/Release/lib/libwebkit2gtk-4.0.so.37
#19 0x00007f560a103b3f in WebKit::NetworkProcessConnection::didReceiveMessage(IPC::Connection&amp;, IPC::Decoder&amp;) () from /home/slave/webkitgtk/gtk-linux-64-release/build/WebKitBuild/Release/lib/libwebkit2gtk-4.0.so.37
#20 0x00007f5609dc9418 in IPC::Connection::dispatchMessage(IPC::Decoder&amp;) () from /home/slave/webkitgtk/gtk-linux-64-release/build/WebKitBuild/Release/lib/libwebkit2gtk-4.0.so.37
#21 0x00007f5609dca735 in IPC::Connection::dispatchMessage(std::unique_ptr&lt;IPC::Decoder, std::default_delete&lt;IPC::Decoder&gt; &gt;) () from /home/slave/webkitgtk/gtk-linux-64-release/build/WebKitBuild/Release/lib/libwebkit2gtk-4.0.so.37
#22 0x00007f5609dcae4b in IPC::Connection::dispatchOneIncomingMessage() () from /home/slave/webkitgtk/gtk-linux-64-release/build/WebKitBuild/Release/lib/libwebkit2gtk-4.0.so.37
#23 0x00007f560794178d in WTF::RunLoop::performWork() () from /home/slave/webkitgtk/gtk-linux-64-release/build/WebKitBuild/Release/lib/libjavascriptcoregtk-4.0.so.18
#24 0x00007f560799f369 in WTF::RunLoop::RunLoop()::{lambda(void*)#1}::_FUN(void*) () from /home/slave/webkitgtk/gtk-linux-64-release/build/WebKitBuild/Release/lib/libjavascriptcoregtk-4.0.so.18
#25 0x00007f560272b6b8 in g_main_dispatch () at ../../Source/glib-2.58.1/glib/gmain.c:3182
#26 g_main_context_dispatch () at ../../Source/glib-2.58.1/glib/gmain.c:3847
#27 0x00007f560272ba78 in g_main_context_iterate () at ../../Source/glib-2.58.1/glib/gmain.c:3920
#28 0x00007f560272bd62 in g_main_loop_run () at ../../Source/glib-2.58.1/glib/gmain.c:4116
#29 0x00007f560799fe80 in WTF::RunLoop::run() () from /home/slave/webkitgtk/gtk-linux-64-release/build/WebKitBuild/Release/lib/libjavascriptcoregtk-4.0.so.18
#30 0x00007f560a1ca14d in int WebKit::AuxiliaryProcessMain&lt;WebKit::WebProcess, WebKit::WebProcessMainGtk&gt;(int, char**) () from /home/slave/webkitgtk/gtk-linux-64-release/build/WebKitBuild/Release/lib/libwebkit2gtk-4.0.so.37
#31 0x00007f56010e409b in __libc_start_main (main=0x5559a964dc00 &lt;main&gt;, argc=4, argv=0x7ffcdf62bcd8, init=&lt;optimized out&gt;, fini=&lt;optimized out&gt;, rtld_fini=&lt;optimized out&gt;, stack_end=0x7ffcdf62bcc8) at ../csu/libc-start.c:308
#32 0x00005559a964dc7a in _start ()</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1633325</commentid>
    <comment_count>4</comment_count>
    <who name="Darin Adler">darin</who>
    <bug_when>2020-03-24 09:13:10 -0700</bug_when>
    <thetext>Should be trivial to fix. Presumably the it’s dereferencing a null pointer. I think this code needs checks for null:

    postLayoutData.surroundingContextCursorPosition = characterCount(*makeRange(surroundingStart, selectionStart));
    postLayoutData.surroundingContextSelectionPosition = characterCount(*makeRange(surroundingStart, selection.visibleEnd()));

If surroundingStart, selectionStart, or selection.visibleEnd is null, then we can’t call characterCount. Maybe have it be zero instead. Simplest change is probably this:

    if (surroundingStart.isNull() || selectionStart.isNull())
        postLayoutData.surroundingContextCursorPosition = 0;
    else
        postLayoutData.surroundingContextCursorPosition = characterCount(*makeRange(surroundingStart, selectionStart));

Then something similar for surroundingContextSelectionPosition.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1633326</commentid>
    <comment_count>5</comment_count>
    <who name="Darin Adler">darin</who>
    <bug_when>2020-03-24 09:14:46 -0700</bug_when>
    <thetext>I may not have time to quickly fix this myself, so someone could give this a try if they have a GTK system to test on, or could roll out my patch (sad if necessary; lack of a GTK test bot in EWS, I guess).

The issues are in code in WebPageGLib.cpp.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1633328</commentid>
    <comment_count>6</comment_count>
    <who name="Diego Pino">dpino</who>
    <bug_when>2020-03-24 09:16:22 -0700</bug_when>
    <thetext>Thanks, I can give it a try.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1633336</commentid>
    <comment_count>7</comment_count>
      <attachid>394371</attachid>
    <who name="Diego Pino">dpino</who>
    <bug_when>2020-03-24 09:27:53 -0700</bug_when>
    <thetext>Created attachment 394371
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1633339</commentid>
    <comment_count>8</comment_count>
      <attachid>394372</attachid>
    <who name="Diego Pino">dpino</who>
    <bug_when>2020-03-24 09:29:46 -0700</bug_when>
    <thetext>Created attachment 394372
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1633340</commentid>
    <comment_count>9</comment_count>
      <attachid>394371</attachid>
    <who name="Darin Adler">darin</who>
    <bug_when>2020-03-24 09:30:06 -0700</bug_when>
    <thetext>Comment on attachment 394371
Patch

Looks good as a speculative fix. Even better if it’s proven to work on GTK!</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1633341</commentid>
    <comment_count>10</comment_count>
    <who name="Darin Adler">darin</who>
    <bug_when>2020-03-24 09:30:53 -0700</bug_when>
    <thetext>If this works, would be nice to return later and make the code more elegant, but for now this seems great.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1633346</commentid>
    <comment_count>11</comment_count>
    <who name="Diego Pino">dpino</who>
    <bug_when>2020-03-24 09:33:35 -0700</bug_when>
    <thetext>Yep, it works on GTK. I also run those 3 tests that were previously crashing and they&apos;re all passing now.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1633383</commentid>
    <comment_count>12</comment_count>
    <who name="Diego Pino">dpino</who>
    <bug_when>2020-03-24 10:53:45 -0700</bug_when>
    <thetext>Committed r258918: &lt;https://trac.webkit.org/changeset/258918&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1633384</commentid>
    <comment_count>13</comment_count>
    <who name="Radar WebKit Bug Importer">webkit-bug-importer</who>
    <bug_when>2020-03-24 10:54:16 -0700</bug_when>
    <thetext>&lt;rdar://problem/60831727&gt;</thetext>
  </long_desc>
      
          <attachment
              isobsolete="1"
              ispatch="1"
              isprivate="0"
          >
            <attachid>394371</attachid>
            <date>2020-03-24 09:27:53 -0700</date>
            <delta_ts>2020-03-24 09:30:06 -0700</delta_ts>
            <desc>Patch</desc>
            <filename>bug-209467-20200324162752.patch</filename>
            <type>text/plain</type>
            <size>2358</size>
            <attacher name="Diego Pino">dpino</attacher>
            
              <data encoding="base64">U3VidmVyc2lvbiBSZXZpc2lvbjogMjU4OTA4CmRpZmYgLS1naXQgYS9Tb3VyY2UvV2ViS2l0L0No
YW5nZUxvZyBiL1NvdXJjZS9XZWJLaXQvQ2hhbmdlTG9nCmluZGV4IGM2YmRhMzEyYTQ3NmVhYzc1
OGQwYTQ0MjNlNGI5OGExMDNhNzZkYTIuLjgzNzA1ZDJiMWQzOWNkMTM4YzkyOTQxNGE5ZmIyMWM0
NTM3MDIyMmYgMTAwNjQ0Ci0tLSBhL1NvdXJjZS9XZWJLaXQvQ2hhbmdlTG9nCisrKyBiL1NvdXJj
ZS9XZWJLaXQvQ2hhbmdlTG9nCkBAIC0xLDMgKzEsMTQgQEAKKzIwMjAtMDMtMjQgIERpZWdvIFBp
bm8gR2FyY2lhICA8ZHBpbm9AaWdhbGlhLmNvbT4KKworICAgICAgICBSRUdSRVNTSU9OKHIyNTg4
NzEpOiBbR1RLXSB0ZXN0IGJvdCBleGl0aW5nIGVhcmx5IGR1ZSB0byB0b28gbWFueSBjcmFzaGVz
CisgICAgICAgIGh0dHBzOi8vYnVncy53ZWJraXQub3JnL3Nob3dfYnVnLmNnaT9pZD0yMDk0NjcK
KworICAgICAgICBSZXZpZXdlZCBieSBOT0JPRFkgKE9PUFMhKS4KKworICAgICAgICAqIFdlYlBy
b2Nlc3MvV2ViUGFnZS9nbGliL1dlYlBhZ2VHTGliLmNwcDoKKyAgICAgICAgKFdlYktpdDo6V2Vi
UGFnZTo6cGxhdGZvcm1FZGl0b3JTdGF0ZSBjb25zdCk6IEFkZCBhIGNoZWNrIHRvIHByZXZlbnQg
c3RhcnQKKwkJYW5kIGVuZCBwb3NpdGlvbiBvZiBzZWxlY3Rpb24gcmFuZ2UgaXMgbm90IG51bGwu
CisKIDIwMjAtMDMtMjMgIERhdmlkIEtpbHplciAgPGRka2lsemVyQGFwcGxlLmNvbT4KIAogICAg
ICAgICBJUEM6OkRlY29kZXI6OmRlY29kZUZpeGVkTGVuZ3RoRGF0YSgpIHNob3VsZCBiZSBtYXJr
ZWQgV0FSTl9VTlVTRURfUkVUVVJOCmRpZmYgLS1naXQgYS9Tb3VyY2UvV2ViS2l0L1dlYlByb2Nl
c3MvV2ViUGFnZS9nbGliL1dlYlBhZ2VHTGliLmNwcCBiL1NvdXJjZS9XZWJLaXQvV2ViUHJvY2Vz
cy9XZWJQYWdlL2dsaWIvV2ViUGFnZUdMaWIuY3BwCmluZGV4IGUyYWMyM2FlNTU3ZDdhMTdlOTY3
YzAzYmI4MzAxYTc4N2Q3Y2Q0ZDYuLmQ1NDViOThmYmNlMzZkNWZhZjU5MTYwMjRhMzJkYWZhZTVm
ZTg1MjYgMTAwNjQ0Ci0tLSBhL1NvdXJjZS9XZWJLaXQvV2ViUHJvY2Vzcy9XZWJQYWdlL2dsaWIv
V2ViUGFnZUdMaWIuY3BwCisrKyBiL1NvdXJjZS9XZWJLaXQvV2ViUHJvY2Vzcy9XZWJQYWdlL2ds
aWIvV2ViUGFnZUdMaWIuY3BwCkBAIC0xMjAsOCArMTIwLDE0IEBAIHZvaWQgV2ViUGFnZTo6cGxh
dGZvcm1FZGl0b3JTdGF0ZShGcmFtZSYgZnJhbWUsIEVkaXRvclN0YXRlJiByZXN1bHQsIEluY2x1
ZGVQb3N0CiAgICAgICAgICAgICBwb3N0TGF5b3V0RGF0YS5zdXJyb3VuZGluZ0NvbnRleHRTZWxl
Y3Rpb25Qb3NpdGlvbiA9IHBvc3RMYXlvdXREYXRhLnN1cnJvdW5kaW5nQ29udGV4dEN1cnNvclBv
c2l0aW9uOwogICAgICAgICB9IGVsc2UgewogICAgICAgICAgICAgcG9zdExheW91dERhdGEuc3Vy
cm91bmRpbmdDb250ZXh0ID0gcGxhaW5UZXh0KHN1cnJvdW5kaW5nUmFuZ2UuZ2V0KCkpOwotICAg
ICAgICAgICAgcG9zdExheW91dERhdGEuc3Vycm91bmRpbmdDb250ZXh0Q3Vyc29yUG9zaXRpb24g
PSBjaGFyYWN0ZXJDb3VudCgqbWFrZVJhbmdlKHN1cnJvdW5kaW5nU3RhcnQsIHNlbGVjdGlvblN0
YXJ0KSk7Ci0gICAgICAgICAgICBwb3N0TGF5b3V0RGF0YS5zdXJyb3VuZGluZ0NvbnRleHRTZWxl
Y3Rpb25Qb3NpdGlvbiA9IGNoYXJhY3RlckNvdW50KCptYWtlUmFuZ2Uoc3Vycm91bmRpbmdTdGFy
dCwgc2VsZWN0aW9uLnZpc2libGVFbmQoKSkpOworICAgICAgICAgICAgaWYgKHN1cnJvdW5kaW5n
U3RhcnQuaXNOdWxsKCkgfHwgc2VsZWN0aW9uU3RhcnQuaXNOdWxsKCkpCisgICAgICAgICAgICAg
ICAgcG9zdExheW91dERhdGEuc3Vycm91bmRpbmdDb250ZXh0Q3Vyc29yUG9zaXRpb24gPSAwOwor
ICAgICAgICAgICAgZWxzZQorICAgICAgICAgICAgICAgIHBvc3RMYXlvdXREYXRhLnN1cnJvdW5k
aW5nQ29udGV4dEN1cnNvclBvc2l0aW9uID0gY2hhcmFjdGVyQ291bnQoKm1ha2VSYW5nZShzdXJy
b3VuZGluZ1N0YXJ0LCBzZWxlY3Rpb25TdGFydCkpOworICAgICAgICAgICAgaWYgKHN1cnJvdW5k
aW5nU3RhcnQuaXNOdWxsKCkgfHwgc2VsZWN0aW9uLnZpc2libGVFbmQoKS5pc051bGwoKSkKKyAg
ICAgICAgICAgICAgICBwb3N0TGF5b3V0RGF0YS5zdXJyb3VuZGluZ0NvbnRleHRTZWxlY3Rpb25Q
b3NpdGlvbiA9IDA7CisgICAgICAgICAgICBlbHNlCisgICAgICAgICAgICAgICAgcG9zdExheW91
dERhdGEuc3Vycm91bmRpbmdDb250ZXh0U2VsZWN0aW9uUG9zaXRpb24gPSBjaGFyYWN0ZXJDb3Vu
dCgqbWFrZVJhbmdlKHN1cnJvdW5kaW5nU3RhcnQsIHNlbGVjdGlvbi52aXNpYmxlRW5kKCkpKTsK
ICAgICAgICAgfQogICAgIH0KIH0K
</data>
<flag name="review"
          id="409794"
          type_id="1"
          status="+"
          setter="darin"
    />
          </attachment>
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>394372</attachid>
            <date>2020-03-24 09:29:46 -0700</date>
            <delta_ts>2020-03-24 09:30:19 -0700</delta_ts>
            <desc>Patch</desc>
            <filename>bug-209467-20200324162945.patch</filename>
            <type>text/plain</type>
            <size>2364</size>
            <attacher name="Diego Pino">dpino</attacher>
            
              <data encoding="base64">U3VidmVyc2lvbiBSZXZpc2lvbjogMjU4OTA4CmRpZmYgLS1naXQgYS9Tb3VyY2UvV2ViS2l0L0No
YW5nZUxvZyBiL1NvdXJjZS9XZWJLaXQvQ2hhbmdlTG9nCmluZGV4IGM2YmRhMzEyYTQ3NmVhYzc1
OGQwYTQ0MjNlNGI5OGExMDNhNzZkYTIuLjlkYjkwMDVlODg1MTdkOTBmZjI2YmQwM2QwMTgwOTAz
YmNjNzY1ZjkgMTAwNjQ0Ci0tLSBhL1NvdXJjZS9XZWJLaXQvQ2hhbmdlTG9nCisrKyBiL1NvdXJj
ZS9XZWJLaXQvQ2hhbmdlTG9nCkBAIC0xLDMgKzEsMTQgQEAKKzIwMjAtMDMtMjQgIERpZWdvIFBp
bm8gR2FyY2lhICA8ZHBpbm9AaWdhbGlhLmNvbT4KKworICAgICAgICBSRUdSRVNTSU9OKHIyNTg4
NzEpOiBbR1RLXSB0ZXN0IGJvdCBleGl0aW5nIGVhcmx5IGR1ZSB0byB0b28gbWFueSBjcmFzaGVz
CisgICAgICAgIGh0dHBzOi8vYnVncy53ZWJraXQub3JnL3Nob3dfYnVnLmNnaT9pZD0yMDk0NjcK
KworICAgICAgICBSZXZpZXdlZCBieSBOT0JPRFkgKE9PUFMhKS4KKworICAgICAgICAqIFdlYlBy
b2Nlc3MvV2ViUGFnZS9nbGliL1dlYlBhZ2VHTGliLmNwcDoKKyAgICAgICAgKFdlYktpdDo6V2Vi
UGFnZTo6cGxhdGZvcm1FZGl0b3JTdGF0ZSBjb25zdCk6IEFkZCBhIGNoZWNrIHRvIHByZXZlbnQg
c3RhcnQKKyAgICAgICAgYW5kIGVuZCBwb3NpdGlvbiBvZiBzZWxlY3Rpb24gcmFuZ2UgaXMgbm90
IG51bGwuCisKIDIwMjAtMDMtMjMgIERhdmlkIEtpbHplciAgPGRka2lsemVyQGFwcGxlLmNvbT4K
IAogICAgICAgICBJUEM6OkRlY29kZXI6OmRlY29kZUZpeGVkTGVuZ3RoRGF0YSgpIHNob3VsZCBi
ZSBtYXJrZWQgV0FSTl9VTlVTRURfUkVUVVJOCmRpZmYgLS1naXQgYS9Tb3VyY2UvV2ViS2l0L1dl
YlByb2Nlc3MvV2ViUGFnZS9nbGliL1dlYlBhZ2VHTGliLmNwcCBiL1NvdXJjZS9XZWJLaXQvV2Vi
UHJvY2Vzcy9XZWJQYWdlL2dsaWIvV2ViUGFnZUdMaWIuY3BwCmluZGV4IGUyYWMyM2FlNTU3ZDdh
MTdlOTY3YzAzYmI4MzAxYTc4N2Q3Y2Q0ZDYuLmQ1NDViOThmYmNlMzZkNWZhZjU5MTYwMjRhMzJk
YWZhZTVmZTg1MjYgMTAwNjQ0Ci0tLSBhL1NvdXJjZS9XZWJLaXQvV2ViUHJvY2Vzcy9XZWJQYWdl
L2dsaWIvV2ViUGFnZUdMaWIuY3BwCisrKyBiL1NvdXJjZS9XZWJLaXQvV2ViUHJvY2Vzcy9XZWJQ
YWdlL2dsaWIvV2ViUGFnZUdMaWIuY3BwCkBAIC0xMjAsOCArMTIwLDE0IEBAIHZvaWQgV2ViUGFn
ZTo6cGxhdGZvcm1FZGl0b3JTdGF0ZShGcmFtZSYgZnJhbWUsIEVkaXRvclN0YXRlJiByZXN1bHQs
IEluY2x1ZGVQb3N0CiAgICAgICAgICAgICBwb3N0TGF5b3V0RGF0YS5zdXJyb3VuZGluZ0NvbnRl
eHRTZWxlY3Rpb25Qb3NpdGlvbiA9IHBvc3RMYXlvdXREYXRhLnN1cnJvdW5kaW5nQ29udGV4dEN1
cnNvclBvc2l0aW9uOwogICAgICAgICB9IGVsc2UgewogICAgICAgICAgICAgcG9zdExheW91dERh
dGEuc3Vycm91bmRpbmdDb250ZXh0ID0gcGxhaW5UZXh0KHN1cnJvdW5kaW5nUmFuZ2UuZ2V0KCkp
OwotICAgICAgICAgICAgcG9zdExheW91dERhdGEuc3Vycm91bmRpbmdDb250ZXh0Q3Vyc29yUG9z
aXRpb24gPSBjaGFyYWN0ZXJDb3VudCgqbWFrZVJhbmdlKHN1cnJvdW5kaW5nU3RhcnQsIHNlbGVj
dGlvblN0YXJ0KSk7Ci0gICAgICAgICAgICBwb3N0TGF5b3V0RGF0YS5zdXJyb3VuZGluZ0NvbnRl
eHRTZWxlY3Rpb25Qb3NpdGlvbiA9IGNoYXJhY3RlckNvdW50KCptYWtlUmFuZ2Uoc3Vycm91bmRp
bmdTdGFydCwgc2VsZWN0aW9uLnZpc2libGVFbmQoKSkpOworICAgICAgICAgICAgaWYgKHN1cnJv
dW5kaW5nU3RhcnQuaXNOdWxsKCkgfHwgc2VsZWN0aW9uU3RhcnQuaXNOdWxsKCkpCisgICAgICAg
ICAgICAgICAgcG9zdExheW91dERhdGEuc3Vycm91bmRpbmdDb250ZXh0Q3Vyc29yUG9zaXRpb24g
PSAwOworICAgICAgICAgICAgZWxzZQorICAgICAgICAgICAgICAgIHBvc3RMYXlvdXREYXRhLnN1
cnJvdW5kaW5nQ29udGV4dEN1cnNvclBvc2l0aW9uID0gY2hhcmFjdGVyQ291bnQoKm1ha2VSYW5n
ZShzdXJyb3VuZGluZ1N0YXJ0LCBzZWxlY3Rpb25TdGFydCkpOworICAgICAgICAgICAgaWYgKHN1
cnJvdW5kaW5nU3RhcnQuaXNOdWxsKCkgfHwgc2VsZWN0aW9uLnZpc2libGVFbmQoKS5pc051bGwo
KSkKKyAgICAgICAgICAgICAgICBwb3N0TGF5b3V0RGF0YS5zdXJyb3VuZGluZ0NvbnRleHRTZWxl
Y3Rpb25Qb3NpdGlvbiA9IDA7CisgICAgICAgICAgICBlbHNlCisgICAgICAgICAgICAgICAgcG9z
dExheW91dERhdGEuc3Vycm91bmRpbmdDb250ZXh0U2VsZWN0aW9uUG9zaXRpb24gPSBjaGFyYWN0
ZXJDb3VudCgqbWFrZVJhbmdlKHN1cnJvdW5kaW5nU3RhcnQsIHNlbGVjdGlvbi52aXNpYmxlRW5k
KCkpKTsKICAgICAgICAgfQogICAgIH0KIH0K
</data>
<flag name="review"
          id="409793"
          type_id="1"
          status="+"
          setter="darin"
    />
          </attachment>
      

    </bug>

</bugzilla>