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

Issue 220873002: Revert of Add a whole-app-list unit test. (Closed)

Created:
6 years, 8 months ago by Mike West
Modified:
6 years, 8 months ago
Reviewers:
tapted, calamity
CC:
chromium-reviews, chrome-apps-syd-reviews_chromium.org, tfarina
Visibility:
Public.

Description

Revert of Add a whole-app-list unit test. (https://codereview.chromium.org/210083009/) Reason for revert: This patch broke 'sizes' by adding a static initializer. Seehttp://build.chromium.org/p/chromium/builders/Linux/builds/48686/steps/sizes/logs/stdio for the detailed dump. Original issue's description: > Add a whole-app-list unit test. > > This establishes a test harness for doing tests that touch on multiple > parts of the app launcher UI, or that need coverage for the various app > launcher configurations: Ash or Desktop; regular or experimental > "landscape" mode. > > It currently just tests displaying and closing the app launcher in its > default state with a flat (no folders), 2- or 3-page model. > > BUG=169114 > TEST=Added unit tests AppListViewTest{Aura,Desktop}.* > > Previously committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=260762 > > Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=260803 TBR=calamity@chromium.org,tapted@chromium.org NOTREECHECKS=true NOTRY=true BUG=169114 Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=260826

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+7 lines, -271 lines) Patch
M chrome/browser/ui/views/app_list/win/app_list_service_win.cc View 1 chunk +1 line, -1 line 0 comments Download
M ui/app_list/app_list.gyp View 1 chunk +0 lines, -1 line 0 comments Download
M ui/app_list/views/app_list_view.h View 2 chunks +1 line, -2 lines 0 comments Download
M ui/app_list/views/app_list_view.cc View 3 chunks +5 lines, -6 lines 0 comments Download
D ui/app_list/views/app_list_view_unittest.cc View 1 chunk +0 lines, -261 lines 0 comments Download

Messages

Total messages: 3 (0 generated)
Mike West
Created Revert of Add a whole-app-list unit test.
6 years, 8 months ago (2014-04-01 11:07:49 UTC) #1
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/mkwst@chromium.org/220873002/1
6 years, 8 months ago (2014-04-01 11:08:10 UTC) #2
commit-bot: I haz the power
6 years, 8 months ago (2014-04-01 11:08:43 UTC) #3
Message was sent while issue was closed.
Change committed as 260826

Powered by Google App Engine
This is Rietveld 408576698