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

Unified Diff: third_party/WebKit/Source/core/testing/Internals.cpp

Issue 1896793002: Rename (updateLayout/updateStyle).*.() to updateStyleAndLayout.*.() (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 8 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: third_party/WebKit/Source/core/testing/Internals.cpp
diff --git a/third_party/WebKit/Source/core/testing/Internals.cpp b/third_party/WebKit/Source/core/testing/Internals.cpp
index be9f12d15210d8861813274e286c3558c15cd091..f28d7d488a1f27414bdd0ed420cb140532281641 100644
--- a/third_party/WebKit/Source/core/testing/Internals.cpp
+++ b/third_party/WebKit/Source/core/testing/Internals.cpp
@@ -302,7 +302,7 @@ GCObservation* Internals::observeGC(ScriptValue scriptValue)
return GCObservation::create(observedValue);
}
-unsigned Internals::updateStyleAndReturnAffectedElementCount(ExceptionState& exceptionState) const
+unsigned Internals::updateStyleAndLayoutAndReturnAffectedElementCount(ExceptionState& exceptionState) const
{
Document* document = contextDocument();
if (!document) {
@@ -311,7 +311,7 @@ unsigned Internals::updateStyleAndReturnAffectedElementCount(ExceptionState& exc
}
unsigned beforeCount = document->styleEngine().styleForElementCount();
- document->updateLayoutTree();
+ document->updateStyleAndLayoutTree();
return document->styleEngine().styleForElementCount() - beforeCount;
}
@@ -821,7 +821,7 @@ ClientRect* Internals::boundingBox(Element* element)
{
ASSERT(element);
- element->document().updateLayoutIgnorePendingStylesheets();
+ element->document().updateStyleAndLayoutIgnorePendingStylesheets();
LayoutObject* layoutObject = element->layoutObject();
if (!layoutObject)
return ClientRect::create();
@@ -892,7 +892,7 @@ String Internals::markerDescriptionForNode(Node* node, const String& markerType,
void Internals::addTextMatchMarker(const Range* range, bool isActive)
{
ASSERT(range);
- range->ownerDocument().updateLayoutIgnorePendingStylesheets();
+ range->ownerDocument().updateStyleAndLayoutIgnorePendingStylesheets();
range->ownerDocument().markers().addTextMatchMarker(range, isActive);
}
@@ -934,7 +934,7 @@ String Internals::viewportAsText(Document* document, float, int availableWidth,
return String();
}
- document->updateLayoutIgnorePendingStylesheets();
+ document->updateStyleAndLayoutIgnorePendingStylesheets();
Page* page = document->page();
@@ -1043,7 +1043,7 @@ Range* Internals::rangeFromLocationAndLength(Element* scope, int rangeLocation,
ASSERT(scope);
// TextIterator depends on Layout information, make sure layout it up to date.
- scope->document().updateLayoutIgnorePendingStylesheets();
+ scope->document().updateStyleAndLayoutIgnorePendingStylesheets();
return createRange(PlainTextRange(rangeLocation, rangeLocation + rangeLength).createRange(*scope));
}
@@ -1052,7 +1052,7 @@ unsigned Internals::locationFromRange(Element* scope, const Range* range)
{
ASSERT(scope && range);
// PlainTextRange depends on Layout information, make sure layout it up to date.
- scope->document().updateLayoutIgnorePendingStylesheets();
+ scope->document().updateStyleAndLayoutIgnorePendingStylesheets();
return PlainTextRange::create(*scope, *range).start();
}
@@ -1061,7 +1061,7 @@ unsigned Internals::lengthFromRange(Element* scope, const Range* range)
{
ASSERT(scope && range);
// PlainTextRange depends on Layout information, make sure layout it up to date.
- scope->document().updateLayoutIgnorePendingStylesheets();
+ scope->document().updateStyleAndLayoutIgnorePendingStylesheets();
return PlainTextRange::create(*scope, *range).length();
}
@@ -1083,7 +1083,7 @@ DOMPoint* Internals::touchPositionAdjustedToBestClickableNode(long x, long y, lo
return 0;
}
- document->updateLayout();
+ document->updateStyleAndLayout();
IntSize radius(width / 2, height / 2);
IntPoint point(x + radius.width(), y + radius.height());
@@ -1110,7 +1110,7 @@ Node* Internals::touchNodeAdjustedToBestClickableNode(long x, long y, long width
return 0;
}
- document->updateLayout();
+ document->updateStyleAndLayout();
IntSize radius(width / 2, height / 2);
IntPoint point(x + radius.width(), y + radius.height());
@@ -1133,7 +1133,7 @@ DOMPoint* Internals::touchPositionAdjustedToBestContextMenuNode(long x, long y,
return 0;
}
- document->updateLayout();
+ document->updateStyleAndLayout();
IntSize radius(width / 2, height / 2);
IntPoint point(x + radius.width(), y + radius.height());
@@ -1160,7 +1160,7 @@ Node* Internals::touchNodeAdjustedToBestContextMenuNode(long x, long y, long wid
return 0;
}
- document->updateLayout();
+ document->updateStyleAndLayout();
IntSize radius(width / 2, height / 2);
IntPoint point(x + radius.width(), y + radius.height());
@@ -1183,7 +1183,7 @@ ClientRect* Internals::bestZoomableAreaForTouchPoint(long x, long y, long width,
return nullptr;
}
- document->updateLayout();
+ document->updateStyleAndLayout();
IntSize radius(width / 2, height / 2);
IntPoint point(x + radius.width(), y + radius.height());
@@ -1694,7 +1694,7 @@ String Internals::layerTreeAsText(Document* document, unsigned flags, ExceptionS
String Internals::elementLayerTreeAsText(Element* element, unsigned flags, ExceptionState& exceptionState) const
{
ASSERT(element);
- element->document().updateLayout();
+ element->document().updateStyleAndLayout();
LayoutObject* layoutObject = element->layoutObject();
if (!layoutObject || !layoutObject->isBox()) {
@@ -1975,7 +1975,7 @@ void Internals::stopTrackingRepaints(Document* document, ExceptionState& excepti
frameView->setTracksPaintInvalidations(false);
}
-void Internals::updateLayoutIgnorePendingStylesheetsAndRunPostLayoutTasks(Node* node, ExceptionState& exceptionState)
+void Internals::updateStyleAndLayoutIgnorePendingStylesheetsAndRunPostLayoutTasks(Node* node, ExceptionState& exceptionState)
{
Document* document;
if (!node) {
@@ -1988,7 +1988,7 @@ void Internals::updateLayoutIgnorePendingStylesheetsAndRunPostLayoutTasks(Node*
exceptionState.throwTypeError("The node provided is neither a document nor an IFrame.");
return;
}
- document->updateLayoutIgnorePendingStylesheets(Document::RunPostLayoutTasksSynchronously);
+ document->updateStyleAndLayoutIgnorePendingStylesheets(Document::RunPostLayoutTasksSynchronously);
}
void Internals::forceFullRepaint(Document* document, ExceptionState& exceptionState)
@@ -2047,7 +2047,7 @@ ClientRectList* Internals::annotatedRegions(Document* document, bool draggable,
return ClientRectList::create();
}
- document->updateLayout();
+ document->updateStyleAndLayout();
document->view()->updateDocumentAnnotatedRegions();
Vector<AnnotatedRegionValue> regions = document->annotatedRegions();

Powered by Google App Engine
This is Rietveld 408576698