Add @VisibleForTesting to fix ChromePublic release build.

Add @VisibleForTesting to fix ChromePublic release build.

These two methods were being obfuscated or removed by proguard, which
cause the release build to fail for chrome_public_test_apk.

BUG=484934
R=pvalenzuela@chromium.org

Committed: https://chromium.googlesource.com/chromium/src/+/2338a2e91cf971709acdfb1dedfa70274ab6d867

[newt (away), 2015-05-21 01:09:07]

newt@chromium.org changed reviewers:
+ pvalenzuela@chromium.org

[newt (away), 2015-05-21 01:09:08]

PTAL

[pval...(no longer on Chromium), 2015-05-21 01:13:55]

lgtm

My apologies for creating this breakage. Thanks for the fix.

https://codereview.chromium.org/1150853003/diff/1/chrome/android/java/src/org...
File
chrome/android/java/src/org/chromium/chrome/browser/sync/ProfileSyncService.java
(right):

https://codereview.chromium.org/1150853003/diff/1/chrome/android/java/src/org...
chrome/android/java/src/org/chromium/chrome/browser/sync/ProfileSyncService.java:67:
public boolean hasResult() {
This method can be deleted entirely. I ended up deleting its usage and forgot to
remove it from the CL.

[newt (away), 2015-05-21 01:23:40]

On 2015/05/21 01:13:55, pvalenzuela wrote:
> lgtm
> 
> My apologies for creating this breakage. Thanks for the fix.
> 
>
https://codereview.chromium.org/1150853003/diff/1/chrome/android/java/src/org...
> File
>
chrome/android/java/src/org/chromium/chrome/browser/sync/ProfileSyncService.java
> (right):
> 
>
https://codereview.chromium.org/1150853003/diff/1/chrome/android/java/src/org...
>
chrome/android/java/src/org/chromium/chrome/browser/sync/ProfileSyncService.java:67:
> public boolean hasResult() {
> This method can be deleted entirely. I ended up deleting its usage and forgot
to
> remove it from the CL.

In the interest of landing this CL very quickly, I'll remove hasResult() as a
follow-up.

[pval...(no longer on Chromium), 2015-05-21 01:26:16]

On 2015/05/21 01:23:40, newt wrote:
> On 2015/05/21 01:13:55, pvalenzuela wrote:
> > lgtm
> > 
> > My apologies for creating this breakage. Thanks for the fix.
> > 
> >
>
https://codereview.chromium.org/1150853003/diff/1/chrome/android/java/src/org...
> > File
> >
>
chrome/android/java/src/org/chromium/chrome/browser/sync/ProfileSyncService.java
> > (right):
> > 
> >
>
https://codereview.chromium.org/1150853003/diff/1/chrome/android/java/src/org...
> >
>
chrome/android/java/src/org/chromium/chrome/browser/sync/ProfileSyncService.java:67:
> > public boolean hasResult() {
> > This method can be deleted entirely. I ended up deleting its usage and
forgot
> to
> > remove it from the CL.
> 
> In the interest of landing this CL very quickly, I'll remove hasResult() as a
> follow-up.

I'll send out the follow-up CL. Thanks again.

[commit-bot: I haz the power, 2015-05-21 01:36:53]

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

[newt (away), 2015-05-21 01:37:04]

Committed patchset #1 (id:1) manually as
2338a2e91cf971709acdfb1dedfa70274ab6d867 (presubmit successful).