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

Side by Side Diff: chrome/android/junit/src/org/chromium/chrome/browser/suggestions/TileGroupTest.java

Issue 2758753003: 📰 Refresh the tiles when a removal is undone (Closed)
Patch Set: address comments Created 3 years, 9 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 | « chrome/android/java/src/org/chromium/chrome/browser/suggestions/TileGroupDelegateImpl.java ('k') | 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 2017 The Chromium Authors. All rights reserved. 1 // Copyright 2017 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.suggestions; 5 package org.chromium.chrome.browser.suggestions;
6 6
7 import static org.hamcrest.CoreMatchers.is; 7 import static org.hamcrest.CoreMatchers.is;
8 import static org.junit.Assert.assertNotNull; 8 import static org.junit.Assert.assertNotNull;
9 import static org.junit.Assert.assertThat; 9 import static org.junit.Assert.assertThat;
10 import static org.junit.Assert.fail; 10 import static org.junit.Assert.fail;
(...skipping 29 matching lines...) Expand all
40 import org.mockito.Mock; 40 import org.mockito.Mock;
41 import org.mockito.MockitoAnnotations; 41 import org.mockito.MockitoAnnotations;
42 import org.mockito.invocation.InvocationOnMock; 42 import org.mockito.invocation.InvocationOnMock;
43 import org.mockito.stubbing.Answer; 43 import org.mockito.stubbing.Answer;
44 import org.robolectric.RuntimeEnvironment; 44 import org.robolectric.RuntimeEnvironment;
45 import org.robolectric.annotation.Config; 45 import org.robolectric.annotation.Config;
46 import org.robolectric.annotation.Implementation; 46 import org.robolectric.annotation.Implementation;
47 import org.robolectric.annotation.Implements; 47 import org.robolectric.annotation.Implements;
48 import org.robolectric.shadows.ShadowResources; 48 import org.robolectric.shadows.ShadowResources;
49 49
50 import org.chromium.base.Callback;
50 import org.chromium.chrome.R; 51 import org.chromium.chrome.R;
51 import org.chromium.chrome.browser.ChromeFeatureList; 52 import org.chromium.chrome.browser.ChromeFeatureList;
52 import org.chromium.chrome.browser.Features; 53 import org.chromium.chrome.browser.Features;
53 import org.chromium.chrome.browser.favicon.LargeIconBridge.LargeIconCallback; 54 import org.chromium.chrome.browser.favicon.LargeIconBridge.LargeIconCallback;
54 import org.chromium.chrome.browser.ntp.ContextMenuManager; 55 import org.chromium.chrome.browser.ntp.ContextMenuManager;
55 import org.chromium.chrome.browser.ntp.NTPTileSource; 56 import org.chromium.chrome.browser.ntp.NTPTileSource;
56 import org.chromium.chrome.browser.offlinepages.OfflinePageBridge; 57 import org.chromium.chrome.browser.offlinepages.OfflinePageBridge;
57 import org.chromium.testing.local.LocalRobolectricTestRunner; 58 import org.chromium.testing.local.LocalRobolectricTestRunner;
58 59
59 /** 60 /**
(...skipping 327 matching lines...) Expand 10 before | Expand all | Expand 10 after
387 }) 388 })
388 .when(tileView) 389 .when(tileView)
389 .initialize(any(Tile.class), anyInt(), anyBoolean()); 390 .initialize(any(Tile.class), anyInt(), anyBoolean());
390 return tileView; 391 return tileView;
391 } 392 }
392 393
393 private class FakeTileGroupDelegate implements TileGroup.Delegate { 394 private class FakeTileGroupDelegate implements TileGroup.Delegate {
394 public MostVisitedSites.Observer mObserver; 395 public MostVisitedSites.Observer mObserver;
395 396
396 @Override 397 @Override
397 public void removeMostVisitedItem(Tile tile) {} 398 public void removeMostVisitedItem(Tile tile, Callback<String> removalUnd oneCallback) {}
398 399
399 @Override 400 @Override
400 public void openMostVisitedItem(int windowDisposition, Tile tile) {} 401 public void openMostVisitedItem(int windowDisposition, Tile tile) {}
401 402
402 @Override 403 @Override
403 public void setMostVisitedSitesObserver( 404 public void setMostVisitedSitesObserver(
404 MostVisitedSites.Observer observer, int maxResults) { 405 MostVisitedSites.Observer observer, int maxResults) {
405 mObserver = observer; 406 mObserver = observer;
406 } 407 }
407 408
(...skipping 39 matching lines...) Expand 10 before | Expand all | Expand 10 after
447 @LayoutRes 448 @LayoutRes
448 int layoutId = invocation.getArgument(0); 449 int layoutId = invocation.getArgument(0);
449 if (layoutId != R.layout.tile_view) fail("Unexpected resource id."); 450 if (layoutId != R.layout.tile_view) fail("Unexpected resource id.");
450 return createMockTileView(); 451 return createMockTileView();
451 } 452 }
452 }); 453 });
453 return layoutInflater; 454 return layoutInflater;
454 } 455 }
455 } 456 }
456 } 457 }
OLDNEW
« no previous file with comments | « chrome/android/java/src/org/chromium/chrome/browser/suggestions/TileGroupDelegateImpl.java ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698