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

Issues Search
    My Issues | Starred     Open | Closed | All

Issue 2933173002: Deleting unused story sets (Closed)

Created:
3 years, 6 months ago by ashleymarie1
Modified:
3 years, 6 months ago
Reviewers:
nednguyen
CC:
chromium-reviews, telemetry-reviews_chromium.org
Target Ref:
refs/heads/master
Project:
chromium
Visibility:
Public.

Description

Deleting unused story sets BUG=chromium:720002 Review-Url: https://codereview.chromium.org/2933173002 Cr-Commit-Position: refs/heads/master@{#478770} Committed: https://chromium.googlesource.com/chromium/src/+/72b0661cd1fbc0297200f9ca88be19bdeb865508

Patch Set 1 #

Patch Set 2 : Deleting unused story sets #

Unified diffs Side-by-side diffs Delta from patch set Stats (+-1 lines, -2275 lines) Patch
D tools/perf/page_sets/alexa1-10000.py View 1 chunk +0 lines, -44 lines 0 comments Download
D tools/perf/page_sets/android_acceptance.py View 1 chunk +0 lines, -40 lines 0 comments Download
M tools/perf/page_sets/blank_page.py View 2 chunks +0 lines, -28 lines 0 comments Download
D tools/perf/page_sets/diagonal_scrolling_supported_shared_state.py View 1 chunk +0 lines, -16 lines 0 comments Download
M tools/perf/page_sets/dummy_story_set.py View 1 chunk +0 lines, -13 lines 0 comments Download
M tools/perf/page_sets/google_pages.py View 1 chunk +0 lines, -17 lines 0 comments Download
D tools/perf/page_sets/key_desktop_sites.py View 1 chunk +0 lines, -845 lines 0 comments Download
M tools/perf/page_sets/key_mobile_sites_pages.py View 1 chunk +0 lines, -19 lines 0 comments Download
M tools/perf/page_sets/key_mobile_sites_smooth.py View 1 chunk +0 lines, -11 lines 0 comments Download
D tools/perf/page_sets/key_search_mobile.py View 1 chunk +0 lines, -65 lines 0 comments Download
M tools/perf/page_sets/memory_top_10_mobile.py View 1 chunk +0 lines, -8 lines 0 comments Download
D tools/perf/page_sets/mobile_facebook_page.py View 1 chunk +0 lines, -19 lines 0 comments Download
D tools/perf/page_sets/mobile_memory.py View 1 chunk +0 lines, -98 lines 0 comments Download
D tools/perf/page_sets/page_cycler/__init__.py View 1 0 chunks +-1 lines, --1 lines 0 comments Download
D tools/perf/page_sets/page_cycler/alexa_us.py View 1 1 chunk +0 lines, -48 lines 0 comments Download
D tools/perf/page_sets/page_cycler/bloat.py View 1 1 chunk +0 lines, -26 lines 0 comments Download
D tools/perf/page_sets/page_cycler/dom.py View 1 1 chunk +0 lines, -36 lines 0 comments Download
D tools/perf/page_sets/page_cycler/intl1.py View 1 1 chunk +0 lines, -82 lines 0 comments Download
D tools/perf/page_sets/page_cycler/intl2.py View 1 1 chunk +0 lines, -59 lines 0 comments Download
D tools/perf/page_sets/simple_mobile_sites_fling.py View 1 chunk +0 lines, -54 lines 0 comments Download
D tools/perf/page_sets/startup_pages_record.py View 1 chunk +0 lines, -38 lines 0 comments Download
M tools/perf/page_sets/top_10.py View 1 chunk +0 lines, -24 lines 0 comments Download
M tools/perf/page_sets/top_10_mobile.py View 1 chunk +0 lines, -9 lines 0 comments Download
M tools/perf/page_sets/top_25_smooth.py View 1 chunk +0 lines, -6 lines 0 comments Download
D tools/perf/page_sets/top_7_stress.py View 1 chunk +0 lines, -324 lines 0 comments Download
D tools/perf/page_sets/top_desktop_sites_2012Q3.py View 1 chunk +0 lines, -273 lines 0 comments Download
D tools/perf/page_sets/tough_layer_cases.py View 1 chunk +0 lines, -22 lines 0 comments Download
M tools/perf/page_sets/typical_10_mobile.py View 2 chunks +0 lines, -23 lines 0 comments Download
M tools/perf/page_sets/typical_25.py View 1 chunk +0 lines, -29 lines 0 comments Download

Messages

Total messages: 10 (6 generated)
ashleymarie1
Let's get rid of some unused code!
3 years, 6 months ago (2017-06-12 19:30:32 UTC) #4
nednguyen
On 2017/06/12 19:30:32, ashleymarie1 wrote: > Let's get rid of some unused code! Amazing! lgtm
3 years, 6 months ago (2017-06-12 19:33:32 UTC) #5
commit-bot: I haz the power
CQ is trying da patch. Follow status at: https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.org/2933173002/20001
3 years, 6 months ago (2017-06-12 19:36:13 UTC) #7
commit-bot: I haz the power
3 years, 6 months ago (2017-06-12 21:24:05 UTC) #10
Message was sent while issue was closed. 
Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/chromium/src/+/72b0661cd1fbc0297200f9ca88be...

Powered by Google App Engine
This is Rietveld 408576698