diff options
author | mukai <mukai@chromium.org> | 2014-12-15 16:00:30 -0800 |
---|---|---|
committer | Commit bot <commit-bot@chromium.org> | 2014-12-16 00:01:05 +0000 |
commit | 5dad2e71ac9f6b229590e23d2ddfef99dceb11b9 (patch) | |
tree | 86ef3faa1e888dceb5907d38d1ba7f6ece95caf9 /ui/gfx/platform_font_win.h | |
parent | 5f0be344632763de0e79549527fbf65efed1fb7a (diff) | |
download | chromium_src-5dad2e71ac9f6b229590e23d2ddfef99dceb11b9.zip chromium_src-5dad2e71ac9f6b229590e23d2ddfef99dceb11b9.tar.gz chromium_src-5dad2e71ac9f6b229590e23d2ddfef99dceb11b9.tar.bz2 |
Updates subpixel positioning and hinting when DSF is changed
See the bug to see the screenshot of problematic rendering
and the result of this fix.
There are several TODOs though:
- Renderers still remember the old font params.
- Changing hinting params will change the bounding box of the
text, which means we should re-layout several views. This will
be addressed by other CLs.
- Ideally, the font render params should change based on the display
where the text is rendered.
BUG=441439
R=oshima@chromium.org, derat@chromium.org, msw@chromium.org
TEST=the new test case covers
Review URL: https://codereview.chromium.org/789583002
Cr-Commit-Position: refs/heads/master@{#308462}
Diffstat (limited to 'ui/gfx/platform_font_win.h')
-rw-r--r-- | ui/gfx/platform_font_win.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/ui/gfx/platform_font_win.h b/ui/gfx/platform_font_win.h index 757f234..264c5a7 100644 --- a/ui/gfx/platform_font_win.h +++ b/ui/gfx/platform_font_win.h @@ -66,7 +66,7 @@ class GFX_EXPORT PlatformFontWin : public PlatformFont { virtual std::string GetFontName() const override; virtual std::string GetActualFontNameForTesting() const override; virtual int GetFontSize() const override; - virtual const FontRenderParams& GetFontRenderParams() const override; + virtual const FontRenderParams& GetFontRenderParams() override; virtual NativeFont GetNativeFont() const override; // Called once during initialization if we should be retrieving font metrics |