Refactoring DevToolsBridgeServer.

Refactoring DevToolsBridgeServer.

After prototyping integration of DevToolsBridgeServer into Clank there are several fixed found needed:
1. Remove UI and service lifetime management from DevToolsBridgeServer. While it causes some duplication
in tests it makes UI much more flexible. Lifetime is easier to understand when places closer to context.
2. Decouple DevToolsBridgeServer from Service. It makes testing simpler.
3. Support for asynchronous socket resolution. Socket name in Clank may change (when service runs in a separate process) and name resolution involves IO.

BUG=383418
Committed: https://crrev.com/7cd18625ce3d1ad965edf6798871499e589242b6
Cr-Commit-Position: refs/heads/master@{#308802}

[SeRya, 2014-12-17 13:32:48]

Patchset #1 (id:1) has been deleted

[SeRya, 2014-12-17 13:40:40]

serya@chromium.org changed reviewers:
+ mnaganov@chromium.org

[SeRya, 2014-12-17 13:40:41]

PTAL

[mnaganov (inactive), 2014-12-17 15:56:43]

LGTM

https://codereview.chromium.org/811623005/diff/20001/components/devtools_brid...
File
components/devtools_bridge/android/java/src/org/chromium/components/devtools_bridge/DevToolsBridgeServiceBase.java
(right):

https://codereview.chromium.org/811623005/diff/20001/components/devtools_brid...
components/devtools_bridge/android/java/src/org/chromium/components/devtools_bridge/DevToolsBridgeServiceBase.java:101:
* Unkike similar method in IntentService this one runs on service thread.
typo: unlike

https://codereview.chromium.org/811623005/diff/20001/components/devtools_brid...
File
components/devtools_bridge/test/android/javatests/src/org/chromium/components/devtools_bridge/ClientSessionTestingHost.java
(right):

https://codereview.chromium.org/811623005/diff/20001/components/devtools_brid...
components/devtools_bridge/test/android/javatests/src/org/chromium/components/devtools_bridge/ClientSessionTestingHost.java:64:
Log.d(TAG, "Failure: " + message);
Log.e ?

[SeRya, 2014-12-17 16:03:55]

https://codereview.chromium.org/811623005/diff/20001/components/devtools_brid...
File
components/devtools_bridge/android/java/src/org/chromium/components/devtools_bridge/DevToolsBridgeServiceBase.java
(right):

https://codereview.chromium.org/811623005/diff/20001/components/devtools_brid...
components/devtools_bridge/android/java/src/org/chromium/components/devtools_bridge/DevToolsBridgeServiceBase.java:101:
* Unkike similar method in IntentService this one runs on service thread.
On 2014/12/17 15:56:43, mnaganov (cr) wrote:
> typo: unlike

Done.

https://codereview.chromium.org/811623005/diff/20001/components/devtools_brid...
File
components/devtools_bridge/test/android/javatests/src/org/chromium/components/devtools_bridge/ClientSessionTestingHost.java
(right):

https://codereview.chromium.org/811623005/diff/20001/components/devtools_brid...
components/devtools_bridge/test/android/javatests/src/org/chromium/components/devtools_bridge/ClientSessionTestingHost.java:64:
Log.d(TAG, "Failure: " + message);
On 2014/12/17 15:56:43, mnaganov (cr) wrote:
> Log.e ?

Done.

[SeRya, 2014-12-17 16:04:01]

The CQ bit was checked by serya@chromium.org

[commit-bot: I haz the power, 2014-12-17 16:04:45]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/811623005/40001

[commit-bot: I haz the power, 2014-12-17 17:19:40]

Committed patchset #2 (id:40001)

[commit-bot: I haz the power, 2014-12-17 17:25:17]

Patchset 2 (id:??) landed as
https://crrev.com/7cd18625ce3d1ad965edf6798871499e589242b6
Cr-Commit-Position: refs/heads/master@{#308802}