Follow-up of bug 161610: (In reply to Simon Fraser (smfr) from comment #3) > > > Source/WebCore/dom/Element.idl:75 > > > - attribute long scrollLeft; > > > - attribute long scrollTop; > > > + attribute long scrollLeft; // FIXME: should be unrestricted double > > > + attribute long scrollTop; // FIXME: should be unrestricted double > > > > Seems relatively straightforward to fix this. > > I didn't want to make a web-facing change here in the same patch.
Created attachment 345845 [details] Patch Just some quick patch for testing purpose...
> (In reply to Simon Fraser (smfr) from comment #3) > > > > Source/WebCore/dom/Element.idl:75 > > > > - attribute long scrollLeft; > > > > - attribute long scrollTop; > > > > + attribute long scrollLeft; // FIXME: should be unrestricted double > > > > + attribute long scrollTop; // FIXME: should be unrestricted double > > > > > > Seems relatively straightforward to fix this. > > > > I didn't want to make a web-facing change here in the same patch. @Simon: Any idea how this change could be visible to the user? And hence whether we can can/should test it? I tried using values larger than the max long or non-integer values, but scrollable overflow elements/frames seem to have a limited size and to round up scroll position, so that does not seem easy.