<?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>54142</bug_id>
          
          <creation_ts>2011-02-09 14:07:22 -0800</creation_ts>
          <short_desc>Crash in DrawingAreaProxyImpl::paint when WKView paints after being resized while hidden</short_desc>
          <delta_ts>2011-02-10 14:52:06 -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>Layout and Rendering</component>
          <version>528+ (Nightly build)</version>
          <rep_platform>All</rep_platform>
          <op_sys>All</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>
          <dependson>54141</dependson>
          <blocked>53805</blocked>
          <everconfirmed>1</everconfirmed>
          <reporter name="Adam Roben (:aroben)">aroben</reporter>
          <assigned_to name="Nobody">webkit-unassigned</assigned_to>
          <cc>andersca</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>348554</commentid>
    <comment_count>0</comment_count>
    <who name="Adam Roben (:aroben)">aroben</who>
    <bug_when>2011-02-09 14:07:22 -0800</bug_when>
    <thetext>If WKView is resized while hidden (and you don&apos;t run into bug 54141, because it&apos;s been fixed), and is then shown and asked to paint, you&apos;ll crash inside DrawingAreaProxyImpl::paint due to a null BackingStore. The crashing line is this:

    m_backingStore-&gt;paint(context, rect);

m_backingStore is null. Here&apos;s the backtrace:

&gt;	WebKit.dll!WTF::OwnPtr&lt;WebKit::BackingStore&gt;::operator-&gt;()  Line 66 + 0x35 bytes	C++
 	WebKit.dll!WebKit::DrawingAreaProxyImpl::paint(HDC__ * context=0xaa011176, const WebCore::IntRect &amp; rect={...}, WebKit::Region &amp; unpaintedRegion={...})  Line 83 + 0x13 bytes	C++
 	WebKit.dll!WebKit::WebView::onPaintEvent(HWND__ * hWnd=0x00120418, unsigned int message=15, unsigned int __formal=0, unsigned int __formal=0, bool &amp; handled=true)  Line 427	C++
 	WebKit.dll!WebKit::WebView::wndProc(HWND__ * hWnd=0x00120418, unsigned int message=15, unsigned int wParam=0, long lParam=0)  Line 127 + 0x1c bytes	C++
 	WebKit.dll!WebKit::WebView::WebViewWndProc(HWND__ * hWnd=0x00120418, unsigned int message=15, unsigned int wParam=0, long lParam=0)  Line 97 + 0x18 bytes	C++
 	user32.dll!_InternalCallWinProc@20()  + 0x28 bytes</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>348555</commentid>
    <comment_count>1</comment_count>
    <who name="Adam Roben (:aroben)">aroben</who>
    <bug_when>2011-02-09 14:08:06 -0800</bug_when>
    <thetext>&lt;rdar://problem/8979365&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>348562</commentid>
    <comment_count>2</comment_count>
      <attachid>81874</attachid>
    <who name="Adam Roben (:aroben)">aroben</who>
    <bug_when>2011-02-09 14:16:04 -0800</bug_when>
    <thetext>Created attachment 81874
Notice when the backing store goes away after DidSetSize</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>348945</commentid>
    <comment_count>3</comment_count>
    <who name="Adam Roben (:aroben)">aroben</who>
    <bug_when>2011-02-10 04:43:16 -0800</bug_when>
    <thetext>Committed r78199: &lt;http://trac.webkit.org/changeset/78199&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>349401</commentid>
    <comment_count>4</comment_count>
    <who name="Anders Carlsson">andersca</who>
    <bug_when>2011-02-10 14:50:38 -0800</bug_when>
    <thetext>*** Bug 54247 has been marked as a duplicate of this bug. ***</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>81874</attachid>
            <date>2011-02-09 14:16:04 -0800</date>
            <delta_ts>2011-02-09 15:31:22 -0800</delta_ts>
            <desc>Notice when the backing store goes away after DidSetSize</desc>
            <filename>bug-54142-20110209171601.patch</filename>
            <type>text/plain</type>
            <size>4223</size>
            <attacher name="Adam Roben (:aroben)">aroben</attacher>
            
              <data encoding="base64">U3VidmVyc2lvbiBSZXZpc2lvbjogNzgwNTQKZGlmZiAtLWdpdCBhL1NvdXJjZS9XZWJLaXQyL0No
