Source/modules/battery/BatteryManager.idl - Issue 203683002: BatteryStatus: use 'unrestricted double' instead of 'double'

Side by Side Diff

Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.

Keyboard Shortcuts

	File
u :	up to issue
j / k :	jump to file after / before current file
J / K :	jump to next file with a comment after / before current file
	Side-by-side diff
i :	toggle intra-line diffs
e :	expand all comments
c :	collapse all comments
s :	toggle showing all comments
n / p :	next / previous diff chunk or comment
N / P :	next / previous comment
<Up> / <Down> :	next / previous line

	Issue
u :	up to list of issues
j / k :	jump to patch after / before current patch
o / <Enter> :	open current patch in side-by-side view
i :	open current patch in unified diff view

	Issue List
j / k :	jump to issue after / before current issue
o / <Enter> :	open current issue

Side by Side Diff: Source/modules/battery/BatteryManager.idl

Issue 203683002: BatteryStatus: use 'unrestricted double' instead of 'double' (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master

Patch Set: Created 6 years, 9 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

« no previous file with comments | « no previous file | no next file » | no next file with comments »

OLD	NEW
1 // Copyright 2014 The Chromium Authors. All rights reserved.	1 // Copyright 2014 The Chromium Authors. All rights reserved.

2 // Use of this source code is governed by a BSD-style license that can be	2 // Use of this source code is governed by a BSD-style license that can be

3 // found in the LICENSE file.	3 // found in the LICENSE file.

4	4

5 // http://dev.w3.org/2009/dap/system-info/battery-status.html	5 // http://dev.w3.org/2009/dap/system-info/battery-status.html
	Nils Barth (inactive) 2014/05/01 01:19:50 More precise URL? https://dvcs.w3.org/hg/dap/raw- More precise URL? https://dvcs.w3.org/hg/dap/raw-file/tip/battery/Overview.html#batterymanager-... mlamouri (slow - plz ping) 2014/05/01 09:33:16 Done. Show quoted text On 2014/05/01 01:19:50, Nils Barth wrote: > More precise URL? > > https://dvcs.w3.org/hg/dap/raw-file/tip/battery/Overview.html#batterymanager-... Done.
6 [	6 [

7 NoInterfaceObject,	7 NoInterfaceObject,

8 RuntimeEnabled=BatteryStatus,	8 RuntimeEnabled=BatteryStatus,
	Nils Barth (inactive) 2014/05/01 01:19:50 Could you add: TypeChecking=Unrestricted, ...h Could you add: TypeChecking=Unrestricted, ...here? Thanks! mlamouri (slow - plz ping) 2014/05/01 09:33:16 Done. Show quoted text On 2014/05/01 01:19:50, Nils Barth wrote: > Could you add: > TypeChecking=Unrestricted, > ...here? > Thanks! Done.
9 ] interface BatteryManager : EventTarget {	9 ] interface BatteryManager : EventTarget {

10 readonly attribute boolean charging;	10 readonly attribute boolean charging;

11 readonly attribute double chargingTime;	11 readonly attribute unrestricted double chargingTime;

12 readonly attribute double dischargingTime;	12 readonly attribute unrestricted double dischargingTime;

13 readonly attribute double level;	13 readonly attribute double level;

14	14

15 attribute EventHandler onchargingchange;	15 attribute EventHandler onchargingchange;

16 attribute EventHandler onchargingtimechange;	16 attribute EventHandler onchargingtimechange;

17 attribute EventHandler ondischargingtimechange;	17 attribute EventHandler ondischargingtimechange;

18 attribute EventHandler onlevelchange;	18 attribute EventHandler onlevelchange;

19	19
	Nils Barth (inactive) 2014/05/01 01:19:50 Could you remove the empty line? Could you remove the empty line? mlamouri (slow - plz ping) 2014/05/01 09:33:16 Done. Show quoted text On 2014/05/01 01:19:50, Nils Barth wrote: > Could you remove the empty line? Done.
20 };	20 };

21	21
	Nils Barth (inactive) 2014/05/01 01:19:50 ...here too? ...here too? mlamouri (slow - plz ping) 2014/05/01 09:33:16 Done. Show quoted text On 2014/05/01 01:19:50, Nils Barth wrote: > ...here too? Done.
OLD	NEW