Merge lp://staging/~karni/ubuntuone-android-files/fix-media-scan-api7 into lp://staging/ubuntuone-android-files

Proposed by Michał Karnicki
Status: Merged
Approved by: Chad Miller
Approved revision: 302
Merged at revision: 302
Proposed branch: lp://staging/~karni/ubuntuone-android-files/fix-media-scan-api7
Merge into: lp://staging/ubuntuone-android-files
Diff against target: 193 lines (+100/-8)
4 files modified
ant.properties (+2/-2)
src/com/ubuntuone/android/files/UbuntuOneFiles.java (+9/-0)
src/com/ubuntuone/android/files/service/UpDownService.java (+5/-6)
src/com/ubuntuone/android/files/util/MediaScannerHelper.java (+84/-0)
To merge this branch: bzr merge lp://staging/~karni/ubuntuone-android-files/fix-media-scan-api7
Reviewer Review Type Date Requested Status
Chad Miller (community) Approve
Review via email: mp+95683@code.staging.launchpad.net

Commit message

Removed API 8 method call to MediaScannerConnection.scanFile().

Description of the change

Since U1F minSdkVersion="7", a call to class method MediaScannerConnection.scanFile() has been replaced with helper class, MediaScannerHelper. Refernce to it is kept in UbuntuOneFiles Application instance, as calls to MediaScannerConnection are async, and we don't want them to get GC'ed. Reproduced the problem on HTC Hero 2.1, this fixes the bug.

To post a comment you must log in.
Revision history for this message
Chad Miller (cmiller) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches

to status/vote changes: