Skip to content

Commit

Permalink
[ppapi] Remove PPB_TrueTypeFont resource.
Browse files Browse the repository at this point in the history
- UMA shows this interface has no usage.

Change-Id: Ifcb68620aceab0d8a857b4a3a0c768990bf47905
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1546762
Commit-Queue: Bill Budge <bbudge@chromium.org>
Reviewed-by: Raymes Khoury <raymes@chromium.org>
Reviewed-by: Tom Sepez <tsepez@chromium.org>
Reviewed-by: Scott Violet <sky@chromium.org>
Reviewed-by: Sam Clegg <sbc@chromium.org>
Cr-Commit-Position: refs/heads/master@{#652404}
  • Loading branch information
Bill Budge authored and Commit Bot committed Apr 18, 2019
1 parent 5187432 commit a10bfe5
Show file tree
Hide file tree
Showing 59 changed files with 6 additions and 3,959 deletions.
1 change: 0 additions & 1 deletion chrome/common/ppapi_utils.cc
Original file line number Diff line number Diff line change
Expand Up @@ -21,7 +21,6 @@
#include "ppapi/c/dev/ppb_printing_dev.h"
#include "ppapi/c/dev/ppb_text_input_dev.h"
#include "ppapi/c/dev/ppb_trace_event_dev.h"
#include "ppapi/c/dev/ppb_truetype_font_dev.h"
#include "ppapi/c/dev/ppb_url_util_dev.h"
#include "ppapi/c/dev/ppb_var_deprecated.h"
#include "ppapi/c/dev/ppb_video_capture_dev.h"
Expand Down
8 changes: 0 additions & 8 deletions chrome/test/ppapi/ppapi_browsertest.cc
Original file line number Diff line number Diff line change
Expand Up @@ -2105,14 +2105,6 @@ TEST_PPAPI_NACL(MouseCursor)

TEST_PPAPI_NACL(NetworkProxy)

// TODO(crbug.com/619765): get working on CrOS build.
#if defined(OS_CHROMEOS)
#define MAYBE_TrueTypeFont DISABLED_TrueTypeFont
#else
#define MAYBE_TrueTypeFont TrueTypeFont
#endif
TEST_PPAPI_NACL(MAYBE_TrueTypeFont)

// TODO(crbug.com/602875), TODO(crbug.com/602876) Flaky on Win and CrOS.
#if defined(OS_CHROMEOS) || defined(OS_WIN)
#define MAYBE_VideoDecoder DISABLED_VideoDecoder
Expand Down
17 changes: 0 additions & 17 deletions content/browser/BUILD.gn
Original file line number Diff line number Diff line change
Expand Up @@ -2193,17 +2193,6 @@ jumbo_source_set("browser") {
"renderer_host/pepper/pepper_tcp_server_socket_message_filter.h",
"renderer_host/pepper/pepper_tcp_socket_message_filter.cc",
"renderer_host/pepper/pepper_tcp_socket_message_filter.h",
"renderer_host/pepper/pepper_truetype_font.h",
"renderer_host/pepper/pepper_truetype_font_host.cc",
"renderer_host/pepper/pepper_truetype_font_host.h",
"renderer_host/pepper/pepper_truetype_font_linux.cc",
"renderer_host/pepper/pepper_truetype_font_list.h",
"renderer_host/pepper/pepper_truetype_font_list_host.cc",
"renderer_host/pepper/pepper_truetype_font_list_host.h",
"renderer_host/pepper/pepper_truetype_font_list_mac.mm",
"renderer_host/pepper/pepper_truetype_font_list_win.cc",
"renderer_host/pepper/pepper_truetype_font_mac.mm",
"renderer_host/pepper/pepper_truetype_font_win.cc",
"renderer_host/pepper/pepper_udp_socket_message_filter.cc",
"renderer_host/pepper/pepper_udp_socket_message_filter.h",
"renderer_host/pepper/pepper_vpn_provider_message_filter_chromeos.cc",
Expand All @@ -2219,12 +2208,6 @@ jumbo_source_set("browser") {
"//ppapi/proxy:ipc",
"//ppapi/shared_impl",
]
if (use_ozone) {
sources += [ "renderer_host/pepper/pepper_truetype_font_list_ozone.cc" ]
}
if (use_pangocairo) {
sources += [ "renderer_host/pepper/pepper_truetype_font_list_pango.cc" ]
}
if (is_linux) {
deps += [ "//components/services/font:ppapi_fontconfig_matching" ]
}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -21,8 +21,6 @@
#include "content/browser/renderer_host/pepper/pepper_printing_host.h"
#include "content/browser/renderer_host/pepper/pepper_tcp_server_socket_message_filter.h"
#include "content/browser/renderer_host/pepper/pepper_tcp_socket_message_filter.h"
#include "content/browser/renderer_host/pepper/pepper_truetype_font_host.h"
#include "content/browser/renderer_host/pepper/pepper_truetype_font_list_host.h"
#include "content/browser/renderer_host/pepper/pepper_udp_socket_message_filter.h"
#include "ppapi/host/message_filter_host.h"
#include "ppapi/host/ppapi_host.h"
Expand Down Expand Up @@ -175,25 +173,6 @@ ContentBrowserPepperHostFactory::CreateResourceHost(
new PepperPrintingHost(host_->GetPpapiHost(), instance, resource,
std::move(manager)));
}
case PpapiHostMsg_TrueTypeFont_Create::ID: {
ppapi::proxy::SerializedTrueTypeFontDesc desc;
if (!ppapi::UnpackMessage<PpapiHostMsg_TrueTypeFont_Create>(message,
&desc)) {
NOTREACHED();
return std::unique_ptr<ppapi::host::ResourceHost>();
}
// Check that the family name is valid UTF-8 before passing it to the
// host OS.
if (!base::IsStringUTF8(desc.family))
return std::unique_ptr<ppapi::host::ResourceHost>();

return std::unique_ptr<ppapi::host::ResourceHost>(
new PepperTrueTypeFontHost(host_, instance, resource, desc));
}
case PpapiHostMsg_TrueTypeFontSingleton_Create::ID: {
return std::unique_ptr<ppapi::host::ResourceHost>(
new PepperTrueTypeFontListHost(host_, instance, resource));
}
#if defined(OS_CHROMEOS)
case PpapiHostMsg_VpnProvider_Create::ID: {
scoped_refptr<PepperVpnProviderMessageFilter> vpn_provider(
Expand Down
57 changes: 0 additions & 57 deletions content/browser/renderer_host/pepper/pepper_truetype_font.h

This file was deleted.

153 changes: 0 additions & 153 deletions content/browser/renderer_host/pepper/pepper_truetype_font_host.cc

This file was deleted.

72 changes: 0 additions & 72 deletions content/browser/renderer_host/pepper/pepper_truetype_font_host.h

This file was deleted.

Loading

0 comments on commit a10bfe5

Please sign in to comment.