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

Side by Side Diff: tools/perf/profile_creators/profile_safe_url_list.py

Issue 918893002: telemetry: Create a safe url generator. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Comments from nednguyen. Created 5 years, 10 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 unified diff | Download patch
OLDNEW
(Empty)
1 # Copyright 2015 The Chromium Authors. All rights reserved.
2 # Use of this source code is governed by a BSD-style license that can be
3 # found in the LICENSE file.
4 import json
5 import os
6 import random
7
dtu 2015/02/12 23:46:12 style guide nit: 2 blank spaces between top level
erikchen 2015/02/13 03:58:58 Done.
8 def GetShuffledSafeUrls():
9 """Returns a deterministic shuffling of safe urls. The profile generators
10 access the urls in order, and the urls are grouped by domain. The shuffling
11 reduces the load on external servers."""
12 random.seed(0)
13 url_list_copy = list(_GetSafeUrls())
14 random.shuffle(url_list_copy)
15 return url_list_copy
16
17 def _GetSafeUrls():
18 """Returns a list of safe urls by loading them from a pre-generated file."""
19 safe_url_dir = os.path.dirname(os.path.realpath(__file__))
20 safe_url_path = os.path.join(safe_url_dir, "profile_safe_url_list.json")
21 json_data = open(safe_url_path).read()
22 return json.loads(json_data)
dtu 2015/02/12 23:46:12 To ensure prompt file closing (otherwise file is c
erikchen 2015/02/13 03:58:58 Ah, good point.
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698