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

Unified Diff: Source/platform/graphics/TextRun.cpp

Issue 111833006: Avoiding multiple text/string parsing while creating TextRun (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@FontOptPatch1
Patch Set: Fixing review comments for rebasing files Created 7 years 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: Source/platform/graphics/TextRun.cpp
diff --git a/Source/platform/graphics/TextRun.cpp b/Source/platform/graphics/TextRun.cpp
new file mode 100755
index 0000000000000000000000000000000000000000..3b78f90ba01759d93508c1b0072064a73bfd7815
--- /dev/null
+++ b/Source/platform/graphics/TextRun.cpp
@@ -0,0 +1,72 @@
+/*
+ * Copyright (C) 2011 Apple Inc. All rights reserved.
leviw_travelin_and_unemployed 2013/12/27 04:46:32 I suspect you mean Samsung 2013 here ;)
+ *
+ * Redistribution and use in source and binary forms, with or without
+ * modification, are permitted provided that the following conditions
+ * are met:
+ * 1. Redistributions of source code must retain the above copyright
+ * notice, this list of conditions and the following disclaimer.
+ * 2. Redistributions in binary form must reproduce the above copyright
+ * notice, this list of conditions and the following disclaimer in the
+ * documentation and/or other materials provided with the distribution.
+ *
+ * THIS SOFTWARE IS PROVIDED BY APPLE INC. AND ITS CONTRIBUTORS ``AS IS''
+ * AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO,
+ * THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR
+ * PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL APPLE INC. OR ITS CONTRIBUTORS
+ * BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR
+ * CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF
+ * SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS
+ * INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN
+ * CONTRACT, STRICT LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE)
+ * ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF
+ * THE POSSIBILITY OF SUCH DAMAGE.
+ */
+
+#include "config.h"
+#include "platform/graphics/TextRun.h"
+
+namespace WebCore {
+
+struct ExpectedTextRunSize {
+ const void* pointer;
+ int integers[2];
+ float float1;
+ float float2;
+ float float3;
+ uint32_t bitfields : 10;
+ unsigned anUnsigned;
+ CodePath textCodePathVal;
+ RefPtr<TextRun::RenderingContext> renderingContext;
+};
+
+COMPILE_ASSERT(sizeof(TextRun) == sizeof(ExpectedTextRunSize), TextRun_is_not_of_expected_size);
+
+bool TextRun::s_allowsRoundingHacks = false;
+
+void TextRun::setAllowsRoundingHacks(bool allowsRoundingHacks)
+{
+ s_allowsRoundingHacks = allowsRoundingHacks;
+}
+
+void TextRun::setText(const String& string)
+{
+ m_len = string.length();
+ if (!m_len) {
+ m_data.characters8 = 0;
+ m_is8Bit = true;
+ return;
+ }
+ m_is8Bit = string.is8Bit();
+ if (m_is8Bit)
+ m_data.characters8 = string.characters8();
+ else
+ m_data.characters16 = string.characters16();
+}
+
+bool TextRun::allowsRoundingHacks()
+{
+ return s_allowsRoundingHacks;
+}
+
+}

Powered by Google App Engine
This is Rietveld 408576698