Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(205)

Unified Diff: tools/telemetry/telemetry/page/record_wpr.py

Issue 420893002: Close open file handle before moving file when recording. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 6 years, 5 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « tools/telemetry/telemetry/page/page_set_archive_info.py ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: tools/telemetry/telemetry/page/record_wpr.py
diff --git a/tools/telemetry/telemetry/page/record_wpr.py b/tools/telemetry/telemetry/page/record_wpr.py
index 561a7f9d85e95c960bf8b011499dbb4d6415fa92..61df640ad1a7e13369684ab95ebd23f4e7f45648 100755
--- a/tools/telemetry/telemetry/page/record_wpr.py
+++ b/tools/telemetry/telemetry/page/record_wpr.py
@@ -3,9 +3,7 @@
# Use of this source code is governed by a BSD-style license that can be
# found in the LICENSE file.
import logging
-import os
import sys
-import tempfile
import time
from telemetry import benchmark
@@ -119,7 +117,6 @@ class WprRecorder(object):
def __init__(self, base_dir, target, extra_args=None):
action_names_to_run = FindAllActionNames(base_dir)
self._record_page_test = RecorderPageTest(action_names_to_run)
- self._temp_target_wpr_file_path = tempfile.mkstemp()[1]
self._options = self._CreateOptions()
self._benchmark = _MaybeGetInstanceOfClass(target, base_dir,
@@ -169,8 +166,7 @@ class WprRecorder(object):
return ps
def Record(self):
- self._page_set.wpr_archive_info.AddNewTemporaryRecording(
- self._temp_target_wpr_file_path)
+ self._page_set.wpr_archive_info.AddNewTemporaryRecording()
self._record_page_test.CustomizeBrowserOptions(self._options)
return page_runner.Run(self._record_page_test, self._page_set,
test_expectations.TestExpectations(), self._options)
@@ -180,12 +176,7 @@ class WprRecorder(object):
logging.warning('Some pages failed and/or were skipped. The recording '
'has not been updated for these pages.')
results.PrintSummary()
-
- if results.successes:
- # Update the metadata for the pages which were recorded.
- self._page_set.wpr_archive_info.AddRecordedPages(results.successes)
- else:
- os.remove(self._temp_target_wpr_file_path)
+ self._page_set.wpr_archive_info.AddRecordedPages(results.successes)
def Main(base_dir):
« no previous file with comments | « tools/telemetry/telemetry/page/page_set_archive_info.py ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698