[Webkit-unassigned] [Bug 104498] [EFL][WK2] Enable TEXT_AUTOSIZING

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=104498





--- Comment #18 from Jaehun Lim <ljaehun.lim at samsung.com>  2013-08-12 17:34:56 PST ---
(In reply to comment #17)
> (From update of attachment 208534 [details])
> View in context: https://bugs.webkit.org/attachment.cgi?id=208534&action=review
> 
> > Source/WebCore/page/Settings.cpp:131
> >      , m_textAutosizingWindowSizeOverride(320, 480)
> 
> Why are we keeping this line if we are removing HACK_FORCE_TEXT_AUTOSIZING_ON_DESKTOP?

"windows size" is used to adjust the font size. If it is not override, "visible contents size" is used instead of it. http://trac.webkit.org/browser/trunk/Source/WebCore/rendering/TextAutosizer.cpp#L116

I think this line is necessary to activate TextAutosizing for the big screen. But I don't like it.

Johe Mellor filed the related bug, https://bugs.webkit.org/show_bug.cgi?id=94371

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.