Simplify Physical Web hook in Chromium startup

Simplify Physical Web hook in Chromium startup

This will allow us to do more on startup without having to put more
lines into both the DeferredStartupHandler and the PhysicalWeb
class.  Eventually, we may want to record some UMA stats on startup,
and this will allow us to do so easily.

BUG=594194
Committed: https://crrev.com/dd04070aa5257204200985a97148f7f8de4d508b
Cr-Commit-Position: refs/heads/master@{#387785}

[cco3, 2016-04-13 00:14:42]

cco3@chromium.org changed reviewers:
+ mattreynolds@chromium.org

[mattreynolds, 2016-04-13 17:27:36]

lgtm

[cco3, 2016-04-13 17:28:50]

cco3@chromium.org changed reviewers:
+ yfriedman@chromium.org

[cco3, 2016-04-13 17:28:50]

This is a safe part of a recently reverted commit.

https://codereview.chromium.org/1878063002/

[Yaron, 2016-04-14 01:35:55]

lgtm

https://codereview.chromium.org/1882333002/diff/1/chrome/android/java/src/org...
File
chrome/android/java/src/org/chromium/chrome/browser/physicalweb/PhysicalWeb.java
(right):

https://codereview.chromium.org/1882333002/diff/1/chrome/android/java/src/org...
chrome/android/java/src/org/chromium/chrome/browser/physicalweb/PhysicalWeb.java:110:
public static void onChromeStart(ChromeApplication application) {
I'd love to see this dependency on ChromeApplication removed. This should just
be an app-context which is sufficient for an application global. There's a bug
filed (can find if desird) with desire to trim down ChromeApplication.

Happy to have this in a separate CL.

[cco3, 2016-04-15 22:09:04]

The CQ bit was checked by cco3@chromium.org

[commit-bot: I haz the power, 2016-04-15 22:10:00]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1882333002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1882333002/1

[cco3, 2016-04-15 22:10:27]

https://codereview.chromium.org/1882333002/diff/1/chrome/android/java/src/org...
File
chrome/android/java/src/org/chromium/chrome/browser/physicalweb/PhysicalWeb.java
(right):

https://codereview.chromium.org/1882333002/diff/1/chrome/android/java/src/org...
chrome/android/java/src/org/chromium/chrome/browser/physicalweb/PhysicalWeb.java:110:
public static void onChromeStart(ChromeApplication application) {
On 2016/04/14 01:35:55, Yaron wrote:
> I'd love to see this dependency on ChromeApplication removed. This should just
> be an app-context which is sufficient for an application global. There's a bug
> filed (can find if desird) with desire to trim down ChromeApplication.
> 
> Happy to have this in a separate CL.

Some of these calls rely on ChromeApplication.createPhysicalWebBleClient(). 
Where would that go instead?

[cco3, 2016-04-15 22:10:48]

On 2016/04/15 22:10:27, cco3 wrote:
>
https://codereview.chromium.org/1882333002/diff/1/chrome/android/java/src/org...
> File
>
chrome/android/java/src/org/chromium/chrome/browser/physicalweb/PhysicalWeb.java
> (right):
> 
>
https://codereview.chromium.org/1882333002/diff/1/chrome/android/java/src/org...
>
chrome/android/java/src/org/chromium/chrome/browser/physicalweb/PhysicalWeb.java:110:
> public static void onChromeStart(ChromeApplication application) {
> On 2016/04/14 01:35:55, Yaron wrote:
> > I'd love to see this dependency on ChromeApplication removed. This should
just
> > be an app-context which is sufficient for an application global. There's a
bug
> > filed (can find if desird) with desire to trim down ChromeApplication.
> > 
> > Happy to have this in a separate CL.
> 
> Some of these calls rely on ChromeApplication.createPhysicalWebBleClient(). 
> Where would that go instead?

Also, bug # would be appreciated.

[commit-bot: I haz the power, 2016-04-16 01:48:19]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-04-16 01:49:24]

Description was changed from

==========
Simplify Physical Web hook in Chromium startup

This will allow us to do more on startup without having to put more
lines into both the DeferredStartupHandler and the PhysicalWeb
class.  Eventually, we may want to record some UMA stats on startup,
and this will allow us to do so easily.

BUG=594194
==========

to

==========
Simplify Physical Web hook in Chromium startup

This will allow us to do more on startup without having to put more
lines into both the DeferredStartupHandler and the PhysicalWeb
class.  Eventually, we may want to record some UMA stats on startup,
and this will allow us to do so easily.

BUG=594194

Committed: https://crrev.com/dd04070aa5257204200985a97148f7f8de4d508b
Cr-Commit-Position: refs/heads/master@{#387785}
==========

[commit-bot: I haz the power, 2016-04-16 01:49:25]

Patchset 1 (id:??) landed as
https://crrev.com/dd04070aa5257204200985a97148f7f8de4d508b
Cr-Commit-Position: refs/heads/master@{#387785}