<?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>200218</bug_id>
          
          <creation_ts>2019-07-29 00:00:40 -0700</creation_ts>
          <short_desc>[Win][MiniBrowser][WK2] Automatically supplement &quot;file://&quot; or &quot;http://&quot; for an input URL as well as WK1 browser window</short_desc>
          <delta_ts>2019-07-29 18:48:18 -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>Tools / Tests</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>pvollan</cc>
    
    <cc>ross.kirsling</cc>
    
    <cc>webkit-bug-importer</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>1556503</commentid>
    <comment_count>0</comment_count>
    <who name="Fujii Hironori">fujii</who>
    <bug_when>2019-07-29 00:00:40 -0700</bug_when>
    <thetext>[Win][MiniBrowser][WK2] Automatically supplement &quot;file://&quot; or &quot;http://&quot; for an input URL as well as WK1 browser window

* Move the logic from WebKitLegacyBrowserWindow::loadURL to MainWindow::loadURL</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1556504</commentid>
    <comment_count>1</comment_count>
      <attachid>375066</attachid>
    <who name="Fujii Hironori">fujii</who>
    <bug_when>2019-07-29 00:02:52 -0700</bug_when>
    <thetext>Created attachment 375066
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1556609</commentid>
    <comment_count>2</comment_count>
      <attachid>375066</attachid>
    <who name="Brent Fulgham">bfulgham</who>
    <bug_when>2019-07-29 10:42:04 -0700</bug_when>
    <thetext>Comment on attachment 375066
Patch

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

&gt; Tools/MiniBrowser/win/MainWindow.cpp:442
&gt; +        url = L&quot;http://&quot; + url;

http, or https? :-)</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1556612</commentid>
    <comment_count>3</comment_count>
      <attachid>375066</attachid>
    <who name="Alex Christensen">achristensen</who>
    <bug_when>2019-07-29 10:51:42 -0700</bug_when>
    <thetext>Comment on attachment 375066
Patch

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

&gt;&gt; Tools/MiniBrowser/win/MainWindow.cpp:442
&gt;&gt; +        url = L&quot;http://&quot; + url;
&gt; 
&gt; http, or https? :-)

You could be the first browser to default to https if you want. If people want the plaintext page, they can type in the protocol themselves.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1556857</commentid>
    <comment_count>4</comment_count>
    <who name="Fujii Hironori">fujii</who>
    <bug_when>2019-07-29 18:36:24 -0700</bug_when>
    <thetext>Somewhat too early. After Mac port MiniBrowser will change, I&apos;ll change Windows port MiniBrowser.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1556863</commentid>
    <comment_count>5</comment_count>
    <who name="Fujii Hironori">fujii</who>
    <bug_when>2019-07-29 18:48:00 -0700</bug_when>
    <thetext>Committed r247935: &lt;https://trac.webkit.org/changeset/247935&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1556864</commentid>
    <comment_count>6</comment_count>
    <who name="Radar WebKit Bug Importer">webkit-bug-importer</who>
    <bug_when>2019-07-29 18:48:18 -0700</bug_when>
    <thetext>&lt;rdar://problem/53690116&gt;</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>375066</attachid>
            <date>2019-07-29 00:02:52 -0700</date>
            <delta_ts>2019-07-29 10:39:04 -0700</delta_ts>
            <desc>Patch</desc>
            <filename>bug-200218-20190729160250.patch</filename>
            <type>text/plain</type>
            <size>4302</size>
            <attacher name="Fujii Hironori">fujii</attacher>
            
              <data encoding="base64">U3VidmVyc2lvbiBSZXZpc2lvbjogMjQ3ODk4CmRpZmYgLS1naXQgYS9Ub29scy9DaGFuZ2VMb2cg
