Replace Increment/DecrementKeepAliveCount by ScopedKeepAlives

chrome/browser/devtools/devtools_sanity_browsertest.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 <stddef.h>
 
 #include "base/bind.h"
 #include "base/cancelable_callback.h"
 #include "base/command_line.h"
 #include "base/compiler_specific.h"
@@ skipping 753 matching lines @@
     chrome::CloseWindow(browser());
     AcceptModalDialog();
     CancelModalDialog();
     cancel_browser.Wait();
   }
   // Try to exit application.
   {
     content::WindowedNotificationObserver close_observer(
         chrome::NOTIFICATION_BROWSER_CLOSED,
         content::Source<Browser>(browser()));
-    chrome::IncrementKeepAliveCount();

[sky, 2016/03/10 04:18:20]

Why is this no longer necessary?

[dgn, 2016/03/10 18:31:51]

That was added to unflake the test a long time ago.
(crbug.com/333554, 323847). I can't find which call it's supposed to pair with.
We currently didn't do anything when the count is in excess, so it might be
superfluous now. I'll investigate some more if I see some new flakes.

     chrome::CloseAllBrowsers();
     AcceptModalDialog();
     AcceptModalDialog();
     close_observer.Wait();
   }
   for (size_t i = 0; i < close_observers.size(); ++i) {
     close_observers[i]->Wait();
     delete close_observers[i];
   }
 }
@@ skipping 457 matching lines @@
 
   SimulateTapAt(web_contents, gfx::Point(30, 60));
   DispatchInPageAndWait("waitForEvent", "gesturetap");
 
   DispatchAndWait("stopTimeline");
   RunTestMethod("checkInputEventsPresent", "MouseMove", "MouseDown",
                 "MouseWheel", "GestureTap");
 
   CloseDevToolsWindow();
 }