Make time_to_first_suspension's default 1800sec

chrome/browser/memory/tab_manager_unittest.cc

 // Copyright (c) 2012 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.
 
 #include "chrome/browser/memory/tab_manager.h"
 
 #include <algorithm>
 #include <map>
 #include <vector>
 
@@ skipping 663 matching lines @@
 
 TEST_F(TabManagerTest, NextPurgeAndSuspendState) {
   TabManager tab_manager;
   TabStripDummyDelegate delegate;
   TabStripModel tabstrip(&delegate, profile());
   tabstrip.AddObserver(&tab_manager);
 
   WebContents* test_contents = CreateWebContents();
   tabstrip.AppendWebContents(test_contents, false);
 
-  base::TimeDelta threshold = base::TimeDelta::FromSeconds(180);
+  // Use default time-to-first-suspension seconds defined in TabManager.
+  base::TimeDelta threshold = base::TimeDelta::FromSeconds(
+      TabManager::kDefaultTimeToFirstPurgeInSeconds);
   base::SimpleTestTickClock test_clock;
 
   tab_manager.GetWebContentsData(test_contents)
       ->SetPurgeAndSuspendState(TabManager::RUNNING);
   tab_manager.GetWebContentsData(test_contents)
       ->SetLastPurgeAndSuspendModifiedTimeForTesting(test_clock.NowTicks());
 
-  test_clock.Advance(base::TimeDelta::FromSeconds(180));
+  // Wait 30 minutes. Should be still RUNNING.

[Wez, 2017/02/22 01:25:10]

nit: Comment style is to prefer complete sentences, e.g. "Wait 30 minutes and
verify that the tab is still RUNNING." and then "Wait another second and verify
that it is now SUSPENDED."

[tasak, 2017/02/22 04:18:46]

Done.

+  test_clock.Advance(base::TimeDelta::FromMinutes(30));
   EXPECT_EQ(TabManager::RUNNING,
             tab_manager.GetNextPurgeAndSuspendState(
                 test_contents, test_clock.NowTicks(), threshold));
 
+  // > 30 minutes. Should be still SUSPENDED.

[Wez, 2017/02/22 01:25:10]

Do you mean "Should _now_ be SUSPENDED."?

[tasak, 2017/02/22 04:18:46]

Yes, thanks.
Done.

   test_clock.Advance(base::TimeDelta::FromSeconds(1));
   EXPECT_EQ(TabManager::SUSPENDED,
             tab_manager.GetNextPurgeAndSuspendState(
                 test_contents, test_clock.NowTicks(), threshold));
 
   tab_manager.GetWebContentsData(test_contents)
       ->SetPurgeAndSuspendState(TabManager::SUSPENDED);
   tab_manager.GetWebContentsData(test_contents)
       ->SetLastPurgeAndSuspendModifiedTimeForTesting(test_clock.NowTicks());
 
@@ skipping 63 matching lines @@
   tabstrip.ActivateTabAt(0, true /* user_gesture */);
   EXPECT_EQ(TabManager::RUNNING,
             tab_manager.GetWebContentsData(tab1)->GetPurgeAndSuspendState());
 
   // Clean up the tabstrip.
   tabstrip.CloseAllTabs();
   EXPECT_TRUE(tabstrip.empty());
 }
 
 }  // namespace memory