Issue metadata
Sign in to add a comment
|
VT2 terminal font size is very small |
||||||||||||||||||||||
Issue descriptionGoogle Chrome 52.0.2712.0 (Official Build) dev (64-bit) Revision 0 Platform 8218.0.0 (Official Build) dev-channel link What steps will reproduce the problem? Open VT2 terminal What do you see instead? Font size is very small Issue seen only on Link.
,
Apr 20 2016
Reproduced on Samus as well.
,
Apr 21 2016
to dbehr@ who is frecon expert.
,
Apr 24 2016
Issue 605797 has been merged into this issue.
,
Apr 24 2016
Yeah, after I added starting frecon with no DRM objects scaling is initialized to 1 and then is not reinitialized when we create real DRM object and set scaling because of refcount in font module. Working on a fix.
,
Apr 26 2016
,
Apr 26 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromiumos/platform/frecon/+/a3f65714568aaadaa3122c76c1187935f0d26247 commit a3f65714568aaadaa3122c76c1187935f0d26247 Author: Dominik Behr <dbehr@chromium.org> Date: Tue Apr 26 00:42:14 2016 frecon: free font before resizing Font module creates prescaled glyphs on first init. term_resize() reinitializes font module but so far it has been only freed in term_close() so font module refcount keeps on increasing. The font needs to be freed before every term_resize() to balance font_init() in term_resize(). BUG= chromium:605324 TEST=boot Samus and switch to developer console using CTRL-ALT-F2 and\ observe font size Change-Id: I08449bdb5f57b4137599ec5ecccb8b339e26b278 Signed-off-by: Dominik Behr <dbehr@chromium.org> Reviewed-on: https://chromium-review.googlesource.com/340565 Commit-Ready: Dominik Behr <dbehr@google.com> Tested-by: Dominik Behr <dbehr@google.com> Reviewed-by: Haixia Shi <hshi@chromium.org> [modify] https://crrev.com/a3f65714568aaadaa3122c76c1187935f0d26247/term.c
,
Apr 29 2016
,
May 2 2016
Verified on ChromeOS 8270.0.0, 52.0.2721.0 |
|||||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||||
Comment 1 by abodenha@chromium.org
, Apr 20 2016