Fix StackOverFlow in AdvancedMockContext.

Fix StackOverFlow in AdvancedMockContext.

The AdvancedMockContext previously did not override registerComponentCallbacks
and unregisterComponentCallbacks. The implementation of these methods in
Context calls getApplicationContext before delegating the call to it, and since
AdvancedMockContext returns |this| in getApplicationContext, this leads to a
loop.

This CL adds overrides for these two methods that make the calls to the base
context instead, which will typically either be a MockContext or the target
context being instrumented.

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

[nyquist, 2014-07-17 17:51:51]

yfriedman: PTAL

[Yaron, 2014-07-17 20:06:47]

https://codereview.chromium.org/404553005/diff/1/base/test/android/javatests/...
File
base/test/android/javatests/src/org/chromium/base/test/util/AdvancedMockContext.java
(right):

https://codereview.chromium.org/404553005/diff/1/base/test/android/javatests/...
base/test/android/javatests/src/org/chromium/base/test/util/AdvancedMockContext.java:70:
getBaseContext().registerComponentCallbacks(callback);
I think for contexts which don't override this, it'll still come back to us from
the base context impl.

[nyquist, 2014-07-18 00:35:18]

https://codereview.chromium.org/404553005/diff/1/base/test/android/javatests/...
File
base/test/android/javatests/src/org/chromium/base/test/util/AdvancedMockContext.java
(right):

https://codereview.chromium.org/404553005/diff/1/base/test/android/javatests/...
base/test/android/javatests/src/org/chromium/base/test/util/AdvancedMockContext.java:70:
getBaseContext().registerComponentCallbacks(callback);
On 2014/07/17 20:06:47, Yaron wrote:
> I think for contexts which don't override this, it'll still come back to us
from
> the base context impl.

Oddly enough, base != super in this case.

getBaseContext() returns the mBase member field of ContextWrapper
(https://android.googlesource.com/platform/frameworks/base/+/master/core/java/...),
so I think this is fine.

Added tests to ensure this works as well.

[Yaron, 2014-07-18 00:52:58]

lgtm

[commit-bot: I haz the power, 2014-07-18 00:56:17]

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

[commit-bot: I haz the power, 2014-07-18 05:40:32]

FYI, CQ is re-trying this CL (attempt #1).
The failing builders are:
  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/...)

[commit-bot: I haz the power, 2014-07-18 07:12:58]

Change committed as 284023