Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(517)

Side by Side Diff: chrome/android/java/src/org/chromium/chrome/browser/BookmarksBridge.java

Issue 669753002: [Android] Remove @VisibleForTesting from getOtherFolder(). (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 6 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright 2013 The Chromium Authors. All rights reserved. 1 // Copyright 2013 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 package org.chromium.chrome.browser; 5 package org.chromium.chrome.browser;
6 6
7 import org.chromium.base.CalledByNative; 7 import org.chromium.base.CalledByNative;
8 import org.chromium.base.ObserverList; 8 import org.chromium.base.ObserverList;
9 import org.chromium.base.VisibleForTesting; 9 import org.chromium.base.VisibleForTesting;
10 import org.chromium.chrome.browser.profiles.Profile; 10 import org.chromium.chrome.browser.profiles.Profile;
(...skipping 258 matching lines...) Expand 10 before | Expand all | Expand 10 after
269 * @return The BookmarkId for Mobile folder node 269 * @return The BookmarkId for Mobile folder node
270 */ 270 */
271 public BookmarkId getMobileFolderId() { 271 public BookmarkId getMobileFolderId() {
272 assert mIsNativeBookmarkModelLoaded; 272 assert mIsNativeBookmarkModelLoaded;
273 return nativeGetMobileFolderId(mNativeBookmarksBridge); 273 return nativeGetMobileFolderId(mNativeBookmarksBridge);
274 } 274 }
275 275
276 /** 276 /**
277 * @return Id representing the special "other" folder from bookmark model. 277 * @return Id representing the special "other" folder from bookmark model.
278 */ 278 */
279 @VisibleForTesting
280 public BookmarkId getOtherFolderId() { 279 public BookmarkId getOtherFolderId() {
281 assert mIsNativeBookmarkModelLoaded; 280 assert mIsNativeBookmarkModelLoaded;
282 return nativeGetOtherFolderId(mNativeBookmarksBridge); 281 return nativeGetOtherFolderId(mNativeBookmarksBridge);
283 } 282 }
284 283
285 /** 284 /**
286 * @return BokmarkId representing special "desktop" folder, namely "bookmark bar". 285 * @return BokmarkId representing special "desktop" folder, namely "bookmark bar".
287 */ 286 */
288 @VisibleForTesting 287 @VisibleForTesting
289 public BookmarkId getDesktopFolderId() { 288 public BookmarkId getDesktopFolderId() {
(...skipping 462 matching lines...) Expand 10 before | Expand all | Expand 10 after
752 case GET_CURRENT_FOLDER_HIERARCHY: 751 case GET_CURRENT_FOLDER_HIERARCHY:
753 mHandler.getCurrentFolderHierarchy(mFolderId, mCallback); 752 mHandler.getCurrentFolderHierarchy(mFolderId, mCallback);
754 break; 753 break;
755 default: 754 default:
756 assert false; 755 assert false;
757 break; 756 break;
758 } 757 }
759 } 758 }
760 } 759 }
761 } 760 }
OLDNEW
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698