<?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>59276</bug_id>
          
          <creation_ts>2011-04-23 08:25:04 -0700</creation_ts>
          <short_desc>[Gtk] No need to erase background before loading a new page</short_desc>
          <delta_ts>2011-05-09 17:03:52 -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>WebKitGTK</component>
          <version>528+ (Nightly build)</version>
          <rep_platform>PC</rep_platform>
          <op_sys>Linux</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>FIXED</resolution>
          
          
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords>Gtk</keywords>
          <priority>P2</priority>
          <bug_severity>Normal</bug_severity>
          <target_milestone>---</target_milestone>
          <dependson>59821</dependson>
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Joone Hur">joone.hur</reporter>
          <assigned_to name="Joone Hur">joone.hur</assigned_to>
          <cc>gustavo</cc>
    
    <cc>mrobinson</cc>
    
    <cc>xan.lopez</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>391450</commentid>
    <comment_count>0</comment_count>
    <who name="Joone Hur">joone.hur</who>
    <bug_when>2011-04-23 08:25:04 -0700</bug_when>
    <thetext>WebKitGtk+ erases background before loading a new page.
So, if it takes some time to load the page, users have to see the empty page for a second.
This makes the user think that the page loading is very slow.

However, in the case of other WebKit based browser, 
they leave the current page before preparing the layout of a new page, 
and then just replace the current page with the new one.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>391961</commentid>
    <comment_count>1</comment_count>
      <attachid>90968</attachid>
    <who name="Joone Hur">joone.hur</who>
    <bug_when>2011-04-25 15:45:34 -0700</bug_when>
    <thetext>Created attachment 90968
Propsed Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>392864</commentid>
    <comment_count>2</comment_count>
      <attachid>90968</attachid>
    <who name="Xan Lopez">xan.lopez</who>
    <bug_when>2011-04-26 15:45:46 -0700</bug_when>
    <thetext>Comment on attachment 90968
Propsed Patch

Isn&apos;t finishedLoad extremely late for us to attach the adjusments?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>392881</commentid>
    <comment_count>3</comment_count>
    <who name="Xan Lopez">xan.lopez</who>
    <bug_when>2011-04-26 15:52:18 -0700</bug_when>
    <thetext>OK, this is the load finished for the DOM Document, not for the whole thing, I think. I believe Martin is working to remove the need to attach adjustments entirely in any case, so perhaps we want to wait for that.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>392890</commentid>
    <comment_count>4</comment_count>
      <attachid>90968</attachid>
    <who name="Martin Robinson">mrobinson</who>
    <bug_when>2011-04-26 15:57:04 -0700</bug_when>
    <thetext>Comment on attachment 90968
Propsed Patch

Going to r- this one now, as hopefully in a few days we won&apos;t need to attach adjustments at all. If that approach fails though, please ping me later and we can re-address this one.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>400909</commentid>
    <comment_count>5</comment_count>
    <who name="Joone Hur">joone.hur</who>
    <bug_when>2011-05-09 17:03:52 -0700</bug_when>
    <thetext>#59821 fixed this issue.</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>90968</attachid>
            <date>2011-04-25 15:45:34 -0700</date>
            <delta_ts>2011-04-26 15:57:04 -0700</delta_ts>
            <desc>Propsed Patch</desc>
            <filename>prevent_erase_background.patch</filename>
            <type>text/plain</type>
            <size>2163</size>
            <attacher name="Joone Hur">joone.hur</attacher>
            
              <data encoding="base64">ZGlmZiAtLWdpdCBhL1NvdXJjZS9XZWJLaXQvZ3RrL0NoYW5nZUxvZyBiL1NvdXJjZS9XZWJLaXQv
