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

Unified Diff: third_party/WebKit/Source/core/layout/TextAutosizer.cpp

Issue 2749773002: Remove AUTOSIZING_DOM_DEBUG_INFO sections (Closed)
Patch Set: remove AUTOSIZING_DOM_DEBUG_INFO parts Created 3 years, 9 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
« no previous file with comments | « third_party/WebKit/Source/core/layout/TextAutosizer.h ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: third_party/WebKit/Source/core/layout/TextAutosizer.cpp
diff --git a/third_party/WebKit/Source/core/layout/TextAutosizer.cpp b/third_party/WebKit/Source/core/layout/TextAutosizer.cpp
index 692930e0abeae5e7cad103ea19a4b2fb68bce2f2..878db4d94f1e7990d242ca3101b1c3a1cb59b698 100644
--- a/third_party/WebKit/Source/core/layout/TextAutosizer.cpp
+++ b/third_party/WebKit/Source/core/layout/TextAutosizer.cpp
@@ -51,80 +51,8 @@
#include "core/page/Page.h"
#include "wtf/PtrUtil.h"
-#ifdef AUTOSIZING_DOM_DEBUG_INFO
-#include "core/dom/ExecutionContextTask.h"
-#endif
-
namespace blink {
-#ifdef AUTOSIZING_DOM_DEBUG_INFO
-class WriteDebugInfoTask : public ExecutionContextTask {
- public:
- WriteDebugInfoTask(Element* element, AtomicString value)
- : m_element(element), m_value(value) {}
-
- virtual void performTask(ExecutionContext*) {
- m_element->setAttribute("data-autosizing", m_value, ASSERT_NO_EXCEPTION);
- }
-
- private:
- Persistent<Element> m_element;
- AtomicString m_value;
-};
-
-static void writeDebugInfo(LayoutObject* layoutObject,
- const AtomicString& output) {
- Node* node = layoutObject->node();
- if (!node)
- return;
- if (node->isDocumentNode())
- node = toDocument(node)->documentElement();
- if (!node->isElementNode())
- return;
- node->document().postTask(
- BLINK_FROM_HERE,
- WTF::wrapUnique(new WriteDebugInfoTask(toElement(node), output)));
-}
-
-void TextAutosizer::writeClusterDebugInfo(Cluster* cluster) {
- String explanation = "";
- if (cluster->m_flags & SUPPRESSING) {
- explanation = "[suppressed]";
- } else if (!(cluster->m_flags & (INDEPENDENT | WIDER_OR_NARROWER))) {
- explanation = "[inherited]";
- } else if (cluster->m_supercluster) {
- explanation = "[supercluster]";
- } else if (!clusterHasEnoughTextToAutosize(cluster)) {
- explanation = "[insufficient-text]";
- } else {
- const LayoutBlock* widthProvider = clusterWidthProvider(cluster->m_root);
- if (cluster->m_hasTableAncestor &&
- cluster->m_multiplier < multiplierFromBlock(widthProvider)) {
- explanation = "[table-ancestor-limited]";
- } else {
- explanation =
- String::format("[from width %d of %s]",
- static_cast<int>(widthFromBlock(widthProvider)),
- widthProvider->debugName().utf8().data());
- }
- }
- String pageInfo = "";
- if (cluster->m_root->isLayoutView()) {
- pageInfo =
- String::format("; pageinfo: afsf %f * dsa %f * (lw %d / fw %d)",
- m_pageInfo.m_accessibilityFontScaleFactor,
- m_pageInfo.m_deviceScaleAdjustment,
- m_pageInfo.m_layoutWidth, m_pageInfo.m_frameWidth);
- }
- float multiplier =
- cluster->m_flags & SUPPRESSING ? 1.0 : cluster->m_multiplier;
- writeDebugInfo(const_cast<LayoutBlock*>(cluster->m_root),
- AtomicString(String::format("cluster: %f %s%s", multiplier,
- explanation.utf8().data(),
- pageInfo.utf8().data())));
-}
-#endif
-
static LayoutObject* parentElementLayoutObject(
const LayoutObject* layoutObject) {
// At style recalc, the layoutObject's parent may not be attached,
@@ -886,11 +814,6 @@ TextAutosizer::Cluster* TextAutosizer::maybeCreateCluster(LayoutBlock* block) {
Cluster* cluster = new Cluster(
block, flags, parentCluster,
m_fingerprintMapper.createSuperclusterIfNeeded(block, isNewEntry));
-#ifdef AUTOSIZING_DOM_DEBUG_INFO
- // Non-SUPPRESSING clusters are annotated in clusterMultiplier.
- if (flags & SUPPRESSING)
- writeClusterDebugInfo(cluster);
-#endif
return cluster;
}
@@ -941,10 +864,6 @@ float TextAutosizer::clusterMultiplier(Cluster* cluster) {
cluster->m_supercluster->m_inheritParentMultiplier = InheritMultiplier;
}
-#ifdef AUTOSIZING_DOM_DEBUG_INFO
- writeClusterDebugInfo(cluster);
-#endif
-
ASSERT(cluster->m_multiplier);
return cluster->m_multiplier;
}
« no previous file with comments | « third_party/WebKit/Source/core/layout/TextAutosizer.h ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698