WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED FIXED
4088
Crash: Drag element with CSS ::Before absolute positioning applied to it
https://bugs.webkit.org/show_bug.cgi?id=4088
Summary
Crash: Drag element with CSS ::Before absolute positioning applied to it
David Storey
Reported
2005-07-20 17:41:47 PDT
Steps to reproduce: go to
http://www.literarymoose.info/=/destroy/image-replacement.xhtml
drag image Expected result: nothing. Actual result (10.3.9): Safari crashes. I'm not sure if this happens with later versions. I've created a reduction and the ::before statement needs a content, position: absolute;, and width and height to crash. The text in the testcase can not be be selected, like in other browsers. Testcase follows
Attachments
reduction
(266 bytes, text/html)
2005-07-20 17:42 PDT
,
David Storey
no flags
Details
Stack trace of crash
(19.06 KB, text/plain)
2005-07-28 23:11 PDT
,
Chris Petersen
no flags
Details
patch to add null check so we don't crash
(1.77 KB, patch)
2005-10-10 08:28 PDT
,
Darin Adler
eric
: review+
Details
Formatted Diff
Diff
View All
Add attachment
proposed patch, testcase, etc.
David Storey
Comment 1
2005-07-20 17:42:57 PDT
Created
attachment 3036
[details]
reduction
Chris Petersen
Comment 2
2005-07-28 23:09:12 PDT
Confirmed with Safari 2.0 (412.2) and TOT WebKit under 10.4.2 (8C46).
Chris Petersen
Comment 3
2005-07-28 23:11:46 PDT
Created
attachment 3145
[details]
Stack trace of crash
Darin Adler
Comment 4
2005-10-10 08:25:51 PDT
It's easy to add enough nil checking that this doesn't crash. That still leaves us unable to select in this element, though.
Darin Adler
Comment 5
2005-10-10 08:28:11 PDT
Created
attachment 4290
[details]
patch to add null check so we don't crash
Eric Seidel (no email)
Comment 6
2005-10-11 01:30:25 PDT
Comment on
attachment 4290
[details]
patch to add null check so we don't crash Looks good. r=me.
Timothy Hatcher
Comment 7
2005-10-12 00:24:09 PDT
Landed.
Note
You need to
log in
before you can comment on or make changes to this bug.
Top of Page
Format For Printing
XML
Clone This Bug