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

Unified Diff: third_party/WebKit/Source/core/css/cssom/ComputedStylePropertyMap.cpp

Issue 2812743003: Rename cleanup in comments in css/ directory. (Closed)
Patch Set: Created 3 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/css/cssom/ComputedStylePropertyMap.cpp
diff --git a/third_party/WebKit/Source/core/css/cssom/ComputedStylePropertyMap.cpp b/third_party/WebKit/Source/core/css/cssom/ComputedStylePropertyMap.cpp
index 290561e1002796cb159f88657cd3966f05da6ddc..f05d4328994b14d1a43780540d79c29be463a103 100644
--- a/third_party/WebKit/Source/core/css/cssom/ComputedStylePropertyMap.cpp
+++ b/third_party/WebKit/Source/core/css/cssom/ComputedStylePropertyMap.cpp
@@ -50,7 +50,7 @@ Node* ComputedStylePropertyMap::GetNode() const {
}
if (node_->IsElementNode()) {
// Seems to only support before, after, backdrop, first-letter. See
- // PseudoElementData::pseudoElement.
+ // PseudoElementData::GetPseudoElement.
if (PseudoElement* element =
(ToElement(node_))->GetPseudoElement(pseudo_id_)) {
return element;
@@ -59,7 +59,7 @@ Node* ComputedStylePropertyMap::GetNode() const {
return nullptr;
}
-// ComputedStylePropertyMap::getAllInternal/get should return computed styles
+// ComputedStylePropertyMap::GetAllInternal/Get should return computed styles
// (as opposed to resolved styles a la getComputedStyle()).
//
// Property values are read from an up-to-date ComputedStyle and converted into
@@ -83,9 +83,10 @@ CSSStyleValueVector ComputedStylePropertyMap::GetAllInternal(
return style_value_vector;
}
// I have copied this from
- // CSSComputedStyleDeclaration::computeComputedStyle(). I don't know if there
- // is any use in passing m_pseudoId if node is not already a PseudoElement,
- // but passing pseudo_Id when it IS already a PseudoElement leads to disaster.
+ // CSSComputedStyleDeclaration::ComputeComputedStyle(). I don't know if there
+ // is any use in passing pseudo_id_ if node is not already a PseudoElement,
+ // but passing pseudo_id_ when it IS already a PseudoElement leads to
+ // disaster.
const ComputedStyle* style = node->EnsureComputedStyle(
node->IsPseudoElement() ? kPseudoIdNone : pseudo_id_);
node = this->GetNode();

Powered by Google App Engine
This is Rietveld 408576698