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(bool state) { purge_state_ = state; }

[Wez, 2017/03/02 02:28:32]

Similar to below, this can be set_is_purged()

[tasak, 2017/03/02 09:22:47]

Done.

 
-  // Returns the last purge-and-suspend modified time.
-  // TODO(tasak): remove this after PurgeAndSuspend code is moved into
+  // Returns the current state of purge.
+  // TODO(tasak): remove this after the logic is moved into
   // MemoryCoordinator.
-  base::TimeTicks LastPurgeAndSuspendModifiedTime() const;
+  bool IsPurged() const { return purge_state_; }

[Wez, 2017/03/02 02:28:33]

This can be hacker_case is_purged()

[tasak, 2017/03/02 09:22:47]

Done.

 
-  // 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);
+  // Sets the time to purge after the tab is backgrounded.
+  void SetTimeToPurge(const base::TimeDelta& time_to_purge) {

[Wez, 2017/03/02 02:28:32]

These can be hacker_case setter and getter.

[tasak, 2017/03/02 09:22:47]

Done.

+    time_to_purge_ = time_to_purge;
+  }
 
-  // Returns the current state of purge-and-suspend.
-  // TODO(tasak): remove this after PurgeAndSuspend code is moved into
-  // MemoryCoordinator.
-  TabManager::PurgeAndSuspendState GetPurgeAndSuspendState() const;
+  // Returns the time to first purge after the tab is backgrounded.
+  base::TimeDelta TimeToPurge() const { return time_to_purge_; }
 
  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 purge after the tab is backgrounded.
+  base::TimeDelta time_to_purge_;
 
-  // The current state of purge-and-suspend.
-  PurgeAndSuspendState purge_and_suspend_state_;
+  // The current state of purge. True means PURGED, and false means NOT PURGED.
+  // Initially purge_state_ is false.

[Wez, 2017/03/02 02:28:32]

This isn't very clear; I think you want to say something like:

"True if the tab has been purged since it was last active, and should therefore
not be purged again."

The rest of the detail of the algorithm doesn't belong here; the only detail to
keep here would be:

"When a discarded tab is restored, we set |is_purged_=true|, since purging the
newly-created renderer doesn't make sense."

Although note that your comment "we make purge_state_ true" seems to contradict
the .cc, in which you set it to false.

[tasak, 2017/03/02 09:22:47]

Done.
I think, when a restored tab is activated, ActiveTabChanged will be invoked. So
is_purged_ will updated.
I set new_contents' is_purged_ to false in ActiveTabChanged.
Do you mean the set_is_purged(false)?

Since the tab is activated, it is ok to invoke Purge when the tab is
backgrounded, I think. Because active & foreground tabs will be never purged.

Your suggestion is that we should do for old_contents? Either is fine.

+  // false => true
+  // - A tab is kept inactive and background for more than time-to-purge time.
+  // true => false
+  // - When ActiveTabChaged, the newly activated tab's state will be false.
+  // No need to be copied by CopyState, because CopyState is used when a tab is
+  // discarded. Instead, we make purge_state_ true.
+  bool purge_state_;

[Wez, 2017/03/02 02:28:33]

Rename this is_purged_, to match the getter & setter naming.

[tasak, 2017/03/02 09:22:47]

Done.

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