<?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>190748</bug_id>
          
          <creation_ts>2018-10-19 01:04:19 -0700</creation_ts>
          <short_desc>[Win] Assertion fails while destructing local static AtomicString of FontCache::lastResortFallbackFont</short_desc>
          <delta_ts>2018-10-23 22:00:49 -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="Fujii Hironori">fujii</reporter>
          <assigned_to name="Fujii Hironori">fujii</assigned_to>
          <cc>achristensen</cc>
    
    <cc>bfulgham</cc>
    
    <cc>don.olmstead</cc>
    
    <cc>ews-watchlist</cc>
    
    <cc>mmaxfield</cc>
    
    <cc>pvollan</cc>
    
    <cc>rniwa</cc>
    
    <cc>webkit-bug-importer</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>1470824</commentid>
    <comment_count>0</comment_count>
    <who name="Fujii Hironori">fujii</who>
    <bug_when>2018-10-19 01:04:19 -0700</bug_when>
    <thetext>[WinCairo] Assertion failure &quot;The string being removed is atomic in the string table of an other thread!&quot;

WinCairo, WK1, Debug build, trunk@237282

1. Start MiniBrowser
2. Go to https://webkit.org/
3. Close Window
4. Assertion fails

Callstack:

&gt; WTF.dll!WTFCrash() Line 255	C++
&gt; WTF.dll!WTF::AtomicStringImpl::remove(WTF::AtomicStringImpl * string) Line 489	C++
&gt; WTF.dll!WTF::StringImpl::~StringImpl() Line 119	C++
&gt; [External Code]	
&gt; WTF.dll!WTF::StringImpl::destroy(WTF::StringImpl * stringImpl) Line 151	C++
&gt; WebKit2.dll!WTF::StringImpl::deref() Line 1058	C++
&gt; WebKit2.dll!WTF::derefIfNotNull&lt;WTF::StringImpl&gt;(WTF::StringImpl * ptr) Line 45	C++
&gt; WebKit2.dll!WTF::RefPtr&lt;WTF::StringImpl,WTF::DumbPtrTraits&lt;WTF::StringImpl&gt; &gt;::~RefPtr&lt;WTF::StringImpl,WTF::DumbPtrTraits&lt;WTF::StringImpl&gt; &gt;() Line 69	C++
&gt; WebKit2.dll!WTF::String::~String() Line 377	C++
&gt; [External Code]	
&gt; WebKit2.dll!WebKit::callExit(IPC::Connection * __formal) Line 163	C++
&gt; WebKit2.dll!IPC::Connection::connectionDidClose() Line 824	C++
&gt; WebKit2.dll!IPC::Connection::readEventHandler() Line 159	C++
&gt; WebKit2.dll!IPC::Connection::invokeReadEventHandler::__l2::&lt;lambda&gt;() Line 238	C++
&gt; WebKit2.dll!WTF::Function&lt;void __cdecl(void)&gt;::CallableWrapper&lt;void &lt;lambda&gt;(void) &gt;::call() Line 101	C++
&gt; WTF.dll!WTF::Function&lt;void __cdecl(void)&gt;::operator()() Line 57	C++
&gt; WTF.dll!WTF::WorkQueue::performWorkOnRegisteredWorkThread() Line 62	C++
&gt; WTF.dll!WTF::WorkQueue::workThreadCallback(void * context) Line 44	C++
&gt; [External Code]	


The following assertion failed.

&gt; ASSERT_WITH_MESSAGE(iterator != atomicStringTable.end(), &quot;The string being removed is atomic in the string table of an other thread!&quot;);

