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

Issue 881091 link

Starred by 1 user
Status: Fixed
Owner:
█ sweilun@chromium.org
Last visit 18 days ago
Closed: Sep 10
Cc:
yyushkina@chromium.org
kristip...@chromium.org
█ ramyan@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Windows , Chrome , Mac
Pri: 3
Type: Bug



Sign in to add a comment

Add browsertest for hiding the add shortcut button at max links

Project Member Reported by kristip...@chromium.org, Sep 5

Comment 1 by ramyan@chromium.org, Sep 5

Cc: -ramyan@google.com ramyan@chromium.org

Comment 2 by kristip...@chromium.org, Sep 8

Cc: kristip...@chromium.org
Owner: sweilun@chromium.org
Project Member

Comment 3 by bugdroid1@chromium.org, Sep 10 

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

commit 0f12b02861930e3e06a5d79d612e606a0e9df34a
Author: Weilun Shi <sweilun@chromium.org>
Date: Mon Sep 10 21:25:15 2018

[NTP] Add automated test for custom links hide add button

Add button should be hidden when there are max number of most visited
tiles.

Bug:  881091 
Change-Id: I40d33e533691885204f1fd813ce077fb060678d9
Reviewed-on: https://chromium-review.googlesource.com/1214180
Commit-Queue: Weilun Shi <sweilun@chromium.org>
Reviewed-by: Kristi Park <kristipark@chromium.org>
Cr-Commit-Position: refs/heads/master@{#590068}
[modify] https://crrev.com/0f12b02861930e3e06a5d79d612e606a0e9df34a/chrome/browser/ui/search/local_ntp_browsertest.cc

Comment 4 by sweilun@chromium.org, Sep 10

Status: Fixed (was: Assigned)
►

Sign in to add a comment