Vincent Chang[:vchang][changyihsin] Reporter
	Description • 11 years ago

      No description provided.

	Gavin Lai[:glai]
	Comment 1 • 11 years ago

One of the key issue is how to know which app is using Wi-Fi. 

To my knowledge, Android uses Wifilock to know which app is using Wi-Fi. Acquiring a wifilock can keep Wi-Fi awake until the lock is released. Making use of this mechanism, Android can know which app is acquiring Wi-Fi. Then, using timer to measure the Wi-Fi running time.

	Vincent Chang[:vchang][changyihsin] Reporter
	Comment 2 • 11 years ago

Once we can have network statistic for wifi interface, we can calculate the power consumption for wifi component by using tx/rx packets, connection rate and average power consumption for wifi component.

	Gavin Lai[:glai]
	Comment 3 • 11 years ago

This patch implements in NetworkStatsService to notify PowerStatsService some information. The formation is manifestURL, txBytes + rxBytes and wifi link speed.

	Gavin Lai[:glai]
	Comment 5 • 11 years ago

This patch implements in NetworkStatsService to notify PowerStatsService some information. The formation is manifestURL, txBytes + rxBytes and wifi link speed.
This patch depends on Bug 784575.

	Gavin Lai[:glai]
	Comment 6 • 11 years ago

After getting the information from NetworkStatsService. PowerStatsService estimates power consumption of wifi transmission.

	Gavin Lai[:glai]
	Comment 7 • 11 years ago

Fix bugs.

	Vincent Chang[:vchang][changyihsin] Reporter
	Comment 8 • 11 years ago

Comment on attachment 787968 [details] [diff] [review]
Part1: estimate power consumption of wifi transmission

Review of attachment 787968 [details] [diff] [review]:
-----------------------------------------------------------------

Please fix some nits.

::: dom/power/PowerStatsService.jsm
@@ +37,5 @@
>  
> +XPCOMUtils.defineLazyServiceGetter(this, "networkstatsService",
> +                                   "@mozilla.org/networkstatsService;1",
> +                                   "nsINetworkStatsService");
> +

This seems not used in this patch. Please remove it.

@@ +313,5 @@
>      if (item == "pid") {
>        appPowerInfo[index].pid = value[0];
>      }
>  
> +    if (item === "wifiactive") {

value is a array right ? Please add the comment to explain the format of value.

	BMO Automation
	Comment 11 • 6 years ago

Firefox OS is not being worked on