Bug 1308337 - Pre: clean up wrong annotation r=nalexander
authorGrigory Kruglov <gkruglov@mozilla.com>
Tue, 11 Apr 2017 22:22:28 -0400
changeset 410215 88d65a0297784a2e1ea297419bd4fc155f543d65
parent 410214 96ec270d7778bc59e6deb145ac6ff400090b9d28
child 410216 b129a46b714ed56ed7252b9b96e959762d3a4aea
push id7391
push usermtabara@mozilla.com
push dateMon, 12 Jun 2017 13:08:53 +0000
treeherdermozilla-beta@2191d7f87e2e [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersnalexander
bugs1308337
milestone55.0a1
first release with
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
last release without
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
Bug 1308337 - Pre: clean up wrong annotation r=nalexander It's not just used for testing, and annotation is causing IDE to highlight its uses in code as invalid. MozReview-Commit-ID: JvzX2VgNKom
mobile/android/services/src/main/java/org/mozilla/gecko/sync/synchronizer/RecordsChannel.java
--- a/mobile/android/services/src/main/java/org/mozilla/gecko/sync/synchronizer/RecordsChannel.java
+++ b/mobile/android/services/src/main/java/org/mozilla/gecko/sync/synchronizer/RecordsChannel.java
@@ -351,17 +351,16 @@ public class RecordsChannel implements
 
   @Override
   public RepositorySessionFetchRecordsDelegate deferredFetchDelegate(ExecutorService executor) {
     // Lie outright. We know that all of our fetch methods are safe.
     return this;
   }
 
   @Nullable
-  @VisibleForTesting
   public synchronized ReflowIsNecessaryException getReflowException() {
     return reflowException;
   }
 
   private synchronized void setReflowException(@NonNull ReflowIsNecessaryException e) {
     // It is a mistake to set reflow exception multiple times.
     if (reflowException != null) {
       throw new IllegalStateException("Reflow exception already set: " + reflowException);