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

Issues Search
    My Issues | Starred     Open | Closed | All

Issue 1809023003: Measure the impact of a proposed media element load algorithm change (Closed)

Created:
4 years, 9 months ago by philipj_slow
Modified:
4 years, 9 months ago
CC:
asvitkine+watch_chromium.org, blink-reviews, blink-reviews-html_chromium.org, chromium-reviews, dglazkov+blink, eric.carlson_apple.com, feature-media-reviews_chromium.org, fs, gasubic, mlamouri+watch-blink_chromium.org, philipj_slow, nessy, vcarbune.chromium
Base URL:
https://chromium.googlesource.com/chromium/src.git@master
Target Ref:
refs/pending/heads/master
Project:
chromium
Visibility:
Public.

Description

Measure the impact of a proposed media element load algorithm change https://github.com/whatwg/html/issues/869#issuecomment-196189743 R=mlamouri@chromium.org,isherman@chromium.org Committed: https://crrev.com/6d63e3c42f8a171d849a04dcdb179e06335c109d Cr-Commit-Position: refs/heads/master@{#381787}

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+9 lines, -0 lines) Patch
M third_party/WebKit/Source/core/frame/UseCounter.h View 1 chunk +1 line, -0 lines 0 comments Download
M third_party/WebKit/Source/core/html/HTMLMediaElement.cpp View 1 chunk +7 lines, -0 lines 0 comments Download
M tools/metrics/histograms/histograms.xml View 1 chunk +1 line, -0 lines 0 comments Download

Messages

Total messages: 12 (4 generated)
philipj_slow
4 years, 9 months ago (2016-03-17 15:28:39 UTC) #2
commit-bot: I haz the power
Dry run: CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1809023003/1 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/1809023003/1
4 years, 9 months ago (2016-03-17 15:28:48 UTC) #3
mlamouri (slow - plz ping)
lgtm
4 years, 9 months ago (2016-03-17 15:38:30 UTC) #4
Ilya Sherman
histograms.xml lgtm
4 years, 9 months ago (2016-03-17 16:40:41 UTC) #5
commit-bot: I haz the power
Dry run: This issue passed the CQ dry run.
4 years, 9 months ago (2016-03-17 17:21:31 UTC) #7
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1809023003/1 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/1809023003/1
4 years, 9 months ago (2016-03-17 20:18:28 UTC) #9
commit-bot: I haz the power
Committed patchset #1 (id:1)
4 years, 9 months ago (2016-03-17 20:27:13 UTC) #10
commit-bot: I haz the power
4 years, 9 months ago (2016-03-17 20:29:31 UTC) #12
Message was sent while issue was closed. 
Patchset 1 (id:??) landed as
https://crrev.com/6d63e3c42f8a171d849a04dcdb179e06335c109d
Cr-Commit-Position: refs/heads/master@{#381787}

Powered by Google App Engine
This is Rietveld 408576698