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

Unified Diff: webkit/glue/plugins/pepper_font.cc

Issue 3255003: Pull new PPAPI, rename non-P0 interfaces to Dev, rename DeviceContext2D to Gr... (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: Created 10 years, 4 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
« no previous file with comments | « webkit/glue/plugins/pepper_font.h ('k') | webkit/glue/plugins/pepper_graphics_2d.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: webkit/glue/plugins/pepper_font.cc
===================================================================
--- webkit/glue/plugins/pepper_font.cc (revision 57791)
+++ webkit/glue/plugins/pepper_font.cc (working copy)
@@ -6,8 +6,8 @@
#include "base/logging.h"
#include "base/utf_string_conversions.h"
+#include "third_party/ppapi/c/dev/ppb_font_dev.h"
#include "third_party/ppapi/c/pp_rect.h"
-#include "third_party/ppapi/c/ppb_font.h"
#include "third_party/WebKit/WebKit/chromium/public/WebFont.h"
#include "third_party/WebKit/WebKit/chromium/public/WebFontDescription.h"
#include "third_party/WebKit/WebKit/chromium/public/WebRect.h"
@@ -31,7 +31,7 @@
namespace {
-bool IsPPFontDescriptionValid(const PP_FontDescription& desc) {
+bool IsPPFontDescriptionValid(const PP_FontDescription_Dev& desc) {
// Check validity of UTF-8.
if (desc.face.type != PP_VARTYPE_STRING && desc.face.type != PP_VARTYPE_VOID)
return false;
@@ -59,7 +59,7 @@
static_cast<WebFontDescription::GenericFamily>(f + 1)
// Assumes the given PP_FontDescription has been validated.
-WebFontDescription PPFontDescToWebFontDesc(const PP_FontDescription& font) {
+WebFontDescription PPFontDescToWebFontDesc(const PP_FontDescription_Dev& font) {
// Verify that the enums match so we can just static cast.
COMPILE_ASSERT(static_cast<int>(WebFontDescription::Weight100) ==
static_cast<int>(PP_FONTWEIGHT_100),
@@ -96,7 +96,7 @@
// Converts the given PP_TextRun to a WebTextRun, returning true on success.
// False means the input was invalid.
-bool PPTextRunToWebTextRun(const PP_TextRun* run, WebTextRun* output) {
+bool PPTextRunToWebTextRun(const PP_TextRun_Dev* run, WebTextRun* output) {
String* text_string = GetString(run->text);
if (!text_string)
return false;
@@ -106,7 +106,7 @@
}
PP_Resource Create(PP_Module module_id,
- const PP_FontDescription* description) {
+ const PP_FontDescription_Dev* description) {
PluginModule* module = PluginModule::FromPPModule(module_id);
if (!module)
return 0;
@@ -123,8 +123,8 @@
}
bool Describe(PP_Resource font_id,
- PP_FontDescription* description,
- PP_FontMetrics* metrics) {
+ PP_FontDescription_Dev* description,
+ PP_FontMetrics_Dev* metrics) {
scoped_refptr<Font> font(Resource::GetAs<Font>(font_id));
if (!font.get())
return false;
@@ -133,7 +133,7 @@
bool DrawTextAt(PP_Resource font_id,
PP_Resource image_data,
- const PP_TextRun* text,
+ const PP_TextRun_Dev* text,
const PP_Point* position,
uint32_t color,
const PP_Rect* clip,
@@ -145,7 +145,7 @@
image_data_is_opaque);
}
-int32_t MeasureText(PP_Resource font_id, const PP_TextRun* text) {
+int32_t MeasureText(PP_Resource font_id, const PP_TextRun_Dev* text) {
scoped_refptr<Font> font(Resource::GetAs<Font>(font_id));
if (!font.get())
return -1;
@@ -153,7 +153,7 @@
}
uint32_t CharacterOffsetForPixel(PP_Resource font_id,
- const PP_TextRun* text,
+ const PP_TextRun_Dev* text,
int32_t pixel_position) {
scoped_refptr<Font> font(Resource::GetAs<Font>(font_id));
if (!font.get())
@@ -162,7 +162,7 @@
}
int32_t PixelOffsetForCharacter(PP_Resource font_id,
- const PP_TextRun* text,
+ const PP_TextRun_Dev* text,
uint32_t char_offset) {
scoped_refptr<Font> font(Resource::GetAs<Font>(font_id));
if (!font.get())
@@ -170,7 +170,7 @@
return font->PixelOffsetForCharacter(text, char_offset);
}
-const PPB_Font ppb_font = {
+const PPB_Font_Dev ppb_font = {
&Create,
&IsFont,
&Describe,
@@ -182,7 +182,7 @@
} // namespace
-Font::Font(PluginModule* module, const PP_FontDescription& desc)
+Font::Font(PluginModule* module, const PP_FontDescription_Dev& desc)
: Resource(module) {
WebFontDescription web_font_desc = PPFontDescToWebFontDesc(desc);
font_.reset(WebFont::create(web_font_desc));
@@ -192,12 +192,12 @@
}
// static
-const PPB_Font* Font::GetInterface() {
+const PPB_Font_Dev* Font::GetInterface() {
return &ppb_font;
}
-bool Font::Describe(PP_FontDescription* description,
- PP_FontMetrics* metrics) {
+bool Font::Describe(PP_FontDescription_Dev* description,
+ PP_FontMetrics_Dev* metrics) {
if (description->face.type != PP_VARTYPE_VOID)
return false;
@@ -206,9 +206,9 @@
// While converting the other way in PPFontDescToWebFontDesc we validated
// that the enums can be casted.
description->face = StringToPPVar(UTF16ToUTF8(web_desc.family));
- description->family = static_cast<PP_FontFamily>(web_desc.genericFamily);
+ description->family = static_cast<PP_FontFamily_Dev>(web_desc.genericFamily);
description->size = static_cast<uint32_t>(web_desc.size);
- description->weight = static_cast<PP_FontWeight>(web_desc.weight);
+ description->weight = static_cast<PP_FontWeight_Dev>(web_desc.weight);
description->italic = web_desc.italic;
description->small_caps = web_desc.smallCaps;
@@ -222,7 +222,7 @@
}
bool Font::DrawTextAt(PP_Resource image_data,
- const PP_TextRun* text,
+ const PP_TextRun_Dev* text,
const PP_Point* position,
uint32_t color,
const PP_Rect* clip,
@@ -257,14 +257,14 @@
return true;
}
-int32_t Font::MeasureText(const PP_TextRun* text) {
+int32_t Font::MeasureText(const PP_TextRun_Dev* text) {
WebTextRun run;
if (!PPTextRunToWebTextRun(text, &run))
return -1;
return font_->calculateWidth(run);
}
-uint32_t Font::CharacterOffsetForPixel(const PP_TextRun* text,
+uint32_t Font::CharacterOffsetForPixel(const PP_TextRun_Dev* text,
int32_t pixel_position) {
WebTextRun run;
if (!PPTextRunToWebTextRun(text, &run))
@@ -274,7 +274,7 @@
run, static_cast<float>(pixel_position)));
}
-int32_t Font::PixelOffsetForCharacter(const PP_TextRun* text,
+int32_t Font::PixelOffsetForCharacter(const PP_TextRun_Dev* text,
uint32_t char_offset) {
WebTextRun run;
if (!PPTextRunToWebTextRun(text, &run))
« no previous file with comments | « webkit/glue/plugins/pepper_font.h ('k') | webkit/glue/plugins/pepper_graphics_2d.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698