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

Issue 2703853003: Remove unreferenced workspace_layout_manager_unittest.cc (Closed)

Created:
3 years, 10 months ago by msw
Modified:
3 years, 10 months ago
Reviewers:
sky
CC:
chromium-reviews, kalyank, sadrul
Target Ref:
refs/pending/heads/master
Project:
chromium
Visibility:
Public.

Description

Remove unreferenced workspace_layout_manager_unittest.cc This file is unreferenced after (and should have been removed by): https://codereview.chromium.org/2402643004 BUG=624172 TEST=NONE R=sky@chromium.org Review-Url: https://codereview.chromium.org/2703853003 Cr-Commit-Position: refs/heads/master@{#451391} Committed: https://chromium.googlesource.com/chromium/src/+/b25fc47b3517d87ea086cea8a6b96a562c2dee01

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+0 lines, -1318 lines) Patch
D ash/wm/workspace/workspace_layout_manager_unittest.cc View 1 chunk +0 lines, -1318 lines 0 comments Download

Messages

Total messages: 7 (3 generated)
msw
Hey Scott, please take a look; thanks!
3 years, 10 months ago (2017-02-17 20:32:11 UTC) #1
sky
LGTM - thanks again for catching this!
3 years, 10 months ago (2017-02-17 20:37:40 UTC) #2
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.org/2703853003/1
3 years, 10 months ago (2017-02-17 20:39:46 UTC) #4
commit-bot: I haz the power
3 years, 10 months ago (2017-02-17 22:43:31 UTC) #7
Message was sent while issue was closed.
Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/b25fc47b3517d87ea086cea8a6b9...

Powered by Google App Engine
This is Rietveld 408576698