But, atomicStringTable was null.
I guess the real problem is String&apos;s dtor was called in _exit.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1470826</commentid>
    <comment_count>1</comment_count>
    <who name="Fujii Hironori">fujii</who>
    <bug_when>2018-10-19 02:14:39 -0700</bug_when>
    <thetext>(In reply to Fujii Hironori from comment #0)
&gt; WinCairo, WK1, Debug build, trunk@237282

Oops. I tested with WK2.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1470827</commentid>
    <comment_count>2</comment_count>
    <who name="Fujii Hironori">fujii</who>
    <bug_when>2018-10-19 02:16:20 -0700</bug_when>
    <thetext>The string was &quot;Lucida Sans Unicode&quot;.
It is  https://github.com/WebKit/webkit/blob/53d0b766f0692da943a7b01af78940a41581b702/Source/WebCore/platform/graphics/win/FontCacheWin.cpp#L358</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1470828</commentid>
    <comment_count>3</comment_count>
      <attachid>352775</attachid>
    <who name="Fujii Hironori">fujii</who>
    <bug_when>2018-10-19 02:20:22 -0700</bug_when>
    <thetext>Created attachment 352775
WIP patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1470836</commentid>
    <comment_count>4</comment_count>
      <attachid>352780</attachid>
    <who name="Fujii Hironori">fujii</who>
    <bug_when>2018-10-19 04:10:55 -0700</bug_when>
    <thetext>Created attachment 352780
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1471321</commentid>
    <comment_count>5</comment_count>
      <attachid>352780</attachid>
    <who name="Don Olmstead">don.olmstead</who>
    <bug_when>2018-10-22 13:22:09 -0700</bug_when>
    <thetext>Comment on attachment 352780
Patch

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

&gt; Source/WebCore/platform/graphics/win/FontCacheWin.cpp:354
&gt; -    static AtomicString fallbackFonts[] = {
&gt; -        AtomicString(&quot;Times New Roman&quot;, AtomicString::ConstructFromLiteral),
&gt; -        AtomicString(&quot;Microsoft Sans Serif&quot;, AtomicString::ConstructFromLiteral),
&gt; -        AtomicString(&quot;Tahoma&quot;, AtomicString::ConstructFromLiteral),
&gt; -        AtomicString(&quot;Lucida Sans Unicode&quot;, AtomicString::ConstructFromLiteral),
&gt; -        AtomicString(&quot;Arial&quot;, AtomicString::ConstructFromLiteral)
&gt; +    const auto fallbackFontNames = {

Does NeverDestroyed not work in this context?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1471450</commentid>
    <comment_count>6</comment_count>
      <attachid>352780</attachid>
    <who name="Fujii Hironori">fujii</who>
    <bug_when>2018-10-22 17:59:20 -0700</bug_when>
    <thetext>Comment on attachment 352780
Patch

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

Thank you very much for reviewing my patch.

&gt;&gt; Source/WebCore/platform/graphics/win/FontCacheWin.cpp:354
&gt;&gt; +    const auto fallbackFontNames = {
&gt; 
&gt; Does NeverDestroyed not work in this context?

I don&apos;t use NeverDestroyed because this code path is executed just only once. I think NeverDestroyed should be use only for instances which are used all the time.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1471482</commentid>
    <comment_count>7</comment_count>
    <who name="Fujii Hironori">fujii</who>
    <bug_when>2018-10-22 20:51:53 -0700</bug_when>
    <thetext>Review, please.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1471540</commentid>
    <comment_count>8</comment_count>
      <attachid>352780</attachid>
    <who name="Myles C. Maxfield">mmaxfield</who>
    <bug_when>2018-10-23 04:24:59 -0700</bug_when>
    <thetext>Comment on attachment 352780
Patch

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

&gt;&gt;&gt; Source/WebCore/platform/graphics/win/FontCacheWin.cpp:354
&gt;&gt;&gt; +    const auto fallbackFontNames = {
&gt;&gt; 
&gt;&gt; Does NeverDestroyed not work in this context?
&gt; 
&gt; I don&apos;t use NeverDestroyed because this code path is executed just only once. I think NeverDestroyed should be use only for instances which are used all the time.

Using string literals still makes them live forever, so I&apos;m not sure why one is preferable over the other. We should be using NeverDestroyed instead.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1471749</commentid>
    <comment_count>9</comment_count>
      <attachid>353019</attachid>
    <who name="Fujii Hironori">fujii</who>
    <bug_when>2018-10-23 20:17:03 -0700</bug_when>
    <thetext>Created attachment 353019
Patch

You are right. Applied the review feedback.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1471762</commentid>
    <comment_count>10</comment_count>
      <attachid>353019</attachid>
    <who name="Fujii Hironori">fujii</who>
    <bug_when>2018-10-23 21:59:13 -0700</bug_when>
    <thetext>Comment on attachment 353019
Patch

Clearing flags on attachment: 353019

Committed r237375: &lt;https://trac.webkit.org/changeset/237375&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1471763</commentid>
    <comment_count>11</comment_count>
    <who name="Fujii Hironori">fujii</who>
    <bug_when>2018-10-23 21:59:15 -0700</bug_when>
    <thetext>All reviewed patches have been landed.  Closing bug.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1471764</commentid>
    <comment_count>12</comment_count>
    <who name="Radar WebKit Bug Importer">webkit-bug-importer</who>
    <bug_when>2018-10-23 22:00:49 -0700</bug_when>
    <thetext>&lt;rdar://problem/45510349&gt;</thetext>
  </long_desc>
      
          <attachment
              isobsolete="1"
              ispatch="1"
              isprivate="0"
          >
            <attachid>352775</attachid>
            <date>2018-10-19 02:20:22 -0700</date>
            <delta_ts>2018-10-19 04:10:51 -0700</delta_ts>
            <desc>WIP patch</desc>
            <filename>a.diff</filename>
            <type>text/plain</type>
            <size>1760</size>
            <attacher name="Fujii Hironori">fujii</attacher>
            
              <data encoding="base64">ZGlmZiAtLWdpdCBhL1NvdXJjZS9XZWJDb3JlL3BsYXRmb3JtL2dyYXBoaWNzL3dpbi9Gb250Q2Fj
aGVXaW4uY3BwIGIvU291cmNlL1dlYkNvcmUvcGxhdGZvcm0vZ3JhcGhpY3Mvd2luL0ZvbnRDYWNo
ZVdpbi5jcHAKaW5kZXggYmQzNDI1MzE2MzYuLmRlYWFlMDdmMjg1IDEwMDY0NAotLS0gYS9Tb3Vy
Y2UvV2ViQ29yZS9wbGF0Zm9ybS9ncmFwaGljcy93aW4vRm9udENhY2hlV2luLmNwcAorKysgYi9T
b3VyY2UvV2ViQ29yZS9wbGF0Zm9ybS9ncmFwaGljcy93aW4vRm9udENhY2hlV2luLmNwcApAQCAt
MzUxLDE3ICszNTEsMTggQEAgUmVmPEZvbnQ+IEZvbnRDYWNoZTo6bGFzdFJlc29ydEZhbGxiYWNr
Rm9udChjb25zdCBGb250RGVzY3JpcHRpb24mIGZvbnREZXNjcmlwdGkKICAgICAvLyBTZWFyY2gg
YWxsIHR5cGljYWwgV2luZG93cy1pbnN0YWxsZWQgZnVsbCBVbmljb2RlIGZvbnRzLgogICAgIC8v
IFNvcnRlZCBieSBtb3N0IHRvIGxlYXN0IGdseXBocyBhY2NvcmRpbmcgdG8gaHR0cDovL2VuLndp
a2lwZWRpYS5vcmcvd2lraS9Vbmljb2RlX3R5cGVmYWNlcwogICAgIC8vIFN0YXJ0IHdpdGggVGlt
ZXMgTmV3IFJvbWFuIGFsc28gc2luY2UgaXQgaXMgdGhlIGRlZmF1bHQgaWYgdGhlIHVzZXIgZG9l
c24ndCBjaGFuZ2UgcHJlZnMuCi0gICAgc3RhdGljIEF0b21pY1N0cmluZyBmYWxsYmFja0ZvbnRz
W10gPSB7Ci0gICAgICAgIEF0b21pY1N0cmluZygiVGltZXMgTmV3IFJvbWFuIiwgQXRvbWljU3Ry
aW5nOjpDb25zdHJ1Y3RGcm9tTGl0ZXJhbCksCi0gICAgICAgIEF0b21pY1N0cmluZygiTWljcm9z
b2Z0IFNhbnMgU2VyaWYiLCBBdG9taWNTdHJpbmc6OkNvbnN0cnVjdEZyb21MaXRlcmFsKSwKLSAg
ICAgICAgQXRvbWljU3RyaW5nKCJUYWhvbWEiLCBBdG9taWNTdHJpbmc6OkNvbnN0cnVjdEZyb21M
aXRlcmFsKSwKLSAgICAgICAgQXRvbWljU3RyaW5nKCJMdWNpZGEgU2FucyBVbmljb2RlIiwgQXRv
bWljU3RyaW5nOjpDb25zdHJ1Y3RGcm9tTGl0ZXJhbCksCi0gICAgICAgIEF0b21pY1N0cmluZygi
QXJpYWwiLCBBdG9taWNTdHJpbmc6OkNvbnN0cnVjdEZyb21MaXRlcmFsKQorICAgIGNvbnN0IGF1
dG8gZmFsbGJhY2tGb250TmFtZXMgPSB7CisgICAgICAgICJUaW1lcyBOZXcgUm9tYW4iLAorICAg
ICAgICAiTWljcm9zb2Z0IFNhbnMgU2VyaWYiLAorICAgICAgICAiVGFob21hIiwKKyAgICAgICAg
Ikx1Y2lkYSBTYW5zIFVuaWNvZGUiLAorICAgICAgICAiQXJpYWwiLAogICAgIH07CiAgICAgUmVm
UHRyPEZvbnQ+IHNpbXBsZUZvbnQ7Ci0gICAgZm9yIChzaXplX3QgaSA9IDA7IGkgPCBXVEZfQVJS
QVlfTEVOR1RIKGZhbGxiYWNrRm9udHMpOyArK2kpIHsKLSAgICAgICAgaWYgKHNpbXBsZUZvbnQg
PSBmb250Rm9yRmFtaWx5KGZvbnREZXNjcmlwdGlvbiwgZmFsbGJhY2tGb250c1tpXSkpIHsKLSAg
ICAgICAgICAgIGZhbGxiYWNrRm9udE5hbWUuZ2V0KCkgPSBmYWxsYmFja0ZvbnRzW2ldOworICAg
IGZvciAoYXV0byYgZm9udE5hbWVMaXRlcmFsIDogZmFsbGJhY2tGb250TmFtZXMpIHsKKyAgICAg
ICAgQXRvbWljU3RyaW5nIGZvbnROYW1lKGZvbnROYW1lTGl0ZXJhbCk7CisgICAgICAgIGlmIChz
aW1wbGVGb250ID0gZm9udEZvckZhbWlseShmb250RGVzY3JpcHRpb24sIGZvbnROYW1lKSkgewor
ICAgICAgICAgICAgZmFsbGJhY2tGb250TmFtZS5nZXQoKSA9IFdURk1vdmUoZm9udE5hbWUpOwog
ICAgICAgICAgICAgcmV0dXJuICpzaW1wbGVGb250OwogICAgICAgICB9CiAgICAgfQo=
</data>

          </attachment>
          <attachment
              isobsolete="1"
              ispatch="1"
              isprivate="0"
          >
            <attachid>352780</attachid>
            <date>2018-10-19 04:10:55 -0700</date>
            <delta_ts>2018-10-23 20:17:00 -0700</delta_ts>
            <desc>Patch</desc>
            <filename>bug-190748-20181019201054.patch</filename>
            <type>text/plain</type>
            <size>3100</size>
            <attacher name="Fujii Hironori">fujii</attacher>
            
              <data encoding="base64">U3VidmVyc2lvbiBSZXZpc2lvbjogMjM3MjgyCmRpZmYgLS1naXQgYS9Tb3VyY2UvV2ViQ29yZS9D
aGFuZ2VMb2cgYi9Tb3VyY2UvV2ViQ29yZS9DaGFuZ2VMb2cKaW5kZXggZTVhMDc2ODkzODQzMGM0
ZTk3N2U4YmRmNzE3Zjk1YTAzZDA0M2EyOS4uODU2NDljZGZlMTFjZGRlYTc5OTFkY2JiZTEzYzkx
OTkyOTkwMmMwOCAxMDA2NDQKLS0tIGEvU291cmNlL1dlYkNvcmUvQ2hhbmdlTG9nCisrKyBiL1Nv
dXJjZS9XZWJDb3JlL0NoYW5nZUxvZwpAQCAtMSwzICsxLDI0IEBACisyMDE4LTEwLTE5ICBGdWpp
aSBIaXJvbm9yaSAgPEhpcm9ub3JpLkZ1amlpQHNvbnkuY29tPgorCisgICAgICAgIFtXaW5DYWly
b10gQXNzZXJ0aW9uIGZhaWxzIHdoaWxlIGRlc3RydWN0aW5nIGxvY2FsIHN0YXRpYyBBdG9taWNT
dHJpbmcgb2YgRm9udENhY2hlOjpsYXN0UmVzb3J0RmFsbGJhY2tGb250CisgICAgICAgIGh0dHBz
Oi8vYnVncy53ZWJraXQub3JnL3Nob3dfYnVnLmNnaT9pZD0xOTA3NDgKKworICAgICAgICBSZXZp
ZXdlZCBieSBOT0JPRFkgKE9PUFMhKS4KKworICAgICAgICBGb250Q2FjaGU6Omxhc3RSZXNvcnRG
YWxsYmFja0ZvbnQgaGFkIGxvY2FsIHN0YXRpYyBBdG9taWNTdHJpbmcKKyAgICAgICAgdmFyaWFi
bGVzIHdoaWNoIGFyZSBub3QgY2Fwc3VsZWQgYnkgTmV2ZXJEZXN0cm95ZWQuIFRoZW4sIHRob3Nl
CisgICAgICAgIHZhcmlhYmxlcyBhcmUgZGVzdHJ1Y3RlZCB3aGVuIFdlYlByb2Nlc3MgZXhpdHMu
CisKKyAgICAgICAgQ2hhbmdlZCB0aG9zZSBBdG9taWNTdHJpbmcgdG8gc2ltcGxlIHN0cmluZyBs
aXRlcmFscy4KKworICAgICAgICBUaGlzIGNoYW5nZSBjYW4ndCBiZSB0ZXN0ZWQgYXV0b21hdGlj
YWxseSBiZWNhdXNlIFdlYktpdFRlc3RSdW5uZXIKKyAgICAgICAgZG9lc24ndCBzdXBwb3J0IFdp
bmRvd3MgcG9ydCB5ZXQuCisKKyAgICAgICAgKiBwbGF0Zm9ybS9ncmFwaGljcy93aW4vRm9udENh
Y2hlV2luLmNwcDoKKyAgICAgICAgKFdlYkNvcmU6OkZvbnRDYWNoZTo6bGFzdFJlc29ydEZhbGxi
YWNrRm9udCk6IFJlbmFtZWQKKyAgICAgICAgJ2ZhbGxiYWNrRm9udHMnIHRvICdmYWxsYmFja0Zv
bnROYW1lcycsIGFuZCBjaGFuZ2VkIHRvIHNpbXBsZQorICAgICAgICBzdHJpbmcgbGl0ZXJhbHMu
CisKIDIwMTgtMTAtMTggIEVyaWMgQ2FybHNvbiAgPGVyaWMuY2FybHNvbkBhcHBsZS5jb20+CiAK
ICAgICAgICAgW01lZGlhU3RyZWFtXSBBbGxvdyBwb3J0cyB0byBvcHRpb25hbGx5IGRvIHNjcmVl
biBjYXB0dXJlIGluIHRoZSBVSSBwcm9jZXNzCmRpZmYgLS1naXQgYS9Tb3VyY2UvV2ViQ29yZS9w
bGF0Zm9ybS9ncmFwaGljcy93aW4vRm9udENhY2hlV2luLmNwcCBiL1NvdXJjZS9XZWJDb3JlL3Bs
YXRmb3JtL2dyYXBoaWNzL3dpbi9Gb250Q2FjaGVXaW4uY3BwCmluZGV4IGJkMzQyNTMxNjM2MTkx
MzJjYTMzYmJmYjQ2ZjJmMjJiNTAzOTVhNjQuLmRlYWFlMDdmMjg1NzU4YjQzYjZlNWJmYTdlYjYw
MzI2M2ZjM2YyMGUgMTAwNjQ0Ci0tLSBhL1NvdXJjZS9XZWJDb3JlL3BsYXRmb3JtL2dyYXBoaWNz
L3dpbi9Gb250Q2FjaGVXaW4uY3BwCisrKyBiL1NvdXJjZS9XZWJDb3JlL3BsYXRmb3JtL2dyYXBo
aWNzL3dpbi9Gb250Q2FjaGVXaW4uY3BwCkBAIC0zNTEsMTcgKzM1MSwxOCBAQCBSZWY8Rm9udD4g
Rm9udENhY2hlOjpsYXN0UmVzb3J0RmFsbGJhY2tGb250KGNvbnN0IEZvbnREZXNjcmlwdGlvbiYg
Zm9udERlc2NyaXB0aQogICAgIC8vIFNlYXJjaCBhbGwgdHlwaWNhbCBXaW5kb3dzLWluc3RhbGxl
ZCBmdWxsIFVuaWNvZGUgZm9udHMuCiAgICAgLy8gU29ydGVkIGJ5IG1vc3QgdG8gbGVhc3QgZ2x5
cGhzIGFjY29yZGluZyB0byBodHRwOi8vZW4ud2lraXBlZGlhLm9yZy93aWtpL1VuaWNvZGVfdHlw
ZWZhY2VzCiAgICAgLy8gU3RhcnQgd2l0aCBUaW1lcyBOZXcgUm9tYW4gYWxzbyBzaW5jZSBpdCBp
cyB0aGUgZGVmYXVsdCBpZiB0aGUgdXNlciBkb2Vzbid0IGNoYW5nZSBwcmVmcy4KLSAgICBzdGF0
aWMgQXRvbWljU3RyaW5nIGZhbGxiYWNrRm9udHNbXSA9IHsKLSAgICAgICAgQXRvbWljU3RyaW5n
KCJUaW1lcyBOZXcgUm9tYW4iLCBBdG9taWNTdHJpbmc6OkNvbnN0cnVjdEZyb21MaXRlcmFsKSwK
LSAgICAgICAgQXRvbWljU3RyaW5nKCJNaWNyb3NvZnQgU2FucyBTZXJpZiIsIEF0b21pY1N0cmlu
Zzo6Q29uc3RydWN0RnJvbUxpdGVyYWwpLAotICAgICAgICBBdG9taWNTdHJpbmcoIlRhaG9tYSIs
IEF0b21pY1N0cmluZzo6Q29uc3RydWN0RnJvbUxpdGVyYWwpLAotICAgICAgICBBdG9taWNTdHJp
bmcoIkx1Y2lkYSBTYW5zIFVuaWNvZGUiLCBBdG9taWNTdHJpbmc6OkNvbnN0cnVjdEZyb21MaXRl
cmFsKSwKLSAgICAgICAgQXRvbWljU3RyaW5nKCJBcmlhbCIsIEF0b21pY1N0cmluZzo6Q29uc3Ry
dWN0RnJvbUxpdGVyYWwpCisgICAgY29uc3QgYXV0byBmYWxsYmFja0ZvbnROYW1lcyA9IHsKKyAg
ICAgICAgIlRpbWVzIE5ldyBSb21hbiIsCisgICAgICAgICJNaWNyb3NvZnQgU2FucyBTZXJpZiIs
CisgICAgICAgICJUYWhvbWEiLAorICAgICAgICAiTHVjaWRhIFNhbnMgVW5pY29kZSIsCisgICAg
ICAgICJBcmlhbCIsCiAgICAgfTsKICAgICBSZWZQdHI8Rm9udD4gc2ltcGxlRm9udDsKLSAgICBm
b3IgKHNpemVfdCBpID0gMDsgaSA8IFdURl9BUlJBWV9MRU5HVEgoZmFsbGJhY2tGb250cyk7ICsr
aSkgewotICAgICAgICBpZiAoc2ltcGxlRm9udCA9IGZvbnRGb3JGYW1pbHkoZm9udERlc2NyaXB0
aW9uLCBmYWxsYmFja0ZvbnRzW2ldKSkgewotICAgICAgICAgICAgZmFsbGJhY2tGb250TmFtZS5n
ZXQoKSA9IGZhbGxiYWNrRm9udHNbaV07CisgICAgZm9yIChhdXRvJiBmb250TmFtZUxpdGVyYWwg
OiBmYWxsYmFja0ZvbnROYW1lcykgeworICAgICAgICBBdG9taWNTdHJpbmcgZm9udE5hbWUoZm9u
dE5hbWVMaXRlcmFsKTsKKyAgICAgICAgaWYgKHNpbXBsZUZvbnQgPSBmb250Rm9yRmFtaWx5KGZv
bnREZXNjcmlwdGlvbiwgZm9udE5hbWUpKSB7CisgICAgICAgICAgICBmYWxsYmFja0ZvbnROYW1l
LmdldCgpID0gV1RGTW92ZShmb250TmFtZSk7CiAgICAgICAgICAgICByZXR1cm4gKnNpbXBsZUZv
bnQ7CiAgICAgICAgIH0KICAgICB9Cg==
</data>

          </attachment>
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>353019</attachid>
            <date>2018-10-23 20:17:03 -0700</date>
            <delta_ts>2018-10-23 21:59:13 -0700</delta_ts>
            <desc>Patch</desc>
            <filename>bug-190748-20181024121702.patch</filename>
            <type>text/plain</type>
            <size>2372</size>
            <attacher name="Fujii Hironori">fujii</attacher>
            
              <data encoding="base64">U3VidmVyc2lvbiBSZXZpc2lvbjogMjM3MzY4CmRpZmYgLS1naXQgYS9Tb3VyY2UvV2ViQ29yZS9D
aGFuZ2VMb2cgYi9Tb3VyY2UvV2ViQ29yZS9DaGFuZ2VMb2cKaW5kZXggOWQ0ZmE2ODE5MmVhOTYx
ZGFmYTE4ZDgzNWI0MWVkMTI2YThlOTU4NS4uNzYxMjg4OTNhMmJiNjM1MGEwN2I2OTBkMjY3Nzhi
ZjRhNjU1MDk5YSAxMDA2NDQKLS0tIGEvU291cmNlL1dlYkNvcmUvQ2hhbmdlTG9nCisrKyBiL1Nv
dXJjZS9XZWJDb3JlL0NoYW5nZUxvZwpAQCAtMSwzICsxLDI1IEBACisyMDE4LTEwLTIzICBGdWpp
aSBIaXJvbm9yaSAgPEhpcm9ub3JpLkZ1amlpQHNvbnkuY29tPgorCisgICAgICAgIFtXaW5dIEFz
c2VydGlvbiBmYWlscyB3aGlsZSBkZXN0cnVjdGluZyBsb2NhbCBzdGF0aWMgQXRvbWljU3RyaW5n
IG9mIEZvbnRDYWNoZTo6bGFzdFJlc29ydEZhbGxiYWNrRm9udAorICAgICAgICBodHRwczovL2J1
Z3Mud2Via2l0Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9MTkwNzQ4CisKKyAgICAgICAgUmV2aWV3ZWQg
YnkgTk9CT0RZIChPT1BTISkuCisKKyAgICAgICAgRm9udENhY2hlOjpsYXN0UmVzb3J0RmFsbGJh
Y2tGb250IGhhZCBsb2NhbCBzdGF0aWMgQXRvbWljU3RyaW5nCisgICAgICAgIHZhcmlhYmxlcyB3
aGljaCBhcmUgbm90IGNhcHN1bGVkIGJ5IE5ldmVyRGVzdHJveWVkLiBIZW5jZSwgdGhvc2UKKyAg
ICAgICAgdmFyaWFibGVzIGFyZSBkZXN0cnVjdGVkIG9uIElQQyB0aHJlYWQgd2hlbiBXZWJQcm9j
ZXNzIGV4aXRzLgorCisgICAgICAgIENoYW5nZWQgdGhvc2UgQXRvbWljU3RyaW5nIHRvIE5ldmVy
RGVzdHJveWVkPEF0b21pY1N0cmluZz4gbm90IHRvCisgICAgICAgIGJlIGRlc3RydWN0ZWQgb24g
ZXhpdC4KKworICAgICAgICBUaGlzIGNoYW5nZSBjYW4ndCBiZSB0ZXN0ZWQgYXV0b21hdGljYWxs
eSBiZWNhdXNlIFdlYktpdFRlc3RSdW5uZXIKKyAgICAgICAgZG9lc24ndCBzdXBwb3J0IFdpbmRv
d3MgcG9ydCB5ZXQuCisKKyAgICAgICAgKiBwbGF0Zm9ybS9ncmFwaGljcy93aW4vRm9udENhY2hl
V2luLmNwcDoKKyAgICAgICAgKFdlYkNvcmU6OkZvbnRDYWNoZTo6bGFzdFJlc29ydEZhbGxiYWNr
Rm9udCk6IENoYW5nZWQKKyAgICAgICAgZmFsbGJhY2tGb250cydzIHR5cGUgZnJvbSBBdG9taWNT
dHJpbmcgYXJyYXkgdG8KKyAgICAgICAgTmV2ZXJEZXN0cm95ZWQ8QXRvbWljU3RyaW5nPiBhcnJh
eS4KKwogMjAxOC0xMC0yMyAgSnVzdGluIEZhbiAgPGp1c3Rpbl9mYW5AYXBwbGUuY29tPgogCiAg
ICAgICAgIFtXZWJHUFVdIFJlbmFtZSBvbGQgR1BVKiBpbXBsZW1lbnRhdGlvbiBmaWxlcyB0byBH
UFVMZWdhY3kqIGZvciBXZWJNZXRhbApkaWZmIC0tZ2l0IGEvU291cmNlL1dlYkNvcmUvcGxhdGZv
cm0vZ3JhcGhpY3Mvd2luL0ZvbnRDYWNoZVdpbi5jcHAgYi9Tb3VyY2UvV2ViQ29yZS9wbGF0Zm9y
bS9ncmFwaGljcy93aW4vRm9udENhY2hlV2luLmNwcAppbmRleCBiZDM0MjUzMTYzNjE5MTMyY2Ez
M2JiZmI0NmYyZjIyYjUwMzk1YTY0Li4yODE2Y2I1ODYxYTBjZGFhYjI2ZmZjZWMxYThhOWFkOGYx
NWY1MzdmIDEwMDY0NAotLS0gYS9Tb3VyY2UvV2ViQ29yZS9wbGF0Zm9ybS9ncmFwaGljcy93aW4v
Rm9udENhY2hlV2luLmNwcAorKysgYi9Tb3VyY2UvV2ViQ29yZS9wbGF0Zm9ybS9ncmFwaGljcy93
aW4vRm9udENhY2hlV2luLmNwcApAQCAtMzUxLDcgKzM1MSw3IEBAIFJlZjxGb250PiBGb250Q2Fj
aGU6Omxhc3RSZXNvcnRGYWxsYmFja0ZvbnQoY29uc3QgRm9udERlc2NyaXB0aW9uJiBmb250RGVz
Y3JpcHRpCiAgICAgLy8gU2VhcmNoIGFsbCB0eXBpY2FsIFdpbmRvd3MtaW5zdGFsbGVkIGZ1bGwg
VW5pY29kZSBmb250cy4KICAgICAvLyBTb3J0ZWQgYnkgbW9zdCB0byBsZWFzdCBnbHlwaHMgYWNj
b3JkaW5nIHRvIGh0dHA6Ly9lbi53aWtpcGVkaWEub3JnL3dpa2kvVW5pY29kZV90eXBlZmFjZXMK
ICAgICAvLyBTdGFydCB3aXRoIFRpbWVzIE5ldyBSb21hbiBhbHNvIHNpbmNlIGl0IGlzIHRoZSBk
ZWZhdWx0IGlmIHRoZSB1c2VyIGRvZXNuJ3QgY2hhbmdlIHByZWZzLgotICAgIHN0YXRpYyBBdG9t
aWNTdHJpbmcgZmFsbGJhY2tGb250c1tdID0geworICAgIHN0YXRpYyBOZXZlckRlc3Ryb3llZDxB
dG9taWNTdHJpbmc+IGZhbGxiYWNrRm9udHNbXSA9IHsKICAgICAgICAgQXRvbWljU3RyaW5nKCJU
aW1lcyBOZXcgUm9tYW4iLCBBdG9taWNTdHJpbmc6OkNvbnN0cnVjdEZyb21MaXRlcmFsKSwKICAg
ICAgICAgQXRvbWljU3RyaW5nKCJNaWNyb3NvZnQgU2FucyBTZXJpZiIsIEF0b21pY1N0cmluZzo6
Q29uc3RydWN0RnJvbUxpdGVyYWwpLAogICAgICAgICBBdG9taWNTdHJpbmcoIlRhaG9tYSIsIEF0
b21pY1N0cmluZzo6Q29uc3RydWN0RnJvbUxpdGVyYWwpLAo=
</data>

          </attachment>
      

    </bug>

</bugzilla>