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

Unified Diff: chrome/browser/component_updater/pepper_flash_component_installer.cc

Issue 10905227: Introduce PPB_Flash_Font. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: . Created 8 years, 3 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 | « no previous file | chrome/chrome_renderer.gypi » ('j') | ppapi/api/private/ppb_flash_font.idl » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/component_updater/pepper_flash_component_installer.cc
diff --git a/chrome/browser/component_updater/pepper_flash_component_installer.cc b/chrome/browser/component_updater/pepper_flash_component_installer.cc
index ab38ea8fc316147b066119af60c992f6b99edd87..3b5edbab914b35eb4b800dbee5ea83de5214f491 100644
--- a/chrome/browser/component_updater/pepper_flash_component_installer.cc
+++ b/chrome/browser/component_updater/pepper_flash_component_installer.cc
@@ -30,6 +30,7 @@
#include "content/public/browser/browser_thread.h"
#include "content/public/browser/plugin_service.h"
#include "content/public/common/pepper_plugin_info.h"
+#include "ppapi/c/private/ppb_flash_font.h"
#include "ppapi/c/private/ppb_pdf.h"
#include "webkit/plugins/plugin_constants.h"
#include "webkit/plugins/ppapi/plugin_module.h"
@@ -125,11 +126,12 @@ bool GetPepperFlashDirectory(FilePath* latest_dir,
bool SupportsPepperInterface(const char* interface_name) {
if (webkit::ppapi::PluginModule::SupportsInterface(interface_name))
return true;
- // The PDF interface is invisible to SupportsInterface() on the browser
- // process because it is provided using PpapiInterfaceFactoryManager. We need
- // to check for that as well.
+ // These interfaces are invisible to SupportsInterface() on the browser
+ // process because they are provided using PpapiInterfaceFactoryManager.
+ // We need to check for them as well.
// TODO(cpu): make this more sane.
- return (strcmp(interface_name, PPB_PDF_INTERFACE) == 0);
+ return (strcmp(interface_name, PPB_FLASH_FONT_INTERFACE_0_1) == 0 ||
+ strcmp(interface_name, PPB_PDF_INTERFACE) == 0);
}
bool MakePepperFlashPluginInfo(const FilePath& flash_path,
« no previous file with comments | « no previous file | chrome/chrome_renderer.gypi » ('j') | ppapi/api/private/ppb_flash_font.idl » ('J')

Powered by Google App Engine
This is Rietveld 408576698