Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1021)

Unified Diff: Source/core/html/HTMLFormControlElement.cpp

Issue 977113003: Rename renderer() to layoutObject(). (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Rebase to master Created 5 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: Source/core/html/HTMLFormControlElement.cpp
diff --git a/Source/core/html/HTMLFormControlElement.cpp b/Source/core/html/HTMLFormControlElement.cpp
index 499ee64bb9ddf506a0e2f4539e67a9d317c90692..c96d0bf25dd01e392a25d1241e5ee20be1ddffd0 100644
--- a/Source/core/html/HTMLFormControlElement.cpp
+++ b/Source/core/html/HTMLFormControlElement.cpp
@@ -159,8 +159,8 @@ void HTMLFormControlElement::parseAttribute(const QualifiedName& name, const Ato
if (wasReadOnly != m_isReadOnly) {
setNeedsWillValidateCheck();
setNeedsStyleRecalc(SubtreeStyleChange, StyleChangeReasonForTracing::fromAttribute(name));
- if (renderer() && renderer()->style()->hasAppearance())
- LayoutTheme::theme().stateChanged(renderer(), ReadOnlyControlState);
+ if (layoutObject() && layoutObject()->style()->hasAppearance())
+ LayoutTheme::theme().stateChanged(layoutObject(), ReadOnlyControlState);
}
} else if (name == requiredAttr) {
bool wasRequired = m_isRequired;
@@ -180,8 +180,8 @@ void HTMLFormControlElement::disabledAttributeChanged()
setNeedsWillValidateCheck();
pseudoStateChanged(CSSSelector::PseudoDisabled);
pseudoStateChanged(CSSSelector::PseudoEnabled);
- if (renderer() && renderer()->style()->hasAppearance())
- LayoutTheme::theme().stateChanged(renderer(), EnabledControlState);
+ if (layoutObject() && layoutObject()->style()->hasAppearance())
+ LayoutTheme::theme().stateChanged(layoutObject(), EnabledControlState);
if (isDisabledFormControl() && treeScope().adjustedFocusedElement() == this) {
// We might want to call blur(), but it's dangerous to dispatch events
// here.
@@ -232,13 +232,13 @@ void HTMLFormControlElement::attach(const AttachContext& context)
{
HTMLElement::attach(context);
- if (!renderer())
+ if (!layoutObject())
return;
// The call to updateFromElement() needs to go after the call through
// to the base class's attach() because that can sometimes do a close
// on the renderer.
- renderer()->updateFromElement();
+ layoutObject()->updateFromElement();
// FIXME: Autofocus handling should be moved to insertedInto according to
// the standard.
@@ -360,7 +360,7 @@ String HTMLFormControlElement::resultForDialogSubmit()
void HTMLFormControlElement::didRecalcStyle(StyleRecalcChange)
{
- if (LayoutObject* renderer = this->renderer())
+ if (LayoutObject* renderer = this->layoutObject())
renderer->updateFromElement();
}
@@ -399,8 +399,8 @@ void HTMLFormControlElement::willCallDefaultEventHandler(const Event& event)
if (!event.isKeyboardEvent() || event.type() != EventTypeNames::keydown)
return;
m_wasFocusedByMouse = false;
- if (renderer())
- renderer()->setShouldDoFullPaintInvalidation();
+ if (layoutObject())
+ layoutObject()->setShouldDoFullPaintInvalidation();
}
short HTMLFormControlElement::tabIndex() const
@@ -456,7 +456,7 @@ void HTMLFormControlElement::findCustomValidationMessageTextDirection(const Stri
subMessage = fastGetAttribute(titleAttr);
messageDir = determineDirectionality(message, hasStrongDirection);
if (!subMessage.isEmpty())
- subMessageDir = renderer()->style()->direction();
+ subMessageDir = layoutObject()->style()->direction();
}
void HTMLFormControlElement::updateVisibleValidationMessage()
@@ -465,7 +465,7 @@ void HTMLFormControlElement::updateVisibleValidationMessage()
if (!page)
return;
String message;
- if (renderer() && willValidate())
+ if (layoutObject() && willValidate())
message = validationMessage().stripWhiteSpace();
m_hasValidationMessage = true;
@@ -542,7 +542,7 @@ bool HTMLFormControlElement::reportValidity()
ASSERT(unhandledInvalidControls.size() == 1);
ASSERT(unhandledInvalidControls[0].get() == this);
// Update layout now before calling isFocusable(), which has
- // !renderer()->needsLayout() assertion.
+ // !layoutObject()->needsLayout() assertion.
document().updateLayoutIgnorePendingStylesheets();
if (isFocusable()) {
showValidationMessage();

Powered by Google App Engine
This is Rietveld 408576698