Mac: More pluming for OOP font loading

chrome/common/font_loader_mac.mm

 // Copyright (c) 2010 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/common/font_loader_mac.h"
 
 #import <Cocoa/Cocoa.h>
 
 #include "base/basictypes.h"
 #include "base/file_path.h"
 #include "base/file_util.h"
 #include "base/logging.h"
 #include "base/mac_util.h"
 #include "base/scoped_cftyperef.h"
 #include "base/sys_string_conversions.h"
 
 // static
-bool FontLoader::LoadFontIntoBuffer(const string16& font_name,
-                                    float font_point_size,
+bool FontLoader::LoadFontIntoBuffer(NSFont* font_to_encode,
                                     base::SharedMemory* font_data,
                                     uint32* font_data_size) {
   CHECK(font_data && font_data_size);
   *font_data_size = 0;
 
+  // Used only for logging.
+  std::string font_name([[font_to_encode fontName] UTF8String]);
+
   // Load appropriate NSFont.
-  NSString* font_name_ns = base::SysUTF16ToNSString(font_name);
-  NSFont* font_to_encode =
-      [NSFont fontWithName:font_name_ns size:font_point_size];
   if (!font_to_encode) {
     LOG(ERROR) << "Failed to load font " << font_name;
     return false;
   }
 
   // NSFont -> ATSFontRef.
   ATSFontRef ats_font =
       CTFontGetPlatformFont(reinterpret_cast<CTFontRef>(font_to_encode), NULL);
   if (!ats_font) {
     LOG(ERROR) << "Conversion to ATSFontRef failed for " << font_name;
@@ skipping 44 matching lines @@
   if (amt_read != font_file_size_32) {
     LOG(ERROR) << "Failed to read font data for " << font_path.value();
     return false;
   }
 
   *font_data_size = font_file_size_32;
   return true;
 }
 
 // static
-bool FontLoader::CreateCGFontFromBuffer(base::SharedMemoryHandle font_data,
-                                        uint32 font_data_size,
-                                        CGFontRef *font) {
+bool FontLoader::ATSFontContainerFromBuffer(base::SharedMemoryHandle font_data,
+                                            uint32 font_data_size,
+                                            ATSFontContainerRef* font_container)
+{
+  CHECK(font_container);
+
   using base::SharedMemory;
   DCHECK(SharedMemory::IsHandleValid(font_data));
   DCHECK_GT(font_data_size, 0U);
 
   SharedMemory shm(font_data, true);
   if (!shm.Map(font_data_size))
     return false;
 
-  ATSFontContainerRef font_container = 0;
+  // A value of 3 means the font is private and can't be seen by anyone else.
+  // This is the value used by WebKit when activating remote fonts.

[Avi (use Gerrit), 2010/06/15 13:57:48]

Nice comment; I like it.

+  const ATSFontContext kFontContextPrivate = 3;
   OSStatus err = ATSFontActivateFromMemory(shm.memory(), font_data_size,
-                    kATSFontContextLocal, kATSFontFormatUnspecified,
-                    NULL, kATSOptionFlagsDefault, &font_container );
+                    kFontContextPrivate, kATSFontFormatUnspecified, NULL,
+                    kATSOptionFlagsDefault, font_container);
   if (err != noErr || !font_container)
     return false;
 
-  // Count the number of fonts that were loaded.
-  ItemCount fontCount = 0;
-  err = ATSFontFindFromContainer(font_container, kATSOptionFlagsDefault, 0,
-             NULL, &fontCount);
-
-  if (err != noErr || fontCount < 1) {
-    ATSFontDeactivate(font_container, NULL, kATSOptionFlagsDefault);
-    return false;
-  }
-
-  // Load font from container.
-  ATSFontRef font_ref_ats = 0;
-  ATSFontFindFromContainer(font_container, kATSOptionFlagsDefault, 1,
-      &font_ref_ats, NULL);
-
-  if (!font_ref_ats) {
-    ATSFontDeactivate(font_container, NULL, kATSOptionFlagsDefault);
-    return false;
-  }
-
-  // Convert to cgFont.
-  CGFontRef font_ref_cg = CGFontCreateWithPlatformFont(&font_ref_ats);
-
-  if (!font_ref_cg) {
-    ATSFontDeactivate(font_container, NULL, kATSOptionFlagsDefault);
-    return false;
-  }
-
-  *font = font_ref_cg;
   return true;
 }