Z3RrL0NoYW5nZUxvZwppbmRleCAwYjE1N2UxLi44NzRiN2MwIDEwMDY0NAotLS0gYS9Tb3VyY2Uv
V2ViS2l0L2d0ay9DaGFuZ2VMb2cKKysrIGIvU291cmNlL1dlYktpdC9ndGsvQ2hhbmdlTG9nCkBA
IC0xLDMgKzEsMTggQEAKKzIwMTEtMDQtMjQgIEpvb25lIEh1ciAgPGpvb25lLmh1ckBjb2xsYWJv
cmEuY28udWs+CisKKyAgICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BTISkuCisKKyAgICAg
ICAgW0d0a10gTm8gbmVlZCB0byBlcmFzZSBiYWNrZ3JvdW5kIGJlZm9yZSBsb2FkaW5nIGEgbmV3
IHBhZ2UKKyAgICAgICAgaHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19idWcuY2dpP2lkPTU5
Mjc2CisKKyAgICAgICAgVGhpcyBwYXRjaCBhbGxvd3MgdGhlIHNjcm9sbGJhcnMgdG8gYmUgYXR0
YWNoZWQgdG8gdGhlIHZpZXcgYSBiaXQgbGF0ZXIgCisgICAgICAgIHRvIHByZXZlbnQgZXJhc2lu
ZyB0aGUgYmFja2dyb3VuZCB3aGVuIGxvYWRpbmcgYSBuZXcgcGFnZS4gCisKKyAgICAgICAgKiBX
ZWJDb3JlU3VwcG9ydC9GcmFtZUxvYWRlckNsaWVudEd0ay5jcHA6CisgICAgICAgIChXZWJLaXQ6
OkZyYW1lTG9hZGVyQ2xpZW50OjpmaW5pc2hlZExvYWRpbmcpOiBBdHRhY2ggdGhlIHNjcm9sbGJh
cnMuCisgICAgICAgIChXZWJLaXQ6OnBvc3RDb21taXRGcmFtZVZpZXdTZXR1cCk6IFJlbW92ZWQg
dGhlIGNhbGwgdG8gYXR0YWNoIHRoZSBzY3JvbGxiYXJzIHRvIHByZXZlbnQKKyAgICAgICAgZXJh
c2luZyB0aGUgYmFja2dyb3VuZC4KKwogMjAxMS0wNC0yMiAgTWFydGluIFJvYmluc29uICA8bXJv
Ymluc29uQGlnYWxpYS5jb20+CiAKICAgICAgICAgUmV2aWV3ZWQgYnkgWGFuIExvcGV6LgpkaWZm
IC0tZ2l0IGEvU291cmNlL1dlYktpdC9ndGsvV2ViQ29yZVN1cHBvcnQvRnJhbWVMb2FkZXJDbGll
bnRHdGsuY3BwIGIvU291cmNlL1dlYktpdC9ndGsvV2ViQ29yZVN1cHBvcnQvRnJhbWVMb2FkZXJD
bGllbnRHdGsuY3BwCmluZGV4IDUzOTY3NWEuLmMxODNiMGUgMTAwNjQ0Ci0tLSBhL1NvdXJjZS9X
ZWJLaXQvZ3RrL1dlYkNvcmVTdXBwb3J0L0ZyYW1lTG9hZGVyQ2xpZW50R3RrLmNwcAorKysgYi9T
b3VyY2UvV2ViS2l0L2d0ay9XZWJDb3JlU3VwcG9ydC9GcmFtZUxvYWRlckNsaWVudEd0ay5jcHAK
QEAgLTExMDMsNiArMTEwMywxNCBAQCB2b2lkIEZyYW1lTG9hZGVyQ2xpZW50OjpmaW5pc2hlZExv
YWRpbmcoV2ViQ29yZTo6RG9jdW1lbnRMb2FkZXIqIGRvY3VtZW50TG9hZGVyKQogICAgICAgICBt
X3BsdWdpblZpZXcgPSAwOwogICAgICAgICBtX2hhc1NlbnRSZXNwb25zZVRvUGx1Z2luID0gZmFs
c2U7CiAgICAgfQorCisgICAgRnJhbWUqIGZyYW1lID0gY29yZShtX2ZyYW1lKTsKKyAgICBpZiAo
ZnJhbWUgIT0gZnJhbWUtPnBhZ2UoKS0+bWFpbkZyYW1lKCkpCisgICAgICByZXR1cm47CisKKyAg
ICBXZWJLaXRXZWJWaWV3KiB3ZWJWaWV3ID0gZ2V0Vmlld0Zyb21GcmFtZShtX2ZyYW1lKTsKKyAg
ICBXZWJLaXRXZWJWaWV3UHJpdmF0ZSogcHJpdiA9IHdlYlZpZXctPnByaXY7CisgICAgZnJhbWUt
PnZpZXcoKS0+c2V0R3RrQWRqdXN0bWVudHMocHJpdi0+aG9yaXpvbnRhbEFkanVzdG1lbnQuZ2V0
KCksIHByaXYtPnZlcnRpY2FsQWRqdXN0bWVudC5nZXQoKSwgdHJ1ZSk7CiB9CiAKIApAQCAtMTM2
MCw3ICsxMzY4LDYgQEAgc3RhdGljIHZvaWQgcG9zdENvbW1pdEZyYW1lVmlld1NldHVwKFdlYktp
dFdlYkZyYW1lICpmcmFtZSwgRnJhbWVWaWV3ICp2aWV3LCBib28KICAgICB3ZWJraXRfd2ViX3Zp
ZXdfY2xlYXJfcmVzb3VyY2VzKGNvbnRhaW5pbmdXaW5kb3cpOwogCiAgICAgV2ViS2l0V2ViVmll
d1ByaXZhdGUqIHByaXYgPSBjb250YWluaW5nV2luZG93LT5wcml2OwotICAgIHZpZXctPnNldEd0
a0FkanVzdG1lbnRzKHByaXYtPmhvcml6b250YWxBZGp1c3RtZW50LmdldCgpLCBwcml2LT52ZXJ0
aWNhbEFkanVzdG1lbnQuZ2V0KCksIHJlc2V0VmFsdWVzKTsKIAogICAgIC8vIEludmFsaWRhdGUg
dGhlIHZpZXdwb3J0IGF0dHJpYnV0ZXMgLSB0aGV5IHdpbGwgb25seSBiZSB2YWxpZAogICAgIC8v
IGFnYWluIGlmIHRoZSBwYWdlIHdlJ3JlIGJlZ2lubmluZyB0byBsb2FkIG5vdyBoYXMgYW4K
</data>
<flag name="review"
          id="83737"
          type_id="1"
          status="-"
          setter="mrobinson"
    />
          </attachment>
      

    </bug>

</bugzilla>