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

Side by Side Diff: Source/core/platform/graphics/harfbuzz/HarfBuzzShaper.h

Issue 44253003: Remove virtual destructors for several graphics classes (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: Created 7 years, 1 month 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 unified diff | Download patch
« no previous file with comments | « no previous file | Source/core/platform/graphics/harfbuzz/HarfBuzzShaper.cpp » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 /* 1 /*
2 * Copyright (C) 2012 Google Inc. All rights reserved. 2 * Copyright (C) 2012 Google Inc. All rights reserved.
3 * 3 *
4 * Redistribution and use in source and binary forms, with or without 4 * Redistribution and use in source and binary forms, with or without
5 * modification, are permitted provided that the following conditions are 5 * modification, are permitted provided that the following conditions are
6 * met: 6 * met:
7 * 7 *
8 * * Redistributions of source code must retain the above copyright 8 * * Redistributions of source code must retain the above copyright
9 * notice, this list of conditions and the following disclaimer. 9 * notice, this list of conditions and the following disclaimer.
10 * * Redistributions in binary form must reproduce the above 10 * * Redistributions in binary form must reproduce the above
(...skipping 36 matching lines...) Expand 10 before | Expand all | Expand 10 after
47 class SimpleFontData; 47 class SimpleFontData;
48 48
49 class HarfBuzzShaper { 49 class HarfBuzzShaper {
50 public: 50 public:
51 enum NormalizeMode { 51 enum NormalizeMode {
52 DoNotNormalizeMirrorChars, 52 DoNotNormalizeMirrorChars,
53 NormalizeMirrorChars 53 NormalizeMirrorChars
54 }; 54 };
55 55
56 HarfBuzzShaper(const Font*, const TextRun&); 56 HarfBuzzShaper(const Font*, const TextRun&);
57 virtual ~HarfBuzzShaper();
58 57
59 void setDrawRange(int from, int to); 58 void setDrawRange(int from, int to);
60 bool shape(GlyphBuffer* = 0); 59 bool shape(GlyphBuffer* = 0);
61 FloatPoint adjustStartPoint(const FloatPoint&); 60 FloatPoint adjustStartPoint(const FloatPoint&);
62 float totalWidth() { return m_totalWidth; } 61 float totalWidth() { return m_totalWidth; }
63 int offsetForPosition(float targetX); 62 int offsetForPosition(float targetX);
64 FloatRect selectionRect(const FloatPoint&, int height, int from, int to); 63 FloatRect selectionRect(const FloatPoint&, int height, int from, int to);
65 64
66 private: 65 private:
67 class HarfBuzzRun { 66 class HarfBuzzRun {
(...skipping 83 matching lines...) Expand 10 before | Expand all | Expand 10 after
151 int m_toIndex; 150 int m_toIndex;
152 151
153 float m_totalWidth; 152 float m_totalWidth;
154 153
155 friend struct CachedShapingResults; 154 friend struct CachedShapingResults;
156 }; 155 };
157 156
158 } // namespace WebCore 157 } // namespace WebCore
159 158
160 #endif // HarfBuzzShaper_h 159 #endif // HarfBuzzShaper_h
OLDNEW
« no previous file with comments | « no previous file | Source/core/platform/graphics/harfbuzz/HarfBuzzShaper.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698