New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 659942 link

Starred by 1 user

Issue metadata

Status: WontFix
Owner:
Last visit > 30 days ago
Closed: Feb 2017
Cc:
EstimatedDays: ----
NextAction: ----
OS: iOS
Pri: 3
Type: Bug



Sign in to add a comment

Remove the TabSwitcher experimental flag.

Project Member Reported by jif@chromium.org, Oct 27 2016

Issue description

Work to be done:
-Enable the iPad Tab Switcher by default.
-Remove IsTabSwitcherEnabled()
-Remove kDisableTabSwitcher kEnableTabSwitcher
-Clean the experimental flag plist.
 
Project Member

Comment 1 by bugdroid1@chromium.org, Nov 2 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/0fb9f22e1484df571ee2e776beda35ce02d0e80c

commit 0fb9f22e1484df571ee2e776beda35ce02d0e80c
Author: jif <jif@chromium.org>
Date: Wed Nov 02 09:14:50 2016

Enable the iPad Tab Switcher by default.

BUG= 659942 

Review-Url: https://codereview.chromium.org/2451813002
Cr-Commit-Position: refs/heads/master@{#429242}

[modify] https://crrev.com/0fb9f22e1484df571ee2e776beda35ce02d0e80c/ios/chrome/browser/experimental_flags.mm

Project Member

Comment 2 by bugdroid1@chromium.org, Nov 5 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/70867569e9d6952240d80befbaedf630a26cdeda

commit 70867569e9d6952240d80befbaedf630a26cdeda
Author: rohitrao <rohitrao@chromium.org>
Date: Sat Nov 05 12:47:52 2016

Revert of Enable the iPad Tab Switcher by default. (patchset #1 id:20001 of https://codereview.chromium.org/2451813002/ )

Reason for revert:
Reverting to see if this is the cause of iOS tablet test failures.

Original issue's description:
> Enable the iPad Tab Switcher by default.
>
> BUG= 659942 
>
> Committed: https://crrev.com/0fb9f22e1484df571ee2e776beda35ce02d0e80c
> Cr-Commit-Position: refs/heads/master@{#429242}

TBR=droger@chromium.org,jif@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG= 659942 

Review-Url: https://codereview.chromium.org/2480013003
Cr-Commit-Position: refs/heads/master@{#430164}

[modify] https://crrev.com/70867569e9d6952240d80befbaedf630a26cdeda/ios/chrome/browser/experimental_flags.mm

Comment 3 by jif@chromium.org, Feb 21 2017

Status: WontFix (was: Started)

Sign in to add a comment