YW5nZUxvZyBiL1NvdXJjZS9XZWJLaXQyL0NoYW5nZUxvZwppbmRleCBkZWFkNzBmMGYwZGU2Nzhl
NTYxOTRjMjM3MjdmZDMwNTVlMzI3ZTFjLi4yZDU2YTc3MTkxNmQ0YWVmZGE4OGE4ZmYyY2MwMmQ4
ZWE5NDdmY2I0IDEwMDY0NAotLS0gYS9Tb3VyY2UvV2ViS2l0Mi9DaGFuZ2VMb2cKKysrIGIvU291
cmNlL1dlYktpdDIvQ2hhbmdlTG9nCkBAIC0xLDUgKzEsMjEgQEAKIDIwMTEtMDItMDkgIEFkYW0g
Um9iZW4gIDxhcm9iZW5AYXBwbGUuY29tPgogCisgICAgICAgIE5vdGljZSB3aGVuIHRoZSBiYWNr
aW5nIHN0b3JlIGdvZXMgYXdheSBhZnRlciBEaWRTZXRTaXplCisKKyAgICAgICAgRml4ZXMgPGh0
dHA6Ly93ZWJraXQub3JnL2IvNTQxNDI+IDxyZGFyOi8vcHJvYmxlbS84OTc5MzY1PiBDcmFzaCBp
bgorICAgICAgICBEcmF3aW5nQXJlYVByb3h5SW1wbDo6cGFpbnQgd2hlbiBXS1ZpZXcgcGFpbnRz
IGFmdGVyIGJlaW5nIHJlc2l6ZWQgd2hpbGUgaGlkZGVuCisKKyAgICAgICAgUmV2aWV3ZWQgYnkg
Tk9CT0RZIChPT1BTISkuCisKKyAgICAgICAgKiBVSVByb2Nlc3MvRHJhd2luZ0FyZWFQcm94eUlt
cGwuY3BwOgorICAgICAgICAoV2ViS2l0OjpEcmF3aW5nQXJlYVByb3h5SW1wbDo6cGFpbnQpOiBO
dWxsLWNoZWNrIG1fYmFja2luZ1N0b3JlIGFmdGVyIHdhaXRpbmcgZm9yCisgICAgICAgIERpZFNl
dFNpemUsIGFzIG1fYmFja2luZ1N0b3JlIGNhbiBnbyBhd2F5IHdoZW4gd2UgcmVjZWl2ZSB0aGF0
IG1lc3NhZ2UuIFRoaXMgaXMgdGhlIGJ1ZyBmaXguCisgICAgICAgIEFsc28gbW92ZWQgdGhlIGFj
Y2VsZXJhdGVkIGNvbXBvc2l0aW5nIGNoZWNrIGluc2lkZSB0aGUgYmxvY2sgd2hlcmUgd2Ugd2Fp
dCBmb3IgRGlkU2V0U2l6ZS4KKyAgICAgICAgSWYgd2UgZGlkbid0IHdhaXQgZm9yIERpZFNldFNp
emUsIHdlIGFscmVhZHkga25vdyB0aGF0IHdlJ3JlIG5vdCBpbiBhY2NlbGVyYXRlZCBjb21wb3Np
dGluZworICAgICAgICBtb2RlIChhbmQgYXNzZXJ0ZWQgc28gZWFybGllciBpbiB0aGlzIGZ1bmN0
aW9uKS4KKworMjAxMS0wMi0wOSAgQWRhbSBSb2JlbiAgPGFyb2JlbkBhcHBsZS5jb20+CisKICAg
ICAgICAgRGVjb2RlIGEgbnVsbCBTaGFyZWRNZW1vcnk6OkhhbmRsZSBjb3JyZWN0bHkgb24gV2lu
ZG93cwogCiAgICAgICAgIEZpeGVzIDxodHRwOi8vd2Via2l0Lm9yZy9iLzU0MTQxPiA8cmRhcjov
L3Byb2JsZW0vODk3OTE5NT4gV0tWaWV3IGRyYXdzIGF0IHdyb25nIHNpemUsCmRpZmYgLS1naXQg
YS9Tb3VyY2UvV2ViS2l0Mi9VSVByb2Nlc3MvRHJhd2luZ0FyZWFQcm94eUltcGwuY3BwIGIvU291
cmNlL1dlYktpdDIvVUlQcm9jZXNzL0RyYXdpbmdBcmVhUHJveHlJbXBsLmNwcAppbmRleCAwMWFl
NjhmN2ExZDlkOWZlNTRmMGM4MWUyN2Q4MTIxYjZkY2I2NTUyLi42YTY1ODQxMzgwOWZmNTgxNzE5
MzM1OWU2ZjA3OTgxNWE1MWExOTc1IDEwMDY0NAotLS0gYS9Tb3VyY2UvV2ViS2l0Mi9VSVByb2Nl
c3MvRHJhd2luZ0FyZWFQcm94eUltcGwuY3BwCisrKyBiL1NvdXJjZS9XZWJLaXQyL1VJUHJvY2Vz
cy9EcmF3aW5nQXJlYVByb3h5SW1wbC5jcHAKQEAgLTc0LDExICs3NCwxMSBAQCB2b2lkIERyYXdp
bmdBcmVhUHJveHlJbXBsOjpwYWludChCYWNraW5nU3RvcmU6OlBsYXRmb3JtR3JhcGhpY3NDb250
ZXh0IGNvbnRleHQsCiAgICAgICAgIC8vIFdhaXQgZm9yIGEgRGlkU2V0U2l6ZSBtZXNzYWdlIHRo
YXQgY29udGFpbnMgdGhlIG5ldyBiaXRzIGJlZm9yZSB3ZSBwYWludAogICAgICAgICAvLyB3aGF0
J3MgY3VycmVudGx5IGluIHRoZSBiYWNraW5nIHN0b3JlLgogICAgICAgICB3YWl0Rm9yQW5kRGlz
cGF0Y2hEaWRTZXRTaXplKCk7Ci0gICAgfQogCi0gICAgLy8gRGlzcGF0Y2hpbmcgRGlkU2V0U2l6
ZSBjb3VsZCBjaGFuZ2UgdGhlIGNvbXBvc2l0aW5nIG1vZGUsIHJldHVybiBpZiB0aGF0IGhhcHBl
bnMuCi0gICAgaWYgKGlzSW5BY2NlbGVyYXRlZENvbXBvc2l0aW5nTW9kZSgpKQotICAgICAgICBy
ZXR1cm47CisgICAgICAgIC8vIERpc3BhdGNoaW5nIERpZFNldFNpemUgY291bGQgZGVzdHJveSBv
dXIgYmFja2luZyBzdG9yZSBvciBjaGFuZ2UgdGhlIGNvbXBvc2l0aW5nIG1vZGUuCisgICAgICAg
IGlmICghbV9iYWNraW5nU3RvcmUgfHwgaXNJbkFjY2VsZXJhdGVkQ29tcG9zaXRpbmdNb2RlKCkp
CisgICAgICAgICAgICByZXR1cm47CisgICAgfQogCiAgICAgbV9iYWNraW5nU3RvcmUtPnBhaW50
KGNvbnRleHQsIHJlY3QpOwogICAgIHVucGFpbnRlZFJlZ2lvbi5zdWJ0cmFjdChJbnRSZWN0KElu
dFBvaW50KCksIG1fYmFja2luZ1N0b3JlLT5zaXplKCkpKTsKZGlmZiAtLWdpdCBhL1Rvb2xzL0No
YW5nZUxvZyBiL1Rvb2xzL0NoYW5nZUxvZwppbmRleCBhZmE5NmMxODM5YmU4NjBmNGQyMGEyZjJj
MjNkZGVjYjI3ZjY1YWNiLi5jZGRjYmJkY2Y3ZjcwZjlmMzg2Yzk1MzUwOTJjZDdlM2U1MTcyZDE5
IDEwMDY0NAotLS0gYS9Ub29scy9DaGFuZ2VMb2cKKysrIGIvVG9vbHMvQ2hhbmdlTG9nCkBAIC0x
LDUgKzEsMTcgQEAKIDIwMTEtMDItMDkgIEFkYW0gUm9iZW4gIDxhcm9iZW5AYXBwbGUuY29tPgog
CisgICAgICAgIFRlc3QgdGhhdCBoYXZpbmcgYSBXS1ZpZXcgcGFpbnQgYWZ0ZXIgYmVpbmcgcmVz
aXplZCB3aGlsZSBoaWRkZW4gZG9lc24ndCBjYXVzZSBhIGNyYXNoCisKKyAgICAgICAgVGVzdCBm
b3IgPGh0dHA6Ly93ZWJraXQub3JnL2IvNTQxNDI+IDxyZGFyOi8vcHJvYmxlbS84OTc5MzY1PiBD
cmFzaCBpbgorICAgICAgICBEcmF3aW5nQXJlYVByb3h5SW1wbDo6cGFpbnQgd2hlbiBXS1ZpZXcg
cGFpbnRzIGFmdGVyIGJlaW5nIHJlc2l6ZWQgd2hpbGUgaGlkZGVuCisKKyAgICAgICAgUmV2aWV3
ZWQgYnkgTk9CT0RZIChPT1BTISkuCisKKyAgICAgICAgKiBUZXN0V2ViS2l0QVBJL1Rlc3RzL1dl
YktpdDIvd2luL1Jlc2l6ZVZpZXdXaGlsZUhpZGRlbi5jcHA6CisgICAgICAgIChUZXN0V2ViS2l0
QVBJOjpURVNUKTogRm9yY2UgdGhlIFdLVmlldyB0byBwYWludCBhZnRlciByZXNpemluZyBpdCB3
aGlsZSBoaWRkZW4uCisKKzIwMTEtMDItMDkgIEFkYW0gUm9iZW4gIDxhcm9iZW5AYXBwbGUuY29t
PgorCiAgICAgICAgIEFkZCBhIHRlc3QgdGhhdCByZXNpemVzIGEgV0tWaWV3IHdoaWxlIGl0J3Mg
aGlkZGVuCiAKICAgICAgICAgVGVzdCBmb3IgPGh0dHA6Ly93ZWJraXQub3JnL2IvNTQxNDE+IDxy
ZGFyOi8vcHJvYmxlbS84OTc5MTk1PiBXS1ZpZXcgZHJhd3MgYXQgd3Jvbmcgc2l6ZSwKZGlmZiAt
LWdpdCBhL1Rvb2xzL1Rlc3RXZWJLaXRBUEkvVGVzdHMvV2ViS2l0Mi93aW4vUmVzaXplVmlld1do
aWxlSGlkZGVuLmNwcCBiL1Rvb2xzL1Rlc3RXZWJLaXRBUEkvVGVzdHMvV2ViS2l0Mi93aW4vUmVz
aXplVmlld1doaWxlSGlkZGVuLmNwcAppbmRleCAxM2NlZTc0YjQwZDZkMDFhZDdkMWUxNDZiZTBl
ZTFmZjczYWJhODcwLi43MzEwZTZjNWE5ZmU5ZDAyOTMxMmRlOWQwMzM1YTY0OTllMmVkM2U2IDEw
MDY0NAotLS0gYS9Ub29scy9UZXN0V2ViS2l0QVBJL1Rlc3RzL1dlYktpdDIvd2luL1Jlc2l6ZVZp
ZXdXaGlsZUhpZGRlbi5jcHAKKysrIGIvVG9vbHMvVGVzdFdlYktpdEFQSS9UZXN0cy9XZWJLaXQy
L3dpbi9SZXNpemVWaWV3V2hpbGVIaWRkZW4uY3BwCkBAIC0xMTIsNiArMTEyLDkgQEAgVEVTVChX
ZWJLaXQyLCBSZXNpemVWaWV3V2hpbGVIaWRkZW4pCiAgICAgd2ViVmlldy5yZXNpemVUbyhvcmln
aW5hbFJlY3QucmlnaHQgLSBvcmlnaW5hbFJlY3QubGVmdCwgb3JpZ2luYWxSZWN0LmJvdHRvbSAt
IG9yaWdpbmFsUmVjdC50b3ApOwogICAgIDo6U2hvd1dpbmRvdyh3aW5kb3csIFNXX1NIT1cpOwog
CisgICAgLy8gRm9yY2UgdGhlIFdLVmlldyB0byBwYWludCB0byB0cnkgdG8gdHJpZ2dlciA8aHR0
cDovL3dlYmtpdC5vcmcvYi81NDE0Mj4uCisgICAgOjpTZW5kTWVzc2FnZSh3aW5kb3csIFdNX1BB
SU5ULCAwLCAwKTsKKwogICAgIC8vIEluIERlYnVnIGJ1aWxkcyB3aXRob3V0IHRoZSBmaXggZm9y
IDxodHRwOi8vd2Via2l0Lm9yZy9iLzU0MTQxPiwgdGhlIHdlYiBwcm9jZXNzIHdpbGwgYXNzZXJ0
CiAgICAgLy8gYXQgdGhpcyBwb2ludC4KICAgICAvLyBGSVhNRTogSXQgd291bGQgYmUgZ29vZCB0
byBoYXZlIGEgd2F5IHRvIGNoZWNrIHRoYXQgb3VyIGJlaGF2aW9yIGlzIGNvcnJlY3QgaW4gUmVs
ZWFzZQo=
</data>
<flag name="review"
          id="73485"
          type_id="1"
          status="+"
          setter="darin"
    />
          </attachment>
      

    </bug>

</bugzilla>