Title: [199358] releases/WebKitGTK/webkit-2.12/Source/WebCore
- Revision
- 199358
- Author
- [email protected]
- Date
- 2016-04-12 09:15:19 -0700 (Tue, 12 Apr 2016)
Log Message
Merge r198044 - Fix typo in StyleTreeResolver.cpp
https://bugs.webkit.org/show_bug.cgi?id=139946
Patch by Maksim Kisilev <[email protected]> on 2016-03-10
Reviewed by Andy Estes.
The constructor for CheckForVisibilityChangeOnRecalcStyle was improperly comparing the
result of WKContentChange() (which is not a function) to WKContentVisibilityChange. I
believe the above cast would implicitly resolve to WKContentNoChange in all cases,
whether a visibility change had been observed or not.
This patch corrects this problem. I would expect that this might affect some content
visibility change behavior, but I'm not sure what the appropriate test case would be
since this was apparently found through code inspection.
* style/StyleTreeResolver.cpp:
(WebCore::Style::CheckForVisibilityChangeOnRecalcStyle::CheckForVisibilityChangeOnRecalcStyle):
Modified Paths
Diff
Modified: releases/WebKitGTK/webkit-2.12/Source/WebCore/ChangeLog (199357 => 199358)
--- releases/WebKitGTK/webkit-2.12/Source/WebCore/ChangeLog 2016-04-12 16:11:01 UTC (rev 199357)
+++ releases/WebKitGTK/webkit-2.12/Source/WebCore/ChangeLog 2016-04-12 16:15:19 UTC (rev 199358)
@@ -1,3 +1,22 @@
+2016-03-10 Maksim Kisilev <[email protected]>
+
+ Fix typo in StyleTreeResolver.cpp
+ https://bugs.webkit.org/show_bug.cgi?id=139946
+
+ Reviewed by Andy Estes.
+
+ The constructor for CheckForVisibilityChangeOnRecalcStyle was improperly comparing the
+ result of WKContentChange() (which is not a function) to WKContentVisibilityChange. I
+ believe the above cast would implicitly resolve to WKContentNoChange in all cases,
+ whether a visibility change had been observed or not.
+
+ This patch corrects this problem. I would expect that this might affect some content
+ visibility change behavior, but I'm not sure what the appropriate test case would be
+ since this was apparently found through code inspection.
+
+ * style/StyleTreeResolver.cpp:
+ (WebCore::Style::CheckForVisibilityChangeOnRecalcStyle::CheckForVisibilityChangeOnRecalcStyle):
+
2016-03-10 Jer Noble <[email protected]>
CRASH at WebCore::RenderView::updateVisibleViewportRect
Modified: releases/WebKitGTK/webkit-2.12/Source/WebCore/style/StyleTreeResolver.cpp (199357 => 199358)
--- releases/WebKitGTK/webkit-2.12/Source/WebCore/style/StyleTreeResolver.cpp 2016-04-12 16:11:01 UTC (rev 199357)
+++ releases/WebKitGTK/webkit-2.12/Source/WebCore/style/StyleTreeResolver.cpp 2016-04-12 16:15:19 UTC (rev 199358)
@@ -784,7 +784,7 @@
: m_element(element)
, m_previousDisplay(currentStyle ? currentStyle->display() : NONE)
, m_previousVisibility(currentStyle ? currentStyle->visibility() : HIDDEN)
- , m_previousImplicitVisibility(WKObservingContentChanges() && WKContentChange() != WKContentVisibilityChange ? elementImplicitVisibility(element) : VISIBLE)
+ , m_previousImplicitVisibility(WKObservingContentChanges() && WKObservedContentChange() != WKContentVisibilityChange ? elementImplicitVisibility(element) : VISIBLE)
{
}
~CheckForVisibilityChangeOnRecalcStyle()
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes
