<?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>194905</bug_id>
          
          <creation_ts>2019-02-21 10:07:05 -0800</creation_ts>
          <short_desc>[Win] Implement Minibrowser without WebKitLegacy support</short_desc>
          <delta_ts>2023-02-15 13:22:14 -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>DUPLICATE</resolution>
          <dup_id>252131</dup_id>
          
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords></keywords>
          <priority>P2</priority>
          <bug_severity>Normal</bug_severity>
          <target_milestone>---</target_milestone>
          <dependson>235959</dependson>
          <blocked>194904</blocked>
          <everconfirmed>1</everconfirmed>
          <reporter name="Don Olmstead">don.olmstead</reporter>
          <assigned_to name="Yusuke Suzuki">ysuzuki</assigned_to>
          <cc>annulen</cc>
    
    <cc>darin</cc>
    
    <cc>ews-watchlist</cc>
    
    <cc>fujii</cc>
    
    <cc>gyuyoung.kim</cc>
    
    <cc>lforschler</cc>
    
    <cc>ryuan.choi</cc>
    
    <cc>sergio</cc>
    
    <cc>ysuzuki</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>1508556</commentid>
    <comment_count>0</comment_count>
    <who name="Don Olmstead">don.olmstead</who>
    <bug_when>2019-02-21 10:07:05 -0800</bug_when>
    <thetext>Minibrowser on Windows currently requires WebKitLegacy support. This should not be the case.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1830505</commentid>
    <comment_count>1</comment_count>
      <attachid>449041</attachid>
    <who name="Yusuke Suzuki">ysuzuki</who>
    <bug_when>2022-01-13 03:42:56 -0800</bug_when>
    <thetext>Created attachment 449041
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1830506</commentid>
    <comment_count>2</comment_count>
    <who name="Yusuke Suzuki">ysuzuki</who>
    <bug_when>2022-01-13 03:44:32 -0800</bug_when>
    <thetext>WebKitLegacy and DumpRenderTree has many CF code. So, disabling it makes it easy to replace remaining CF things with generic implementation, and we can drop CFLite more easily.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1830610</commentid>
    <comment_count>3</comment_count>
    <who name="Don Olmstead">don.olmstead</who>
    <bug_when>2022-01-13 10:22:31 -0800</bug_when>
    <thetext>(In reply to Yusuke Suzuki from comment #2)
&gt; WebKitLegacy and DumpRenderTree has many CF code. So, disabling it makes it
&gt; easy to replace remaining CF things with generic implementation, and we can
&gt; drop CFLite more easily.

I am all for doing just that, and have some WIP code lying around that tries to remove more CF code, BUT before we can land this we for sure need to move the testing from Legacy to Modern WebKit.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1830671</commentid>
    <comment_count>4</comment_count>
    <who name="Fujii Hironori">fujii</who>
    <bug_when>2022-01-13 12:45:04 -0800</bug_when>
    <thetext>Yusuke, I&apos;m surprised you are proposing a patch for this. Do you have a concrete problem if WinCairo is still using CFLite?
AppleWin is still using WK1. And, no one is actively working on it these days.
With WinCairo WK1, it&apos;s easy to maintain WK1 part for WinCairo developers. For example, Bug 234503 and Bug 234504.

We should have a WinCairo WK2 testing buildbot and fix more test failures and make the tests stable enough before removing WinCairo WK1.
Because removing WinCairo WK1 isn&apos;t high priority. no one have been working on it.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1831319</commentid>
    <comment_count>5</comment_count>
      <attachid>449041</attachid>
    <who name="Fujii Hironori">fujii</who>
    <bug_when>2022-01-16 12:14:17 -0800</bug_when>
    <thetext>Comment on attachment 449041
Patch

cq-. Don&apos;t land this patch until WinCairo has WK2 testing buildbot.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1933939</commentid>
    <comment_count>6</comment_count>
    <who name="Fujii Hironori">fujii</who>
    <bug_when>2023-02-15 13:22:14 -0800</bug_when>
    <thetext>

*** This bug has been marked as a duplicate of bug 252131 ***</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>449041</attachid>
            <date>2022-01-13 03:42:56 -0800</date>
            <delta_ts>2022-01-16 12:14:17 -0800</delta_ts>
            <desc>Patch</desc>
            <filename>bug-194905-20220113034255.patch</filename>
            <type>text/plain</type>
            <size>1278</size>
            <attacher name="Yusuke Suzuki">ysuzuki</attacher>
            
              <data encoding="base64">U3VidmVyc2lvbiBSZXZpc2lvbjogMjg3OTc1CmRpZmYgLS1naXQgYS9Tb3VyY2UvY21ha2UvT3B0
aW9uc1dpbkNhaXJvLmNtYWtlIGIvU291cmNlL2NtYWtlL09wdGlvbnNXaW5DYWlyby5jbWFrZQpp
bmRleCA1NGRjMDk1ZDlmOThjZjAxMTZhNzA0ZTk5ZTg4MzMzZjExMDhjOGEwLi4xOTUzNTkxYmI0
NDEyY2FkYWZmNmY0NGRlZGM1Y2Q2NWJjZWZjYmJmIDEwMDY0NAotLS0gYS9Tb3VyY2UvY21ha2Uv
T3B0aW9uc1dpbkNhaXJvLmNtYWtlCisrKyBiL1NvdXJjZS9jbWFrZS9PcHRpb25zV2luQ2Fpcm8u
Y21ha2UKQEAgLTEsOSArMSw4IEBACiBzZXQoV1RGX1BMQVRGT1JNX1dJTl9DQUlSTyAxKQogCi1p
bmNsdWRlKE9wdGlvbnNXaW4pCi0KLSMgVE9ETzogTW92ZSB0aGlzIGFib3ZlIE9wdGlvbnNXaW4g
d2hlbiBXZWJLaXRMZWdhY3kgaXMgZGVwcmVjYXRlZAorc2V0KEVOQUJMRV9XRUJLSVRfTEVHQUNZ
IE9GRikKIHNldChFTkFCTEVfV0VCS0lUIE9OKQoraW5jbHVkZShPcHRpb25zV2luKQogCiBmaW5k
X3BhY2thZ2UoQ2Fpcm8gMS4xNS4xMiBSRVFVSVJFRCkKIGZpbmRfcGFja2FnZShDVVJMIDcuNzcu
MCBSRVFVSVJFRCkKZGlmZiAtLWdpdCBhL0NoYW5nZUxvZyBiL0NoYW5nZUxvZwppbmRleCAxOGEx
OGVjNjljZTdkY2IxODAyOTlhYTk5ZWIwMGRkM2JjOTZjZjBiLi44ODlmNjYwMDhlNDdlYmNlOGE1
MjIzOWYwOGVhNGQ0MmIzY2MxNTkzIDEwMDY0NAotLS0gYS9DaGFuZ2VMb2cKKysrIGIvQ2hhbmdl
TG9nCkBAIC0xLDMgKzEsMTUgQEAKKzIwMjItMDEtMTMgIFl1c3VrZSBTdXp1a2kgIDx5c3V6dWtp
QGFwcGxlLmNvbT4KKworICAgICAgICBbV2luXSBJbXBsZW1lbnQgTWluaWJyb3dzZXIgd2l0aG91
dCBXZWJLaXRMZWdhY3kgc3VwcG9ydAorICAgICAgICBodHRwczovL2J1Z3Mud2Via2l0Lm9yZy9z
aG93X2J1Zy5jZ2k/aWQ9MTk0OTA1CisKKyAgICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BT
ISkuCisKKyAgICAgICAgQXMgTWluaUJyb3dzZXIgaXMgYWxyZWFkeSBjb25zdHJ1Y3RlZCB3aXRo
b3V0IFdlYktpdExlZ2FjeSAoR1RLIG9ubHkgaGFzIFdlYktpdCksCisgICAgICAgIHdlIGNhbiBq
dXN0IGRpc2FibGUgV2ViS2l0TGVnYWN5IG9uIFdpbkNhaXJvIGJ1aWxkcy4KKworICAgICAgICAq
IFNvdXJjZS9jbWFrZS9PcHRpb25zV2luQ2Fpcm8uY21ha2U6CisKIDIwMjItMDEtMTIgIENocmlz
IExvcmQgIDxjbG9yZEBpZ2FsaWEuY29tPgogCiAgICAgICAgIFtXUEVdIEZpeCBidWlsZGluZyBX
ZWJYUiB3aXRoIEFOR0xFIGVuYWJsZWQK
</data>
<flag name="review"
          id="475029"
          type_id="1"
          status="+"
          setter="darin"
    />
    <flag name="commit-queue"
          id="475330"
          type_id="3"
          status="-"
          setter="fujii"
    />
          </attachment>
      

    </bug>

</bugzilla>