Don't rely only on __del__ to cleanup PowerMetric when Smoothness.DidRunActions is not called.

tools/perf/measurements/smoothness_unittest.py

 # Copyright 2013 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.
 import sys
 
 from measurements import smoothness
+from telemetry.core import exceptions
 from telemetry.core import wpr_modes
 from telemetry.page import page
 from telemetry.page import page_measurement_unittest_base
+from telemetry.page import page_test
 from telemetry.unittest import options_for_unittests
 
 class FakePlatform(object):
   def IsRawDisplayFrameRateSupported(self):
     return False
   def CanMonitorPower(self):
     return False
 
 
 class FakeBrowser(object):
@@ skipping 104 matching lines @@
     measurement = smoothness.Smoothness()
     results = self.RunMeasurement(measurement, ps, options=self._options)
     self.assertEquals(0, len(results.failures))
 
     mostly_smooth = results.FindAllPageSpecificValuesNamed('mostly_smooth')
     self.assertEquals(len(mostly_smooth), 1)
     self.assertGreaterEqual(mostly_smooth[0].GetRepresentativeNumber(), 0)
 
   def testCleanUpTrace(self):
     self.TestTracingCleanedUp(smoothness.Smoothness, self._options)
+
+  def testCleanUpPowerMetric(self):
+    class FailPage(page.Page):
+      def __init__(self, page_set):
+        super(FailPage, self).__init__(
+            url='file://blank.html',
+            page_set=page_set, base_dir=page_set.base_dir)
+      def RunSmoothness(self, _):
+        raise exceptions.IntentionalException
+
+    ps = self.CreateEmptyPageSet()
+    ps.AddPage(FailPage(ps))
+
+    start_measuring_called = [False]
+    stop_measuring_called = [False]
+    class BuggyMeasurement(smoothness.Smoothness):
+      # Inject fake measuring_power methods to browser
+      def WillStartBrowser(self, browser):
+        super(BuggyMeasurement, self).WillStartBrowser(browser)
+        def FakeStart(_1, _2):
+          start_measuring_called[0] = True
+        def FakeStop(_1, _2):
+          stop_measuring_called[0] = True
+        self._power_metric.Start = FakeStart
+        self._power_metric.Stop = FakeStop
+
+    measurement = BuggyMeasurement()
+    try:
+      self.RunMeasurement(measurement, ps)
+    except page_test.TestNotSupportedOnPlatformFailure:
+      pass
+
+    self.assertTrue(start_measuring_called[0])
+    self.assertTrue(stop_measuring_called[0])