Title: [159186] trunk/Source/WebCore
- Revision
- 159186
- Author
- [email protected]
- Date
- 2013-11-13 03:46:51 -0800 (Wed, 13 Nov 2013)
Log Message
RenderTableSection: Cell structures don't need allocation padding.
<https://webkit.org/b/124263>
The row structure data gets shrunk-to-fit once we get to layout,
but per-row cell structures get no such luxury. Fortuntely we know
ahead of time how many cells a row needs to accomodate, so we can
just use Vector::resizeToFit() instead of Vector::grow().
1.25 MB progression on HTML5-8266 locally.
Reviewed by Antti Koivisto.
Modified Paths
Diff
Modified: trunk/Source/WebCore/ChangeLog (159185 => 159186)
--- trunk/Source/WebCore/ChangeLog 2013-11-13 11:45:47 UTC (rev 159185)
+++ trunk/Source/WebCore/ChangeLog 2013-11-13 11:46:51 UTC (rev 159186)
@@ -1,5 +1,19 @@
2013-11-13 Andreas Kling <[email protected]>
+ RenderTableSection: Cell structures don't need allocation padding.
+ <https://webkit.org/b/124263>
+
+ The row structure data gets shrunk-to-fit once we get to layout,
+ but per-row cell structures get no such luxury. Fortuntely we know
+ ahead of time how many cells a row needs to accomodate, so we can
+ just use Vector::resizeToFit() instead of Vector::grow().
+
+ 1.25 MB progression on HTML5-8266 locally.
+
+ Reviewed by Antti Koivisto.
+
+2013-11-13 Andreas Kling <[email protected]>
+
Avoid unnecessarily padding the FontDescription families vector.
<https://webkit.org/b/124267>
Modified: trunk/Source/WebCore/rendering/RenderTableSection.cpp (159185 => 159186)
--- trunk/Source/WebCore/rendering/RenderTableSection.cpp 2013-11-13 11:45:47 UTC (rev 159185)
+++ trunk/Source/WebCore/rendering/RenderTableSection.cpp 2013-11-13 11:46:51 UTC (rev 159186)
@@ -204,7 +204,7 @@
unsigned effectiveColumnCount = std::max(1u, table()->numEffCols());
for (unsigned row = oldSize; row < m_grid.size(); ++row)
- m_grid[row].row.grow(effectiveColumnCount);
+ m_grid[row].row.resizeToFit(effectiveColumnCount);
}
void RenderTableSection::addCell(RenderTableCell* cell, RenderTableRow* row)
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes
