results.webkit.org: Increase default limit for LimitSlider

[Jonathan Bedard]

Display 1000 commits/results by default. Depending on the frequency of commits (and test runs), this is about 2 weeks worth of data.

[Jonathan Bedard]

Created attachment 377899 [details]
Patch

[Aakash Jain]

Comment on attachment 377899 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=377899&action=review

> Tools/ChangeLog:3
> +        results.webkit.org: Increase default size

Bug title should be more descriptive.

> Tools/ChangeLog:9
> +        (LimitSlider): Increase the default limit, make limit slider liner instead of logarithmic.

why linear now?

Typo: liner.

> Tools/resultsdbpy/resultsdbpy/view/static/js/drawer.js:170
>      const maxRange = 1000;

this variable seems unused after this change.

[Jonathan Bedard]

(In reply to Aakash Jain from comment #2)
> Comment on attachment 377899 [details]
> Patch
> 
> View in context:
> https://bugs.webkit.org/attachment.cgi?id=377899&action=review
> 
> > Tools/ChangeLog:3
> > +        results.webkit.org: Increase default size
> 
> Bug title should be more descriptive.
> 
> > Tools/ChangeLog:9
> > +        (LimitSlider): Increase the default limit, make limit slider liner instead of logarithmic.
> 
> why linear now?

The logarithmic slider was not popular with the bot watchers, they pretty universally said it was strange to use.

> 
> Typo: liner.
> 
> > Tools/resultsdbpy/resultsdbpy/view/static/js/drawer.js:170
> >      const maxRange = 1000;
> 
> this variable seems unused after this change.

[Jonathan Bedard]

Created attachment 377904 [details]
Patch

[WebKit Commit Bot]

Comment on attachment 377904 [details]
Patch

Clearing flags on attachment: 377904

Committed r249432: <https://trac.webkit.org/changeset/249432>

[WebKit Commit Bot]

All reviewed patches have been landed.  Closing bug.

[Radar WebKit Bug Importer]

<rdar://problem/54982366>

[Radar WebKit Bug Importer]

<rdar://problem/54982368>