Chuck Lee [:chucklee, Triggered by NI, Review, and Feedback requests] Assignee
	Description • 11 years ago

Currently holding wifi lock by |navigator.requestWakeLock('wifi')| will keep wifi up while screen is off, which might consume much battery.
Keep wifi up and turn into power save mode by |wifiManager.setPowerSavingMode()| could save more pattery.
Also we are trying to handle the wifi lock state change while screen is off.

Note that holding wifi lock doesn't imply holding CPU lock.

	Chuck Lee [:chucklee, Triggered by NI, Review, and Feedback requests] Assignee
	Comment 1 • 11 years ago

1. Enable wifi power save mode instead of turning wifi off while screen is off and wifi wake lock is held. If wifi wake lock is held after wifi is turn off by screen off, it will enable wifi and turn it into power save mode.
2. Disable wifi power save mode while screen on if its power save mode is enabled by 1.
3. Add CPU lock to protect whole process.
4. Clear timeout after cpu lock is unlocked, so it won't be called again.

	(inactive after 6/18) Alive Kuo [:alive]
	Comment 2 • 11 years ago

Comment on attachment 8385923 [details]
Pull Request

Please add unit test for what you added,
especially to make sure all cpu wake lock is unlocked.

	Chuck Lee [:chucklee, Triggered by NI, Review, and Feedback requests] Assignee
	Comment 3 • 11 years ago

Comment on attachment 8385923 [details]
Pull Request

1. Fix unit test error.
2. Add unit test for wifi wake lock.

	(inactive after 6/18) Alive Kuo [:alive]
	Comment 4 • 11 years ago

Comment on attachment 8385923 [details]
Pull Request

Let's land on master next week.

	Ryan VanderMeulen [:RyanVM]
	Comment 5 • 11 years ago

Master: https://github.com/mozilla-b2g/gaia/commit/9e1b16f8a313bbdc2e9441ba07f620716e5fdd58

	Michael Vines [:m1] [:evilmachines]
	Comment 7 • 11 years ago

Can we uplift this to v1.4 please.  This has been raised as a significant Wifi power issue by test folks since v1.3.  Now that we have a fix, we shouldn't wait for the next train given that we have test folks engaged now to validate the fix.

	Preeti Raghunath(:Preeti)
	Comment 8 • 11 years ago

1.4+ for Wifi betterness

	Ryan VanderMeulen [:RyanVM]
	Comment 9 • 11 years ago

v1.4: https://github.com/mozilla-b2g/gaia/commit/26432916d10434103a822f17be4624a342cadfba