Media Hub

Code review comment for lp:~thomas-voss/media-hub/add-display-controller-interface

  1. add-display-controller-interface
  2. Merge into trunk
Revision history for this message
Jim Hodapp (jhodapp) wrote :

A lot of code but in general looking good. See my comments inline below. Also, I'd like to see unit tests for all of the new classes that you introduced such as BatteryObserver, StateController, etc.

review: Needs Fixing (code)
Reply

« Back to merge proposal