Support DirectWrite with sandbox on

base/base_paths_win.cc

Index: base/base_paths_win.cc
diff --git a/base/base_paths_win.cc b/base/base_paths_win.cc
index d4e11ae5065a866ccb3820da6f81891e547d3a5b..dcef400d1026e9f95b7c506449f17721c976ecc5 100644
--- a/base/base_paths_win.cc
+++ b/base/base_paths_win.cc
@@ -191,6 +191,18 @@ bool PathProviderWin(int key, FilePath* result) {
       cur = cur.AppendASCII("User Pinned");
       cur = cur.AppendASCII("TaskBar");
       break;
+    case base::DIR_WINDOWS_FONTS: {
+      if (win::GetVersion() < win::VERSION_VISTA)

[brettw, 2014/03/25 19:28:18]

be consistent w.r.t. base::win or just ::win. I usually fully qualify nested
namespaces for clarity, so add base:: here.

It seems like not working on XP is a significant limitation that should be
mentioned in the header. Or is it reasonable to code %WINDIR%/Fonts for XP?

[scottmg, 2014/03/25 19:45:58]

Was copy-pasta from DIR_APP_SHORTCUTS case. :/
Switched to SHGetFolderPath which works on XP. This patch will only use it on
Win7+ though.

+        return false;
+
+      base::win::ScopedCoMem<wchar_t> path_buf;
+      if (FAILED(SHGetKnownFolderPath(FOLDERID_Fonts, 0, NULL,
+                                      &path_buf)))
+        return false;
+
+      cur = FilePath(string16(path_buf));
+      break;
+    }
     default:
       return false;
   }