1. 04 Oct, 2019 38 commits
  2. 03 Oct, 2019 2 commits
    • Aidan Beggs's avatar
      Removed a comment that was no longer relevant, and changed the signature · 4f60d6a9
      Aidan Beggs authored
      of GetReputationStatusWithEngagedSites to make more sense.
      
      This CL removes an extraneous comment about skipping top domains. It
      doesn't make sense to pass DomainInfo to
      GetReputationStatusWithEngagedSites, when this method can easily
      calculate that information, and its caller, GetReputationStatus, never
      uses this information outside of passing it on. This CL also includes
      fixes a few transitively included header files.
      
      Change-Id: I528d1f22aaee2b46d8758fec2dfa0e7185f3b419
      Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1838535Reviewed-by: default avatarJoe DeBlasio <jdeblasio@chromium.org>
      Commit-Queue: Joe DeBlasio <jdeblasio@chromium.org>
      Commit-Queue: Aidan Beggs <beggs@google.com>
      Cr-Commit-Position: refs/heads/master@{#702643}
      4f60d6a9
    • Charles Zhao's avatar
      tab_ranker: clean_up. · 5ed1af1d
      Charles Zhao authored
      Some of the tab discarder code are not used any more.
      
      (1) We stop recording total_tab_count and mru_index in the
          ForegroundedOrClosed event because we think it makes more sense to
          capture these in the TabMetrics event which we are already doing.
      
          Related changes include:
          (1.1) Deprecating MRUIndex and TotalTabCount in the ukm event
                ForegroundedOrClosed.
          (1.2) Removing mru_index and total_tab_count from
                ForegroundedOrClosedMetrics
          (1.3) Stopping observing WillCloseAllTabs and CloseAllTabsStopped
          (1.4) Removing TabActivityWatcher::all_closing_tabs_
      
      
      (2) LogOldestNTabFeatures() is also removed because it is unused.
      
      Bug: 1007136
      Change-Id: I25a7b9dbd471804d4680db2e95e80646ca1acb50
      Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1817871Reviewed-by: default avatarSteven Holte <holte@chromium.org>
      Reviewed-by: default avatarCharles . <charleszhao@chromium.org>
      Reviewed-by: default avatarFrançois Doray <fdoray@chromium.org>
      Commit-Queue: Charles . <charleszhao@chromium.org>
      Cr-Commit-Position: refs/heads/master@{#702642}
      5ed1af1d