[Webkit-unassigned] [Bug 68098] ValueAuto and ValueMediumDPI are the same for all platforms.

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Thu Sep 15 11:50:55 PDT 2011


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





--- Comment #4 from Konrad Piascik <kpiascik at yahoo.com>  2011-09-15 11:50:55 PST ---
(In reply to comment #3)
> (From update of attachment 107521 [details])
> View in context: https://bugs.webkit.org/attachment.cgi?id=107521&action=review
> 
> > Source/WebCore/dom/ViewportArguments.cpp:63
> >          args.targetDensityDpi = 120;
> >          break;
> >      case ViewportArguments::ValueAuto:
> > +        args.targetDensityDpi = autoDPI;
> > +        break;
> >      case ViewportArguments::ValueMediumDPI:
> 
> Should we really default-value it to 160, unlike the other parameters? Is it only to not update the call sites?

That's my motivation for the default value.  If it's preferred I can update the call sites.

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



More information about the webkit-unassigned mailing list