Title: [171950] trunk/Source/WebCore
- Revision
- 171950
- Author
- mmaxfi...@apple.com
- Date
- 2014-08-01 15:33:50 -0700 (Fri, 01 Aug 2014)
Log Message
SVGGlyph wastes space due to padding
https://bugs.webkit.org/show_bug.cgi?id=135520
Reviewed by Dean Jackson.
Rearrange members in SVGGlyph so there is no wasted padding space.
No new tests because there is no behavior change.
* platform/graphics/SVGGlyph.h:
(WebCore::SVGGlyph::SVGGlyph):
Modified Paths
Diff
Modified: trunk/Source/WebCore/ChangeLog (171949 => 171950)
--- trunk/Source/WebCore/ChangeLog 2014-08-01 20:58:19 UTC (rev 171949)
+++ trunk/Source/WebCore/ChangeLog 2014-08-01 22:33:50 UTC (rev 171950)
@@ -1,3 +1,17 @@
+2014-08-01 Myles C. Maxfield <mmaxfi...@apple.com>
+
+ SVGGlyph wastes space due to padding
+ https://bugs.webkit.org/show_bug.cgi?id=135520
+
+ Reviewed by Dean Jackson.
+
+ Rearrange members in SVGGlyph so there is no wasted padding space.
+
+ No new tests because there is no behavior change.
+
+ * platform/graphics/SVGGlyph.h:
+ (WebCore::SVGGlyph::SVGGlyph):
+
2014-07-31 Dean Jackson <d...@apple.com>
[Media iOS] Ensure there is a nice default fallback for missing wireless target names
Modified: trunk/Source/WebCore/platform/graphics/SVGGlyph.h (171949 => 171950)
--- trunk/Source/WebCore/platform/graphics/SVGGlyph.h 2014-08-01 20:58:19 UTC (rev 171949)
+++ trunk/Source/WebCore/platform/graphics/SVGGlyph.h 2014-08-01 22:33:50 UTC (rev 171950)
@@ -51,16 +51,16 @@
};
SVGGlyph()
- : isPartOfLigature(false)
- , orientation(Both)
- , arabicForm(None)
+ : unicodeStringLength(0)
, priority(0)
- , tableEntry(0)
- , unicodeStringLength(0)
, horizontalAdvanceX(0)
, verticalOriginX(0)
, verticalOriginY(0)
, verticalAdvanceY(0)
+ , tableEntry(0)
+ , isPartOfLigature(false)
+ , orientation(Both)
+ , arabicForm(None)
{
}
@@ -86,22 +86,25 @@
&& languages == other.languages;
}
- bool isPartOfLigature : 1;
+ Vector<String> languages;
- unsigned orientation : 2; // Orientation
- unsigned arabicForm : 3; // ArabicForm
- int priority;
- Glyph tableEntry;
size_t unicodeStringLength;
String glyphName;
+ Path pathData;
+ int priority;
+
float horizontalAdvanceX;
float verticalOriginX;
float verticalOriginY;
float verticalAdvanceY;
- Path pathData;
- Vector<String> languages;
+ Glyph tableEntry;
+
+ unsigned isPartOfLigature : 1;
+
+ unsigned orientation : 2; // Orientation
+ unsigned arabicForm : 3; // ArabicForm
};
Vector<SVGGlyph::ArabicForm> charactersWithArabicForm(const String& input, bool rtl);
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes