Closed Bug 699744 Opened 13 years ago Closed 13 years ago

Implement battery.dischargingTime

Categories

(Core :: DOM: Core & HTML, defect)

Product:
Core
Component:
DOM: Core & HTML
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED FIXED
Milestone:
mozilla11

People

(Reporter: mounir, Assigned: mounir)

References

Details

Attachments

(1 file)

Add .dischargingTime in the DOM
8.44 KB, patch
sicking
: review+
asa
: approval-mozilla-aurora-
Details | Diff | Splinter Review 
This has been added to the specs while implementing the first version of Battery API. The only issue I see with this new property is that it will tell the author whether or not the device has a battery coupled to battery.chargingTime (both will return Infinity if there is no battery).
Assignee: nobody → mounir
Status: NEW → ASSIGNED

        Attachment #572427 -
        Flags: review?(jonas)

    
  
Whiteboard: [needs review]

    
  
Blocks: 700261

    
  
Flags: in-testsuite?
Whiteboard: [needs review]

    
    

    
  
Comment on attachment 572427 [details] [diff] [review]
Add .dischargingTime in the DOM

This has just landed in mozilla-inbound. Requesting aurora approval for API completeness in Firefox 10. It would be better to ship Firefox 10 with the entire API if we say it is present. The risk is very low given that there is no use of the API for the moment.

        Attachment #572427 -
        Flags: approval-mozilla-aurora?

    
    

    
  
https://hg.mozilla.org/mozilla-central/rev/a6bdf740fa19
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla11

    
    

    
  
Comment on attachment 572427 [details] [diff] [review]
Add .dischargingTime in the DOM

It looks like this feature wasn't ready for 10. Please re-nominate if you have a good justification for trying to get this into 10 rather than just waiting for 11.

        Attachment #572427 -
        Flags: approval-mozilla-aurora? → approval-mozilla-aurora-
Component: DOM → DOM: Core & HTML

          You need to log in
          before you can comment on or make changes to this bug.
        






  

    
  







  

    

      
Attachment

      

      
      
      
      
    

    

      

        

          

            
General

            

              Creator: 



            

            
Created: 

            
Updated: 

            
Size: