Move settings out of process. #4728

Merged
merged 2 commits into from Sep 14, 2012

Projects

None yet

3 participants

@joneschrisg
Member

There might be a regression from this, which is https://bugzilla.mozilla.org/show_bug.cgi?id=791189 .

In exchange, we make the settings app memory reclaimable, and

  • fix bluetooth device scanning
  • improve scrolling fps in settings app by 6-8fps

I think we should merge.

@fabi1cazenave fabi1cazenave commented on an outdated diff Sep 14, 2012
apps/system/js/window_manager.js
@@ -672,9 +672,6 @@ var WindowManager = (function() {
// Once this app goes OOP. (can be done by reverting a commit)
'Messages',
// Crashes when launched OOP (bug 775997)
-
- 'Settings'
- // Bluetooth is not remoted yet (bug 755943)
@fabi1cazenave
fabi1cazenave Sep 14, 2012 Contributor

Nit: gjslint will complain about the trailing comma.

@joneschrisg
Member

Updated. (I wish gh notified me of those comments. Need to investigate.)

Thanks!

@vingtetun vingtetun merged commit c95a81c into mozilla-b2g:master Sep 14, 2012
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment