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

Issue 272543003: Remove unused code for the metro window switcher button (Closed)

Created:
6 years, 7 months ago by tdanderson
Modified:
6 years, 7 months ago
Reviewers:
cpu_(ooo_6.6-7.5), sky
CC:
chromium-reviews, tfarina, sadrul
Visibility:
Public.

Description

Remove unused code for the metro window switcher button We no longer use the "window switcher" button in Win8 metro mode, so remove all of the dead code related to this. The button was used pre-Ash to allow easy switching between a regular fullscreen browser window and an incognito fullscreen window. BUG=370913 TEST=none Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=268955

Patch Set 1 #

Messages

Total messages: 6 (0 generated)
tdanderson
sky@ and erg@, can you please take a look?
6 years, 7 months ago (2014-05-07 14:57:32 UTC) #1
tdanderson
Sorry, I meant cpu@, not erg@.
6 years, 7 months ago (2014-05-07 14:58:07 UTC) #2
sky
Nice, LGTM
6 years, 7 months ago (2014-05-07 16:35:17 UTC) #3
tdanderson
The CQ bit was checked by tdanderson@chromium.org
6 years, 7 months ago (2014-05-07 17:06:32 UTC) #4
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/tdanderson@chromium.org/272543003/1
6 years, 7 months ago (2014-05-07 17:10:25 UTC) #5
commit-bot: I haz the power
6 years, 7 months ago (2014-05-07 21:27:51 UTC) #6
Message was sent while issue was closed.
Change committed as 268955

Powered by Google App Engine
This is Rietveld 408576698