• manuk's avatar
    [omnibox] Add doc provider parameters for supporting max query limit. · 86fcad92
    manuk authored
    With this CL, the doc provider will have 6 query-length related parameters:
    
    DocumentProviderMin(Max)QueryLength
      Limits for which inputs the provider will make a request to the drive backend.
      This is useful to relieving backend load.
    
    DocumentProviderMin(Max)QueryShowLength
      Limits for which inputs the provider will show doc suggestions (assuming the
      backend returned documents). These help analyze experiments by allowing
      observing the effect of changing |min(max)_query_length_| while keeping data
      populations consistent.
    
    DocumentProviderMin(Max)QueryLogLength
      Limits for which inputs the provider will log
      field_trial_triggered_(in_session_). These help analyze experiments by
      restricting data populations to avoid noise when only interested in a range of
      input lengths. E.g. experimenting with |max_query_show_length_| would affect
      only the small subset of long queries.
    
    Bug: 1015977, 864302
    Change-Id: Iacb16d8b67b4adeb20e3e73d8570d45e0174d33a
    Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1869374
    Commit-Queue: manuk hovanesian <manukh@chromium.org>
    Reviewed-by: default avatarTommy Li <tommycli@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#708816}
    86fcad92
document_provider.cc 35.8 KB