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

Issues Search
    My Issues | Starred     Open | Closed | All

Unified Diff: components/font_service/public/cpp/font_service_thread.cc

Issue 2339413004: Allow Mojo structs as map keys (Closed)
Patch Set: Address all review comments Created 4 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 | mojo/android/javatests/src/org/chromium/mojo/bindings/ValidationTest.java » ('j') | mojo/public/cpp/bindings/lib/wtf_hash_util.h » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: components/font_service/public/cpp/font_service_thread.cc
diff --git a/components/font_service/public/cpp/font_service_thread.cc b/components/font_service/public/cpp/font_service_thread.cc
index 5d9645acea19f90b3da3103fc319a5e6323a2384..81f8394eb81abb628979327b03b019d5e006059b 100644
--- a/components/font_service/public/cpp/font_service_thread.cc
+++ b/components/font_service/public/cpp/font_service_thread.cc
@@ -124,7 +124,7 @@ void FontServiceThread::OnMatchFamilyNameComplete(
DCHECK_EQ(GetThreadId(), base::PlatformThread::CurrentId());
pending_waitable_events_.erase(done_event);
- *out_valid = font_identity;
+ *out_valid = !font_identity.is_null();
if (font_identity) {
out_font_identity->fID = font_identity->id;
out_font_identity->fTTCIndex = font_identity->ttc_index;
« no previous file with comments | « no previous file | mojo/android/javatests/src/org/chromium/mojo/bindings/ValidationTest.java » ('j') | mojo/public/cpp/bindings/lib/wtf_hash_util.h » ('J')

Powered by Google App Engine
This is Rietveld 408576698