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

Issue 2204603002: Implement support for loading font files from outside system directory (Closed)

Created:
4 years, 4 months ago by Ilya Kulshin
Modified:
4 years, 4 months ago
Reviewers:
CC:
chromium-reviews
Base URL:
https://chromium.googlesource.com/chromium/src.git@2785
Target Ref:
refs/pending/branch-heads/2785
Project:
chromium
Visibility:
Public.

Description

Implement support for loading font files from outside system directory Implement support for loading font files from outside system directory. When a font file is installed outside the system font dir, we will now have the browser give the renderer a handle to the file that it can use to access the font data. This change will bypass sandbox restrictions that normally prevents the renderer from opening fonts outside the windows fonts directory. BUG=610466 Review-Url: https://codereview.chromium.org/2153343002 Cr-Commit-Position: refs/heads/master@{#407658} (cherry picked from commit 96ce8347f39ce0eebd10de476afb08c0afb406c9) Committed: https://chromium.googlesource.com/chromium/src/+/8d50cfd5f7036ee15954b328b667e55097b1d313

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+215 lines, -66 lines) Patch
M content/browser/renderer_host/dwrite_font_proxy_message_filter_win.h View 4 chunks +12 lines, -2 lines 0 comments Download
M content/browser/renderer_host/dwrite_font_proxy_message_filter_win.cc View 11 chunks +61 lines, -23 lines 0 comments Download
M content/browser/renderer_host/dwrite_font_proxy_message_filter_win_unittest.cc View 4 chunks +31 lines, -2 lines 0 comments Download
M content/child/dwrite_font_proxy/dwrite_font_proxy_win.h View 2 chunks +3 lines, -5 lines 0 comments Download
M content/child/dwrite_font_proxy/dwrite_font_proxy_win.cc View 7 chunks +52 lines, -26 lines 0 comments Download
M content/child/dwrite_font_proxy/dwrite_font_proxy_win_unittest.cc View 1 chunk +33 lines, -0 lines 0 comments Download
M content/common/dwrite_font_proxy_messages.h View 2 chunks +6 lines, -3 lines 0 comments Download
M content/test/dwrite_font_fake_sender_win.h View 5 chunks +11 lines, -2 lines 0 comments Download
M content/test/dwrite_font_fake_sender_win.cc View 2 chunks +6 lines, -3 lines 0 comments Download

Messages

Total messages: 2 (1 generated)
Ilya Kulshin
4 years, 4 months ago (2016-08-01 22:21:45 UTC) #2
Message was sent while issue was closed.
Committed patchset #1 (id:1) manually as
8d50cfd5f7036ee15954b328b667e55097b1d313.

Powered by Google App Engine
This is Rietveld 408576698