New issue
Advanced search Search tips

Issue 857610 link

Starred by 3 users

Issue metadata

Status: Fixed
Owner:
Closed: Jun 2018
Cc:
Components:
EstimatedDays: 1
NextAction: ----
OS: Linux , Windows , Chrome , Mac
Pri: 1
Type: Bug

Blocking:
issue 822063


Show other hotlists

Hotlists containing this issue:
Hotlist-1


Sign in to add a comment

Default new tab button to AFTER_TABS

Project Member Reported by pkasting@chromium.org, Jun 28 2018

Issue description

Per latest UX lead decision
 
EstimatedDays: 1
Labels: Hotlist-Helper
Status: Started (was: Assigned)

Comment 3 Deleted

Project Member

Comment 5 by bugdroid1@chromium.org, Jun 30 2018

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

commit 630979171b981ffefd07c1e31036f322b6cb03f1
Author: Peter Kasting <pkasting@chromium.org>
Date: Sat Jun 30 02:09:03 2018

Change NTB default position to AFTER_TABS, add flags, remove Finch control.

I'm not opposed to leaving this under Finch control for future experimentation,
but I don't really know how to use FEATURE_WITH_PARAMS_VALUE.

Bug:  857610 ,  857282 ,  855729 
Change-Id: Ibe1b9c13003f2da78b241e775f37b2e9ab7e71f7
Reviewed-on: https://chromium-review.googlesource.com/1119362
Commit-Queue: Peter Kasting <pkasting@chromium.org>
Reviewed-by: Bret Sepulveda <bsep@chromium.org>
Cr-Commit-Position: refs/heads/master@{#571739}
[modify] https://crrev.com/630979171b981ffefd07c1e31036f322b6cb03f1/chrome/browser/about_flags.cc
[modify] https://crrev.com/630979171b981ffefd07c1e31036f322b6cb03f1/chrome/browser/flag_descriptions.cc
[modify] https://crrev.com/630979171b981ffefd07c1e31036f322b6cb03f1/chrome/browser/flag_descriptions.h
[modify] https://crrev.com/630979171b981ffefd07c1e31036f322b6cb03f1/chrome/browser/ui/views/tabs/browser_tab_strip_controller.cc
[modify] https://crrev.com/630979171b981ffefd07c1e31036f322b6cb03f1/chrome/browser/ui/views/tabs/tab_drag_controller_interactive_uitest.cc
[modify] https://crrev.com/630979171b981ffefd07c1e31036f322b6cb03f1/chrome/common/chrome_switches.cc
[modify] https://crrev.com/630979171b981ffefd07c1e31036f322b6cb03f1/chrome/common/chrome_switches.h
[modify] https://crrev.com/630979171b981ffefd07c1e31036f322b6cb03f1/tools/metrics/histograms/enums.xml

Status: Fixed (was: Started)

Sign in to add a comment