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

Unified Diff: Source/platform/fonts/opentype/OpenTypeSanitizer.h

Issue 954953002: Adding GetTableAction method to Blink to handle color table tags (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: Updating Test Expectation for Virtual test Created 5 years, 9 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: Source/platform/fonts/opentype/OpenTypeSanitizer.h
diff --git a/Source/platform/fonts/opentype/OpenTypeSanitizer.h b/Source/platform/fonts/opentype/OpenTypeSanitizer.h
index 93d7eecc12c96e28f91766b30a93adf40c4dbf2f..460dd0262ee3a89e3ce70e8ffb96b7c2b3698e91 100644
--- a/Source/platform/fonts/opentype/OpenTypeSanitizer.h
+++ b/Source/platform/fonts/opentype/OpenTypeSanitizer.h
@@ -31,6 +31,7 @@
#ifndef OpenTypeSanitizer_h
#define OpenTypeSanitizer_h
+#include "opentype-sanitiser.h"
#include "wtf/Forward.h"
namespace blink {
@@ -52,6 +53,33 @@ private:
SharedBuffer* const m_buffer;
};
+class BlinkOTSContext: public ots::OTSContext {
+public:
+ // TODO: Implement "Message" to support user friendly messages
+ virtual ots::TableAction GetTableAction(uint32_t tag)
+ {
+#define TABLE_TAG(c1, c2, c3, c4) ((uint32_t)((((uint8_t)(c1)) << 24) | (((uint8_t)(c2)) << 16) | (((uint8_t)(c3)) << 8) | ((uint8_t)(c4))))
+
+ const uint32_t cbdtTag = TABLE_TAG('C', 'B', 'D', 'T');
+ const uint32_t cblcTag = TABLE_TAG('C', 'B', 'L', 'C');
+ const uint32_t colrTag = TABLE_TAG('C', 'O', 'L', 'R');
+ const uint32_t cpalTag = TABLE_TAG('C', 'P', 'A', 'L');
+
+ switch (tag) {
+ // Google Color Emoji Tables
+ case cbdtTag:
+ case cblcTag:
+ // Windows Color Emoji Tables
+ case colrTag:
+ case cpalTag:
+ return ots::TABLE_ACTION_PASSTHRU;
+ default:
+ return ots::TABLE_ACTION_DEFAULT;
+ }
+#undef TABLE_TAG
+ }
+};
+
} // namespace blink
#endif // OpenTypeSanitizer_h
« no previous file with comments | « LayoutTests/third_party/FruityGirl/README.chromium ('k') | Source/platform/fonts/opentype/OpenTypeSanitizer.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698