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

Issue 451583003: Enable Battery Status API by default in Blink. (Closed)

Created:
6 years, 4 months ago by timvolodine
Modified:
6 years, 4 months ago
Reviewers:
abarth-chromium
CC:
benm (inactive), blink-reviews
Project:
blink
Visibility:
Public.

Description

Enable Battery Status API by default in Blink. Set Battery Status API status to stable in blink. We have implementations for all platforms supported by blink now, i.e. on Android, Windows, Linux, ChromeOS and Mac. BUG=122593 TEST=http://jsbin.com/battery-status-test Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=180296

Patch Set 1 #

Patch Set 2 : fix layout test #

Unified diffs Side-by-side diffs Delta from patch set Stats (+2 lines, -1 line) Patch
M LayoutTests/virtual/stable/webexposed/global-constructors-listing-expected.txt View 1 1 chunk +1 line, -0 lines 0 comments Download
M Source/platform/RuntimeEnabledFeatures.in View 1 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 6 (0 generated)
timvolodine
+ Ben for testing with WebView.
6 years, 4 months ago (2014-08-07 17:50:27 UTC) #1
timvolodine
6 years, 4 months ago (2014-08-12 18:40:24 UTC) #2
abarth-chromium
lgtm
6 years, 4 months ago (2014-08-12 19:32:00 UTC) #3
timvolodine
The CQ bit was checked by timvolodine@chromium.org
6 years, 4 months ago (2014-08-14 19:53:45 UTC) #4
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/timvolodine@chromium.org/451583003/20001
6 years, 4 months ago (2014-08-14 19:54:33 UTC) #5
commit-bot: I haz the power
6 years, 4 months ago (2014-08-14 20:00:00 UTC) #6
Message was sent while issue was closed.
Committed patchset #2 (20001) as 180296

Powered by Google App Engine
This is Rietveld 408576698