Factor out a BluetoothChannelMac base class and a BluetoothRfcommChannelMac subclass.

Factor out a BluetoothChannelMac base class and a BluetoothRfcommChannelMac subclass.

These will allow the BluetoothSocketMac code to treat RFCOMM and L2CAP channels
uniformly, abstracting any differences via the BluetoothChannelMac class.

BUG=372495
TEST=BluetoothTest app
R=keybuk@chromium.org

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=276647

[Ilya Sherman, 2014-06-10 22:33:06]

keybuk: owner review
rpaquay, armansito, xiyuan: FYI and extra eyes

[keybuk, 2014-06-10 22:45:52]

lgtm

[Ilya Sherman, 2014-06-10 22:57:46]

The CQ bit was checked by isherman@chromium.org

[rpaquay, 2014-06-10 22:59:36]

On 2014/06/10 22:33:06, Ilya Sherman wrote:
> keybuk: owner review
> rpaquay, armansito, xiyuan: FYI and extra eyes

I don't see the bluetooth_rfcomm_channel_mac.* files in the CL. Am I missing
something?

[Ilya Sherman, 2014-06-10 23:00:15]

The CQ bit was unchecked by isherman@chromium.org

[commit-bot: I haz the power, 2014-06-10 23:00:27]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/isherman@chromium.org/328903002/1

[Ilya Sherman, 2014-06-10 23:04:23]

On 2014/06/10 22:59:36, rpaquay wrote:
> On 2014/06/10 22:33:06, Ilya Sherman wrote:
> > keybuk: owner review
> > rpaquay, armansito, xiyuan: FYI and extra eyes
> 
> I don't see the bluetooth_rfcomm_channel_mac.* files in the CL. Am I missing
> something?

Oops, thanks for catching that!  Now added.

[Ilya Sherman, 2014-06-10 23:04:52]

(keybuk, please take another look -- CL now has the shiny new files git-added)

[keybuk, 2014-06-11 20:17:30]

lgtm

[Ilya Sherman, 2014-06-11 20:19:00]

The CQ bit was checked by isherman@chromium.org

[Ilya Sherman, 2014-06-11 20:19:15]

The CQ bit was unchecked by isherman@chromium.org

[Ilya Sherman, 2014-06-11 20:19:16]

The CQ bit was checked by isherman@chromium.org

[commit-bot: I haz the power, 2014-06-11 20:20:21]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/isherman@chromium.org/328903002/20001

[commit-bot: I haz the power, 2014-06-11 21:38:24]

FYI, CQ is re-trying this CL (attempt #1).
The failing builders are:
  ios_dbg_simulator on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/ios_dbg_simulator/bui...)
  ios_rel_device on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/ios_rel_device/builds...)
  ios_rel_device_ninja on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/ios_rel_device_ninja/...)
  mac_chromium_compile_dbg on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/mac_chromium_compile_...)
  mac_chromium_rel on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/mac_chromium_rel/buil...)
  win_chromium_compile_dbg on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/win_chromium_compile_...)
  win_chromium_rel on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/win_chromium_rel/buil...)
  win_chromium_x64_rel on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/win_chromium_x64_rel/...)
  mac_gpu on tryserver.chromium.gpu
(http://build.chromium.org/p/tryserver.chromium.gpu/builders/mac_gpu/builds/15610)
  win_gpu on tryserver.chromium.gpu
(http://build.chromium.org/p/tryserver.chromium.gpu/builders/win_gpu/builds/18747)

[commit-bot: I haz the power, 2014-06-11 21:41:03]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-06-11 21:41:04]

Try jobs failed on following builders:
  ios_dbg_simulator on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/ios_dbg_simulator/bui...)
  ios_rel_device on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/ios_rel_device/builds...)
  ios_rel_device_ninja on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/ios_rel_device_ninja/...)
  mac_chromium_compile_dbg on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/mac_chromium_compile_...)
  mac_chromium_rel on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/mac_chromium_rel/buil...)
  mac_gpu on tryserver.chromium.gpu
(http://build.chromium.org/p/tryserver.chromium.gpu/builders/mac_gpu/builds/15676)
  win_gpu on tryserver.chromium.gpu
(http://build.chromium.org/p/tryserver.chromium.gpu/builders/win_gpu/builds/18810)

[Ilya Sherman, 2014-06-12 08:39:28]

The CQ bit was checked by isherman@chromium.org

[commit-bot: I haz the power, 2014-06-12 08:42:22]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/isherman@chromium.org/328903002/20001

[commit-bot: I haz the power, 2014-06-12 12:59:22]

Change committed as 276647