CursorToBulkCursorAdapter.close must call mCursor.close instead of mCursor.deactivate. This prevent us to call Cursor.close on cross process ContentProvider and may cause a database leak problem.

nagamatu · nagamatu · commit 69f789a12888 · 2010-03-28T14:23:51.000+09:00
Change-Id: I126457c1b709e853727f460095b518b0420aa34f
diff --git a/core/java/android/database/CursorToBulkCursorAdaptor.java b/core/java/android/database/CursorToBulkCursorAdaptor.java
@@ -143,8 +143,7 @@ public void deactivate() {
 
     public void close() {
         maybeUnregisterObserverProxy();
-        mCursor.deactivate();       
-        
+        mCursor.close();       
     }
 
     public int requery(IContentObserver observer, CursorWindow window) {

Original file line number	Diff line number	Diff line change
`@@ -143,8 +143,7 @@ public void deactivate() {`
`143`	`143`
`144`	`144`	`public void close() {`
`145`	`145`	`maybeUnregisterObserverProxy();`
`146`		`- mCursor.deactivate();`
`147`		`-`
	`146`	`+ mCursor.close();`
`148`	`147`	`}`
`149`	`148`
`150`	`149`	`public int requery(IContentObserver observer, CursorWindow window) {`