Bug fix: Use the last hidden time for purge + suspend

Bug fix: Use the last hidden time for purge + suspend

|TabStats::last_active| is updated when the tab is made active, and
using this as a base time point for purge + suspend is wrong. Instead,
the time point should be the time when the tab is made hidden.

This CL fixes this by replacing this usage with last_hidden, which is
newly introduced.

BUG=551995, 607077
Committed: https://crrev.com/c4f226a843d7af20981be69e545f349de5693eae
Cr-Commit-Position: refs/heads/master@{#426755}

[hajimehoshi, 2016-10-18 09:01:33]

Description was changed from

==========
Bug fix: Use the last hidden time for purge + suspend

|TabStats::last_active| is updated when the tab is made active, and
using this as a base time point for purge + suspend is wrong. Instead,
the time point should be the time when the tab is made hidden.

This CL fixes this by replacing this usage with last_hidden, which is
newly introduced.

BUG=551995, 607077
==========

to

==========
Bug fix: Use the last hidden time for purge + suspend

|TabStats::last_active| is updated when the tab is made active, and
using this as a base time point for purge + suspend is wrong. Instead,
the time point should be the time when the tab is made hidden.

This CL fixes this by replacing this usage with last_hidden, which is
newly introduced.

BUG=551995, 607077
==========

[hajimehoshi, 2016-10-18 09:03:26]

hajimehoshi@chromium.org changed reviewers:
+ avi@chromium.org, chrisha@chromium.org, tasak@google.com

[hajimehoshi, 2016-10-18 09:03:26]

PTAL

[Avi (use Gerrit), 2016-10-18 15:14:33]

lgtm

[chrisha, 2016-10-18 21:24:08]

lgtm

[hajimehoshi, 2016-10-19 03:37:38]

The CQ bit was checked by hajimehoshi@chromium.org

[commit-bot: I haz the power, 2016-10-19 03:38:40]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-10-19 03:44:39]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-10-19 03:44:40]

Try jobs failed on following builders:
  ios-device on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-device/builds...)

[hajimehoshi, 2016-10-21 06:24:40]

The CQ bit was checked by hajimehoshi@chromium.org

[hajimehoshi, 2016-10-21 06:24:41]

The patchset sent to the CQ was uploaded after l-g-t-m from
chrisha@chromium.org, avi@chromium.org
Link to the patchset: https://codereview.chromium.org/2426733004/#ps20001
(title: "(rebase)")

[commit-bot: I haz the power, 2016-10-21 06:24:54]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-10-21 09:02:22]

Description was changed from

==========
Bug fix: Use the last hidden time for purge + suspend

|TabStats::last_active| is updated when the tab is made active, and
using this as a base time point for purge + suspend is wrong. Instead,
the time point should be the time when the tab is made hidden.

This CL fixes this by replacing this usage with last_hidden, which is
newly introduced.

BUG=551995, 607077
==========

to

==========
Bug fix: Use the last hidden time for purge + suspend

|TabStats::last_active| is updated when the tab is made active, and
using this as a base time point for purge + suspend is wrong. Instead,
the time point should be the time when the tab is made hidden.

This CL fixes this by replacing this usage with last_hidden, which is
newly introduced.

BUG=551995, 607077
==========

[commit-bot: I haz the power, 2016-10-21 09:02:23]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-10-21 13:28:20]

Description was changed from

==========
Bug fix: Use the last hidden time for purge + suspend

|TabStats::last_active| is updated when the tab is made active, and
using this as a base time point for purge + suspend is wrong. Instead,
the time point should be the time when the tab is made hidden.

This CL fixes this by replacing this usage with last_hidden, which is
newly introduced.

BUG=551995, 607077
==========

to

==========
Bug fix: Use the last hidden time for purge + suspend

|TabStats::last_active| is updated when the tab is made active, and
using this as a base time point for purge + suspend is wrong. Instead,
the time point should be the time when the tab is made hidden.

This CL fixes this by replacing this usage with last_hidden, which is
newly introduced.

BUG=551995, 607077

Committed: https://crrev.com/c4f226a843d7af20981be69e545f349de5693eae
Cr-Commit-Position: refs/heads/master@{#426755}
==========

[commit-bot: I haz the power, 2016-10-21 13:28:21]

Patchset 2 (id:??) landed as
https://crrev.com/c4f226a843d7af20981be69e545f349de5693eae
Cr-Commit-Position: refs/heads/master@{#426755}