am 625cfe3d: Merge "ResearchLog uploadingService now waits for logging to finish" into jb-mr1-dev

* commit '625cfe3d2119538b508c0d2eb82d1098579e467e':
  ResearchLog uploadingService now waits for logging to finish
main
Kurt Partridge 2012-09-05 10:41:42 -07:00 committed by Android Git Automerger
commit ae6f6e8109
2 changed files with 12 additions and 5 deletions

View File

@ -93,7 +93,7 @@ public class ResearchLog {
mFile = outputFile; mFile = outputFile;
} }
public synchronized void close() { public synchronized void close(final Runnable onClosed) {
mExecutor.submit(new Callable<Object>() { mExecutor.submit(new Callable<Object>() {
@Override @Override
public Object call() throws Exception { public Object call() throws Exception {
@ -111,6 +111,9 @@ public class ResearchLog {
if (mFile.exists()) { if (mFile.exists()) {
mFile.setWritable(false, false); mFile.setWritable(false, false);
} }
if (onClosed != null) {
onClosed.run();
}
} }
return null; return null;
} }

View File

@ -376,11 +376,11 @@ public class ResearchLogger implements SharedPreferences.OnSharedPreferenceChang
if (mMainLogBuffer != null) { if (mMainLogBuffer != null) {
publishLogBuffer(mMainLogBuffer, mMainResearchLog, false /* isIncludingPrivateData */); publishLogBuffer(mMainLogBuffer, mMainResearchLog, false /* isIncludingPrivateData */);
mMainResearchLog.close(); mMainResearchLog.close(null /* callback */);
mMainLogBuffer = null; mMainLogBuffer = null;
} }
if (mFeedbackLogBuffer != null) { if (mFeedbackLogBuffer != null) {
mFeedbackLog.close(); mFeedbackLog.close(null /* callback */);
mFeedbackLogBuffer = null; mFeedbackLogBuffer = null;
} }
} }
@ -549,8 +549,12 @@ public class ResearchLogger implements SharedPreferences.OnSharedPreferenceChang
false /* isPotentiallyPrivate */); false /* isPotentiallyPrivate */);
mFeedbackLogBuffer.shiftIn(feedbackLogUnit); mFeedbackLogBuffer.shiftIn(feedbackLogUnit);
publishLogBuffer(mFeedbackLogBuffer, mFeedbackLog, true /* isIncludingPrivateData */); publishLogBuffer(mFeedbackLogBuffer, mFeedbackLog, true /* isIncludingPrivateData */);
mFeedbackLog.close(); mFeedbackLog.close(new Runnable() {
uploadNow(); @Override
public void run() {
uploadNow();
}
});
mFeedbackLog = new ResearchLog(createLogFile(mFilesDir)); mFeedbackLog = new ResearchLog(createLogFile(mFilesDir));
} }