OSDN Git Service

Merge "MediaBrowser: Make EXTRA_PAGE zero-based" into nyc-dev
authorSungsoo Lim <sungsoo@google.com>
Mon, 4 Apr 2016 06:56:11 +0000 (06:56 +0000)
committerAndroid (Google) Code Review <android-gerrit@google.com>
Mon, 4 Apr 2016 06:56:13 +0000 (06:56 +0000)
media/java/android/media/browse/MediaBrowser.java
media/java/android/media/browse/MediaBrowserUtils.java
media/java/android/service/media/MediaBrowserService.java

index c1805cb..7c6adad 100644 (file)
@@ -72,7 +72,7 @@ public final class MediaBrowser {
 
     /**
      * Used as an int extra field to denote the page number to subscribe.
-     * The value of {@code EXTRA_PAGE} should be greater than or equal to 1.
+     * The value of {@code EXTRA_PAGE} should be greater than or equal to 0.
      *
      * @see #EXTRA_PAGE_SIZE
      */
index b06e598..2943e60 100644 (file)
@@ -50,7 +50,7 @@ public class MediaBrowserUtils {
             startIndex1 = 0;
             endIndex1 = Integer.MAX_VALUE;
         } else {
-            startIndex1 = pageSize1 * (page1 - 1);
+            startIndex1 = pageSize1 * page1;
             endIndex1 = startIndex1 + pageSize1 - 1;
         }
 
@@ -58,7 +58,7 @@ public class MediaBrowserUtils {
             startIndex2 = 0;
             endIndex2 = Integer.MAX_VALUE;
         } else {
-            startIndex2 = pageSize2 * (page2 - 1);
+            startIndex2 = pageSize2 * page2;
             endIndex2 = startIndex2 + pageSize2 - 1;
         }
 
index ae86632..6954045 100644 (file)
@@ -657,9 +657,9 @@ public abstract class MediaBrowserService extends Service {
         if (page == -1 && pageSize == -1) {
             return list;
         }
-        int fromIndex = pageSize * (page - 1);
+        int fromIndex = pageSize * page;
         int toIndex = fromIndex + pageSize;
-        if (page < 1 || pageSize < 1 || fromIndex >= list.size()) {
+        if (page < 0 || pageSize < 1 || fromIndex >= list.size()) {
             return Collections.EMPTY_LIST;
         }
         if (toIndex > list.size()) {