Bug 16939 - Repaint bug on Text selection in foreignObject
Summary: Repaint bug on Text selection in foreignObject
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: SVG (show other bugs)
Version: 528+ (Nightly build)
Hardware: All OS X 10.4
: P2 Normal
Assignee: Nikolas Zimmermann
URL: http://croczilla.com/svg/samples/fore...
Keywords:
: 18889 (view as bug list)
Depends on: 25532
Blocks:
  Show dependency treegraph
 
Reported: 2008-01-19 02:31 PST by Oliver Hunt
Modified: 2010-02-12 14:19 PST (History)
3 users (show)

See Also:


Attachments
Initial patch (472.84 KB, patch)
2010-02-12 13:39 PST, Nikolas Zimmermann
krit: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Oliver Hunt 2008-01-19 02:31:04 PST
build webkit with foriegnObject support  build-webkit --svg-foreign-object then try selecting the text in the linked url
Comment 1 Eric Seidel (no email) 2009-05-03 15:42:58 PDT
This is not fixed in Safari 4, but might be fixed in tip of tree.
Comment 2 Eric Seidel (no email) 2009-05-03 16:59:02 PDT
Turns out I have a fix for this locally.  Now I just need to find a way to turn this into a test case.
Comment 3 Dirk Schulze 2009-12-30 12:26:35 PST
No problems here. It's not possible to start the slection on top of the shape, but works around it. The repaint works. Can we close this bug?
Comment 5 Dirk Schulze 2009-12-30 13:25:20 PST
*** Bug 18889 has been marked as a duplicate of this bug. ***
Comment 6 Nikolas Zimmermann 2010-01-02 13:28:36 PST
I still see a bug, open the linked URL, and start selecting the text from top left, then move to the bottom right, triggers following assertion:
ASSERTION FAILED: useTransforms
(/Users/nikolaszimmermann/Coding/WebKit/WebCore/rendering/RenderSVGRoot.cpp:273 virtual void WebCore::RenderSVGRoot::mapLocalToContainer(WebCore::RenderBoxModelObject*, bool, bool, WebCore::TransformState&) const)
Comment 7 Nikolas Zimmermann 2010-02-12 13:39:50 PST
Created attachment 48661 [details]
Initial patch
Comment 8 Dirk Schulze 2010-02-12 14:09:42 PST
Comment on attachment 48661 [details]
Initial patch

r=me
Comment 9 Nikolas Zimmermann 2010-02-12 14:19:20 PST
Landed in r54735.