WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED FIXED
25472
Replace all uses of m_offset with Position::deprecatedEditingOffset()
https://bugs.webkit.org/show_bug.cgi?id=25472
Summary
Replace all uses of m_offset with Position::deprecatedEditingOffset()
Eric Seidel (no email)
Reported
2009-04-29 17:13:21 PDT
Replace all uses of m_offset with Position::deprecatedEditingOffset() Simple search replace, per Darin's suggestion. See attached patch.
Attachments
Make m_offset private and change all callers to use deprecatedEditingOffset
(70.03 KB, patch)
2009-04-29 17:24 PDT
,
Eric Seidel (no email)
darin
: review+
Details
Formatted Diff
Diff
View All
Add attachment
proposed patch, testcase, etc.
Eric Seidel (no email)
Comment 1
2009-04-29 17:24:59 PDT
Created
attachment 29900
[details]
Make m_offset private and change all callers to use deprecatedEditingOffset 31 files changed, 278 insertions(+), 150 deletions(-)
Eric Seidel (no email)
Comment 2
2009-04-29 17:25:46 PDT
I will follow up this patch with a patch which replaces various deprecatedEditingOffset() calls with calls to offsetInContainer() or other functions which don't use the offset directly.
Eric Seidel (no email)
Comment 3
2009-04-29 17:26:48 PDT
This patch is large, but it's *all* search replace, except for the one change in InsertParagraphSeparatorCommand::doApply which I highlight in the changelog.
Eric Seidel (no email)
Comment 4
2009-04-29 18:10:08 PDT
Committing to
http://svn.webkit.org/repository/webkit/trunk
... M WebCore/ChangeLog M WebCore/dom/Position.cpp M WebCore/dom/Position.h M WebCore/dom/PositionIterator.h M WebCore/dom/Range.cpp M WebCore/editing/ApplyStyleCommand.cpp M WebCore/editing/BreakBlockquoteCommand.cpp M WebCore/editing/CompositeEditCommand.cpp M WebCore/editing/DeleteSelectionCommand.cpp M WebCore/editing/Editor.cpp M WebCore/editing/InsertLineBreakCommand.cpp M WebCore/editing/InsertParagraphSeparatorCommand.cpp M WebCore/editing/InsertTextCommand.cpp M WebCore/editing/MoveSelectionCommand.cpp M WebCore/editing/ReplaceSelectionCommand.cpp M WebCore/editing/SelectionController.cpp M WebCore/editing/TextIterator.cpp M WebCore/editing/TypingCommand.cpp M WebCore/editing/VisiblePosition.cpp M WebCore/editing/VisibleSelection.cpp M WebCore/editing/htmlediting.cpp M WebCore/editing/visible_units.cpp M WebCore/page/AccessibilityObject.cpp M WebCore/page/AccessibilityRenderObject.cpp M WebCore/page/DOMSelection.cpp M WebCore/page/EventHandler.cpp M WebCore/page/Frame.cpp M WebCore/page/mac/AccessibilityObjectWrapper.mm M WebCore/rendering/RenderTextControl.cpp M WebCore/rendering/RenderTreeAsText.cpp M WebKit/mac/WebView/WebFrame.mm Committed
r43035
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