Purge once random minutes(between 30min and 60min) after backgrounded.

chrome/browser/memory/tab_manager_web_contents_data.h

 // Copyright 2015 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef CHROME_BROWSER_MEMORY_TAB_MANAGER_WEB_CONTENTS_DATA_H_
 #define CHROME_BROWSER_MEMORY_TAB_MANAGER_WEB_CONTENTS_DATA_H_
 
 #include "base/macros.h"
 #include "base/time/time.h"
 #include "chrome/browser/memory/tab_manager.h"
@@ skipping 61 matching lines @@
   // |test_tick_clock_| for more details.
   void set_test_tick_clock(base::TickClock* test_tick_clock);
 
   // Returns the auto-discardable state of the tab.
   // See tab_manager.h for more information.
   bool IsAutoDiscardable();
 
   // Sets/clears the auto-discardable state of the tab.
   void SetAutoDiscardableState(bool state);
 
-  // Sets the current purge-and-suspend state, and update the timestamp,
-  // i.e. the last purge-and-suspend modified time.
-  // TODO(tasak): remove this after PurgeAndSuspend code is moved into
+  // Sets the current purge state.
+  // TODO(tasak): remove this after the logic is moved into
   // MemoryCoordinator.
-  void SetPurgeAndSuspendState(TabManager::PurgeAndSuspendState state);
+  void SetPurgeState(TabManager::PurgeState state);
 
-  // Returns the last purge-and-suspend modified time.
-  // TODO(tasak): remove this after PurgeAndSuspend code is moved into
+  // Sets the time to first purge after the tab is backgrounded.
+  void SetTimeToFirstPurge(const base::TimeDelta& time_to_first_purge);

[Wez, 2017/02/27 22:20:56]

since these are simple setter & getter, they can be defined inline:

void set_time_to_purge(const ...& ...) { time_to_purge_ = ...; }

[tasak, 2017/02/28 09:45:40]

Done.

+
+  // Returns the time to first purge after the tab is backgrounded.
+  base::TimeDelta TimeToFirstPurge() const;
+
+  // Returns the current state of purge.
+  // TODO(tasak): remove this after the logic is moved into
   // MemoryCoordinator.
-  base::TimeTicks LastPurgeAndSuspendModifiedTime() const;
-
-  // Sets the timestamp of the last modified time the purge-and-suspend state
-  // of the tab was changed for testing.
-  void SetLastPurgeAndSuspendModifiedTimeForTesting(base::TimeTicks timestamp);
-
-  // Returns the current state of purge-and-suspend.
-  // TODO(tasak): remove this after PurgeAndSuspend code is moved into
-  // MemoryCoordinator.
-  TabManager::PurgeAndSuspendState GetPurgeAndSuspendState() const;
+  TabManager::PurgeState GetPurgeState() const;

[Wez, 2017/02/27 22:20:56]

Why not put the setter and getter for PurgeState next to one another?

[tasak, 2017/02/28 09:45:41]

Done.

 
  private:
   // Needed to access tab_data_.
   FRIEND_TEST_ALL_PREFIXES(TabManagerWebContentsDataTest, CopyState);
 
   struct Data {
     Data();
     bool operator==(const Data& right) const;
     bool operator!=(const Data& right) const;
 
@@ skipping 23 matching lines @@
   // for more details.
   base::TimeTicks NowTicks() const;
 
   // Contains all the needed data for the tab.
   Data tab_data_;
 
   // Pointer to a test clock. If this is set, NowTicks() returns the value of
   // this test clock. Otherwise it returns the system clock's value.
   base::TickClock* test_tick_clock_;
 
-  // The last time purge-and-suspend state was modified.
-  base::TimeTicks last_purge_and_suspend_modified_time_;
+  // The time to first purge after the tab is backgrounded.
+  base::TimeDelta time_to_first_purge_;
 
-  // The current state of purge-and-suspend.
-  PurgeAndSuspendState purge_and_suspend_state_;
+  // The current state of purge.
+  PurgeState purge_state_;

[Wez, 2017/02/27 22:20:56]

Putting these fields here, rather than in tab_data_ means they won't be copied
by CopyState() - is that deliberate? If so then a comment would be good here.

[tasak, 2017/02/28 09:45:42]

Done.

 
   DISALLOW_COPY_AND_ASSIGN(WebContentsData);
 };
 
 }  // namespace memory
 
 #endif  // CHROME_BROWSER_MEMORY_TAB_MANAGER_WEB_CONTENTS_DATA_H_