Yi9Ub29scy9DaGFuZ2VMb2cKaW5kZXggZGFkZDFiODYwMzg1YTllNTI2ZTJmMGIzYTk2MGFkOTAz
MjQ5YTMyOC4uNDZmNWM1M2UyMTNmZWQ1NzZmNDAxMjc3NjBhZjk2NDM1OWJlNDY0MCAxMDA2NDQK
LS0tIGEvVG9vbHMvQ2hhbmdlTG9nCisrKyBiL1Rvb2xzL0NoYW5nZUxvZwpAQCAtMSwzICsxLDIw
IEBACisyMDE5LTA3LTI5ICBGdWppaSBIaXJvbm9yaSAgPEhpcm9ub3JpLkZ1amlpQHNvbnkuY29t
PgorCisgICAgICAgIFtXaW5dW01pbmlCcm93c2VyXVtXSzJdIEF1dG9tYXRpY2FsbHkgc3VwcGxl
bWVudCAiZmlsZTovLyIgb3IgImh0dHA6Ly8iIGZvciBhbiBpbnB1dCBVUkwgYXMgd2VsbCBhcyBX
SzEgYnJvd3NlciB3aW5kb3cKKyAgICAgICAgaHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19i
dWcuY2dpP2lkPTIwMDIxOAorCisgICAgICAgIFJldmlld2VkIGJ5IE5PQk9EWSAoT09QUyEpLgor
CisgICAgICAgIE1vdmUgdGhlIHNjaGVtZSBzdXBwbGVtZW50aW5nIGxvZ2ljIGZyb20KKyAgICAg
ICAgV2ViS2l0TGVnYWN5QnJvd3NlcldpbmRvdzo6bG9hZFVSTCB0byBNYWluV2luZG93Ojpsb2Fk
VVJMLgorCisgICAgICAgICogTWluaUJyb3dzZXIvd2luL01haW5XaW5kb3cuY3BwOgorICAgICAg
ICAoTWFpbldpbmRvdzo6bG9hZFVSTCk6CisgICAgICAgIChNYWluV2luZG93OjpvblVSTEJhckVu
dGVyKToKKyAgICAgICAgKiBNaW5pQnJvd3Nlci93aW4vTWFpbldpbmRvdy5oOgorICAgICAgICAq
IE1pbmlCcm93c2VyL3dpbi9XZWJLaXRMZWdhY3lCcm93c2VyV2luZG93LmNwcDoKKyAgICAgICAg
KFdlYktpdExlZ2FjeUJyb3dzZXJXaW5kb3c6OmxvYWRVUkwpOgorCiAyMDE5LTA3LTI4ICBUaW0g
SG9ydG9uICA8dGltb3RoeV9ob3J0b25AYXBwbGUuY29tPgogCiAgICAgICAgIFJlb3JnYW5pemUg
VUlTY3JpcHRDb250cm9sbGVyIGludG8gcGxhdGZvcm0tc3BlY2lmaWMgc3ViY2xhc3NlcwpkaWZm
IC0tZ2l0IGEvVG9vbHMvTWluaUJyb3dzZXIvd2luL01haW5XaW5kb3cuY3BwIGIvVG9vbHMvTWlu
aUJyb3dzZXIvd2luL01haW5XaW5kb3cuY3BwCmluZGV4IDNmN2Q3NGMwNjYwODU2MGU3NTE5MzA2
ZThlMGMxZWYyODRlYTY1Y2UuLjdkNGMxZjZhODk1M2Y5Yzc4M2FmY2YzZTBiMWZmODE3ZWQyMDFl
MzIgMTAwNjQ0Ci0tLSBhL1Rvb2xzL01pbmlCcm93c2VyL3dpbi9NYWluV2luZG93LmNwcAorKysg
Yi9Ub29scy9NaW5pQnJvd3Nlci93aW4vTWFpbldpbmRvdy5jcHAKQEAgLTQyOSw5ICs0MjksMTkg
QEAgSU5UX1BUUiBDQUxMQkFDSyBNYWluV2luZG93OjpjdXN0b21Vc2VyQWdlbnREaWFsb2dQcm9j
KEhXTkQgaERsZywgVUlOVCBtZXNzYWdlLAogICAgIHJldHVybiAoSU5UX1BUUilGQUxTRTsKIH0K
IAotdm9pZCBNYWluV2luZG93Ojpsb2FkVVJMKEJTVFIgdXJsKQordm9pZCBNYWluV2luZG93Ojps
b2FkVVJMKHN0ZDo6d3N0cmluZyB1cmwpCiB7Ci0gICAgaWYgKEZBSUxFRChtX2Jyb3dzZXJXaW5k
b3ctPmxvYWRVUkwodXJsKSkpCisgICAgaWYgKDo6UGF0aEZpbGVFeGlzdHModXJsLmNfc3RyKCkp
IHx8IDo6UGF0aElzVU5DKHVybC5jX3N0cigpKSkgeworICAgICAgICB3Y2hhcl90IGZpbGVVUkxb
SU5URVJORVRfTUFYX1VSTF9MRU5HVEhdOworICAgICAgICBEV09SRCBmaWxlVVJMTGVuZ3RoID0g
X2NvdW50b2YoZmlsZVVSTCk7CisKKyAgICAgICAgaWYgKFNVQ0NFRURFRCg6OlVybENyZWF0ZUZy
b21QYXRoKHVybC5jX3N0cigpLCBmaWxlVVJMLCAmZmlsZVVSTExlbmd0aCwgMCkpKQorICAgICAg
ICAgICAgdXJsID0gZmlsZVVSTDsKKyAgICB9CisgICAgaWYgKHVybC5maW5kKEwiOi8vIikgPT0g
dXJsLm5wb3MpCisgICAgICAgIHVybCA9IEwiaHR0cDovLyIgKyB1cmw7CisKKyAgICBpZiAoRkFJ
TEVEKG1fYnJvd3NlcldpbmRvdy0+bG9hZFVSTChfYnN0cl90KHVybC5jX3N0cigpKSkpKQogICAg
ICAgICByZXR1cm47CiAKICAgICBTZXRGb2N1cyhtX2Jyb3dzZXJXaW5kb3ctPmh3bmQoKSk7CkBA
IC00NDIsOCArNDUyLDcgQEAgdm9pZCBNYWluV2luZG93OjpvblVSTEJhckVudGVyKCkKICAgICB3
Y2hhcl90IHN0clB0cltJTlRFUk5FVF9NQVhfVVJMX0xFTkdUSF07CiAgICAgR2V0V2luZG93VGV4
dChtX2hVUkxCYXJXbmQsIHN0clB0ciwgSU5URVJORVRfTUFYX1VSTF9MRU5HVEgpOwogICAgIHN0
clB0cltJTlRFUk5FVF9NQVhfVVJMX0xFTkdUSCAtIDFdID0gMDsKLSAgICBfYnN0cl90IGJzdHIo
c3RyUHRyKTsKLSAgICBsb2FkVVJMKGJzdHIuR2V0QlNUUigpKTsKKyAgICBsb2FkVVJMKHN0clB0
cik7CiB9CiAKIHZvaWQgTWFpbldpbmRvdzo6dXBkYXRlRGV2aWNlU2NhbGVGYWN0b3IoKQpkaWZm
IC0tZ2l0IGEvVG9vbHMvTWluaUJyb3dzZXIvd2luL01haW5XaW5kb3cuaCBiL1Rvb2xzL01pbmlC
cm93c2VyL3dpbi9NYWluV2luZG93LmgKaW5kZXggYzU3MjA3NmE2OGM2YmJjYTRlNTE1ZTkyMzM4
YjhlZmI4OWZkMzEyMS4uZjZkZDhmNTk0OGE0ZTU2Y2NkYjAxZTUwZTI4NzEwODFhZGM1YTUxMiAx
MDA2NDQKLS0tIGEvVG9vbHMvTWluaUJyb3dzZXIvd2luL01haW5XaW5kb3cuaAorKysgYi9Ub29s
cy9NaW5pQnJvd3Nlci93aW4vTWFpbldpbmRvdy5oCkBAIC00NCw3ICs0NCw3IEBAIHB1YmxpYzoK
ICAgICBIV05EIGh3bmQoKSBjb25zdCB7IHJldHVybiBtX2hNYWluV25kOyB9CiAgICAgQnJvd3Nl
cldpbmRvdyogYnJvd3NlcldpbmRvdygpIGNvbnN0IHsgcmV0dXJuIG1fYnJvd3NlcldpbmRvdy5n
ZXQoKTsgfQogCi0gICAgdm9pZCBsb2FkVVJMKEJTVFIgdXJsKTsKKyAgICB2b2lkIGxvYWRVUkwo
c3RkOjp3c3RyaW5nKTsKICAgICAKIHByaXZhdGU6CiAgICAgc3RhdGljIExSRVNVTFQgQ0FMTEJB
Q0sgV25kUHJvYyhIV05ELCBVSU5ULCBXUEFSQU0sIExQQVJBTSk7CmRpZmYgLS1naXQgYS9Ub29s
cy9NaW5pQnJvd3Nlci93aW4vV2ViS2l0TGVnYWN5QnJvd3NlcldpbmRvdy5jcHAgYi9Ub29scy9N
aW5pQnJvd3Nlci93aW4vV2ViS2l0TGVnYWN5QnJvd3NlcldpbmRvdy5jcHAKaW5kZXggNTE2YTc4
NWVhOTExZDE5N2M3MWQ5YWZmMDlkNmVmYThhZTQ2MWQ3Yy4uZjBjZDQ0ZTM4MTk5MTEyY2M4NWEz
NjE5MmEyZWViZTJjZDY0ZWQzMSAxMDA2NDQKLS0tIGEvVG9vbHMvTWluaUJyb3dzZXIvd2luL1dl
YktpdExlZ2FjeUJyb3dzZXJXaW5kb3cuY3BwCisrKyBiL1Rvb2xzL01pbmlCcm93c2VyL3dpbi9X
ZWJLaXRMZWdhY3lCcm93c2VyV2luZG93LmNwcApAQCAtNDc0LDE1ICs0NzQsNiBAQCBIUkVTVUxU
IFdlYktpdExlZ2FjeUJyb3dzZXJXaW5kb3c6OmxvYWRVUkwoY29uc3QgQlNUUiYgcGFzc2VkVVJM
KQogICAgIGlmICghcGFzc2VkVVJMKQogICAgICAgICByZXR1cm4gRV9JTlZBTElEQVJHOwogCi0g
ICAgX2JzdHJfdCB1cmxCU3RyKHBhc3NlZFVSTCk7Ci0gICAgaWYgKCEhdXJsQlN0ciAmJiAoOjpQ
YXRoRmlsZUV4aXN0cyh1cmxCU3RyKSB8fCA6OlBhdGhJc1VOQyh1cmxCU3RyKSkpIHsKLSAgICAg
ICAgVENIQVIgZmlsZVVSTFtJTlRFUk5FVF9NQVhfVVJMX0xFTkdUSF07Ci0gICAgICAgIERXT1JE
IGZpbGVVUkxMZW5ndGggPSBzaXplb2YoZmlsZVVSTCkgLyBzaXplb2YoZmlsZVVSTFswXSk7Ci0K
LSAgICAgICAgaWYgKFNVQ0NFRURFRCg6OlVybENyZWF0ZUZyb21QYXRoKHVybEJTdHIsIGZpbGVV
UkwsICZmaWxlVVJMTGVuZ3RoLCAwKSkpCi0gICAgICAgICAgICB1cmxCU3RyID0gZmlsZVVSTDsK
LSAgICB9Ci0KICAgICBJV2ViRnJhbWVQdHIgZnJhbWU7CiAgICAgSFJFU1VMVCBociA9IG1fd2Vi
Vmlldy0+bWFpbkZyYW1lKCZmcmFtZS5HZXRJbnRlcmZhY2VQdHIoKSk7CiAgICAgaWYgKEZBSUxF
RChocikpCkBAIC00OTMsNyArNDg0LDcgQEAgSFJFU1VMVCBXZWJLaXRMZWdhY3lCcm93c2VyV2lu
ZG93Ojpsb2FkVVJMKGNvbnN0IEJTVFImIHBhc3NlZFVSTCkKICAgICBpZiAoRkFJTEVEKGhyKSkK
ICAgICAgICAgcmV0dXJuIGhyOwogCi0gICAgaHIgPSByZXF1ZXN0LT5pbml0V2l0aFVSTCh3Y3Nz
dHIoc3RhdGljX2Nhc3Q8d2NoYXJfdCo+KHVybEJTdHIpLCBMIjovLyIpID8gdXJsQlN0ciA6IF9i
c3RyX3QoTCJodHRwOi8vIikgKyB1cmxCU3RyLCBXZWJVUkxSZXF1ZXN0VXNlUHJvdG9jb2xDYWNo
ZVBvbGljeSwgNjApOworICAgIGhyID0gcmVxdWVzdC0+aW5pdFdpdGhVUkwocGFzc2VkVVJMLCBX
ZWJVUkxSZXF1ZXN0VXNlUHJvdG9jb2xDYWNoZVBvbGljeSwgNjApOwogICAgIGlmIChGQUlMRUQo
aHIpKQogICAgICAgICByZXR1cm4gaHI7CiAK
</data>
<flag name="review"
          id="390852"
          type_id="1"
          status="+"
          setter="achristensen"
    />
          </attachment>
      

    </bug>

</bugzilla>