bluetooth: Improve testability

bluetooth: Improve testability

Make a bunch of changes to the bluetooth code to improve its testability:

 - BluetoothAdapter and BluetoothDevice base observer and delegate classes
   made public, allows a test to static cast the pointer and call those
   methods as if it were the dbus client.

 - Add MockBluetoothAdapter::Observer and MockBluetoothDevice::Observer
   classes; these can be used by tests to check BluetoothAdapter and
   BluetoothDevice call observer methods at the correct time, with the
   correct arguments.

 - Add MockBluetooth*Client::Properties classes; these mock out the
   Get(), GetAll() and Set() methods so a test can verify whether or
   not they're called.

   Tests can also use MockBluetooth*Client::GetProperties() to return
   one of these mock classes constructed statically in the test, using
   ReplaceValue() on each property to set values.

BUG=none
TEST=can write a unit test

Change-Id: I9eaeb6412de891f9b1b3cdaae66d32dfecbac8c9


Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=152234

[keybuk, 2012-08-16 23:09:58]

satorux for OWNERS
bryeung for "omg finally"

[Nico, 2012-08-17 22:02:12]

https://chromiumcodereview.appspot.com/10831360/diff/2002/chromeos/dbus/mock_...
File chromeos/dbus/mock_bluetooth_adapter_client.h (right):

https://chromiumcodereview.appspot.com/10831360/diff/2002/chromeos/dbus/mock_...
chromeos/dbus/mock_bluetooth_adapter_client.h:29: MOCK_METHOD1(ChangedRecevied,
void(dbus::Signal*));
Typo Recevied.

[satorux1, 2012-08-17 22:24:28]

LGTM except Recevied found in many places.

[commit-bot: I haz the power, 2012-08-18 00:49:19]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/keybuk@chromium.org/10831360/14001

[commit-bot: I haz the power, 2012-08-18 03:19:07]

Change committed as 152234