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

Unified Diff: content/renderer/renderer_blink_platform_impl.cc

Issue 2849633002: Get rid of base::SharedMemory::NULLHandle(); (Closed)
Patch Set: Created 3 years, 8 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: content/renderer/renderer_blink_platform_impl.cc
diff --git a/content/renderer/renderer_blink_platform_impl.cc b/content/renderer/renderer_blink_platform_impl.cc
index b8c509a4639be4eb678d72a76e81e132d2e932b4..d16e30189eb50c044241d494f0a6c8ff5c5de989 100644
--- a/content/renderer/renderer_blink_platform_impl.cc
+++ b/content/renderer/renderer_blink_platform_impl.cc
@@ -549,8 +549,7 @@ bool RendererBlinkPlatformImpl::SandboxSupport::LoadFont(NSFont* src_font,
return false;
}
- if (font_data_size == 0 || font_data == base::SharedMemory::NULLHandle() ||
- *font_id == 0) {
+ if (font_data_size == 0 || !font_data.IsValid() || *font_id == 0) {
LOG(ERROR) << "Bad response from RenderProcessHostMsg_LoadFont() for " <<
src_font_descriptor.font_name;
*out = NULL;

Powered by Google App Engine
This is Rietveld 408576698