android: Move JavaExceptionReporter to base

android: Move JavaExceptionReporter to base

So that it can be used in other processes as well.
There as some comment changes, and added @MainDex. But otherwise
a straightforward move.

BUG=680775
Review-Url: https://codereview.chromium.org/2667263002
Cr-Commit-Position: refs/heads/master@{#448000}
Committed: https://chromium.googlesource.com/chromium/src/+/9b51b5184179dd0a76fdc4e0e851b22a8aa842d5

[boliu, 2017-02-02 03:55:49]

boliu@chromium.org changed reviewers:
+ tedchoc@chromium.org, torne@chromium.org

[boliu, 2017-02-02 03:55:50]

ptal

torne for base
tedchoc for android stuff under chrome

[Torne, 2017-02-02 14:33:05]

The UI thread is a concept maintained in base and there's other code in base
that checks it's running/not running on the UI thread, so I don't think that
needs to be removed..

[boliu, 2017-02-02 15:10:32]

Description was changed from

==========
android: Move JavaExceptionReporter to base

So that it can be used in other processes as well.
There as some comment changes, and removed the requirement for UI thread
which makes no sense in base.

BUG=680775
==========

to

==========
android: Move JavaExceptionReporter to base

So that it can be used in other processes as well.
There as some comment changes, but otherwise a straightforward move.

BUG=680775
==========

[boliu, 2017-02-02 15:10:44]

alright, UI check is back

[Torne, 2017-02-02 15:14:03]

lgtm

[Ted C, 2017-02-02 17:34:07]

chrome - lgtm

[boliu, 2017-02-02 17:48:26]

oh, one more change to base, added @MainDex, because intention is this will be
used from child processes

[boliu, 2017-02-02 17:48:51]

Description was changed from

==========
android: Move JavaExceptionReporter to base

So that it can be used in other processes as well.
There as some comment changes, but otherwise a straightforward move.

BUG=680775
==========

to

==========
android: Move JavaExceptionReporter to base

So that it can be used in other processes as well.
There as some comment changes, and added @MainDex. But otherwise
a straightforward move.

BUG=680775
==========

[boliu, 2017-02-02 17:50:30]

boliu@chromium.org changed reviewers:
+ cpu@chromium.org

[boliu, 2017-02-02 17:50:31]

+cpu to stamp chrome/app

[boliu, 2017-02-02 23:35:46]

boliu@chromium.org changed reviewers:
+ grt@chromium.org
- cpu@chromium.org

[boliu, 2017-02-02 23:35:47]

+grt for chrome/app

Not sure if cpu@ is working on chromium..

[grt (UTC plus 2), 2017-02-03 08:18:06]

chrome/app/chrome_main_delegate.cc lgtn

[grt (UTC plus 2), 2017-02-03 08:18:14]

On 2017/02/03 08:18:06, grt (UTC plus 1) wrote:
> chrome/app/chrome_main_delegate.cc lgtn

lgtm, too!

[boliu, 2017-02-03 14:23:13]

The CQ bit was checked by boliu@chromium.org

[boliu, 2017-02-03 14:23:13]

The patchset sent to the CQ was uploaded after l-g-t-m from
tedchoc@chromium.org, torne@chromium.org
Link to the patchset: https://codereview.chromium.org/2667263002/#ps40001
(title: "maindex")

[commit-bot: I haz the power, 2017-02-03 14:23:34]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-02-03 15:35:10]

CQ is committing da patch.

Bot data: {"patchset_id": 40001, "attempt_start_ts": 1486131793003930,
"parent_rev": "9f800ec0044b9c7c7c88553f406cc5e146e72516", "commit_rev":
"9b51b5184179dd0a76fdc4e0e851b22a8aa842d5"}

[commit-bot: I haz the power, 2017-02-03 15:35:50]

Description was changed from

==========
android: Move JavaExceptionReporter to base

So that it can be used in other processes as well.
There as some comment changes, and added @MainDex. But otherwise
a straightforward move.

BUG=680775
==========

to

==========
android: Move JavaExceptionReporter to base

So that it can be used in other processes as well.
There as some comment changes, and added @MainDex. But otherwise
a straightforward move.

BUG=680775

Review-Url: https://codereview.chromium.org/2667263002
Cr-Commit-Position: refs/heads/master@{#448000}
Committed:
https://chromium.googlesource.com/chromium/src/+/9b51b5184179dd0a76fdc4e0e851...
==========

[commit-bot: I haz the power, 2017-02-03 15:35:51]

Committed patchset #3 (id:40001) as
https://chromium.googlesource.com/chromium/src/+/9b51b5184179dd0a76fdc4e0e851...