WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED FIXED
64134
Move innerTextElement() from RenderTextControl to HTMLTextFormControlHTMLTextFormControlElement
https://bugs.webkit.org/show_bug.cgi?id=64134
Summary
Move innerTextElement() from RenderTextControl to HTMLTextFormControlHTMLText...
Ryosuke Niwa
Reported
2011-07-07 16:52:18 PDT
Now that shadow DOM no longer hangs off of renderer, it seems more natural for HTMLTextFormControl instead of RenderTextControl to have inner*Element().
Attachments
Patch
(14.12 KB, patch)
2011-07-08 15:07 PDT
,
Ryosuke Niwa
tkent
: review+
tkent
: commit-queue-
Details
Formatted Diff
Diff
View All
Add attachment
proposed patch, testcase, etc.
Ryosuke Niwa
Comment 1
2011-07-08 15:07:14 PDT
Created
attachment 100167
[details]
Patch
Ryosuke Niwa
Comment 2
2011-07-08 15:08:57 PDT
Comment on
attachment 100167
[details]
Patch View in context:
https://bugs.webkit.org/attachment.cgi?id=100167&action=review
> Source/WebCore/ChangeLog:14 > + shadow DOM for input and textarea elements clear. It accessing shadow DOM of input and textarea elements > + are now less error-prone because it doesn't depend on the lifetime of the render tree.
Oops, typo. It should read "Accessing the shadow DOM of input and textarea elements are now less error prone because it no longer depends on the lifetime of the render tree".
Kent Tamura
Comment 3
2011-07-08 16:44:12 PDT
Comment on
attachment 100167
[details]
Patch View in context:
https://bugs.webkit.org/attachment.cgi?id=100167&action=review
> Source/WebCore/rendering/RenderTextControl.h:56 > + HTMLElement* innerTextElement() const;
Let's add a comment that this function should not be public. Otherwise, one who doesn't know the intention of this patch might make it public and revert the AccessibilityRenderObject.cpp change.
Ryosuke Niwa
Comment 4
2011-07-08 16:47:44 PDT
Comment on
attachment 100167
[details]
Patch View in context:
https://bugs.webkit.org/attachment.cgi?id=100167&action=review
>> Source/WebCore/rendering/RenderTextControl.h:56 >> + HTMLElement* innerTextElement() const; > > Let's add a comment that this function should not be public. > Otherwise, one who doesn't know the intention of this patch might make it public and revert the AccessibilityRenderObject.cpp change.
That makes sense.
Ryosuke Niwa
Comment 5
2011-07-08 16:48:20 PDT
Now that
r90662
has been reverted, I'd have to reland the patch for the
bug 64133
first.
Ryosuke Niwa
Comment 6
2011-07-11 12:06:40 PDT
Committed
r90775
: <
http://trac.webkit.org/changeset/90775
>
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