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

Issue 660386 link

Starred by 3 users
Status: Assigned
Owner:
thegreenfrog@chromium.org
Cc:
stkhapugin@chromium.org
█ lpromero@chromium.org
█ noyau@chromium.org
ios-bugs@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: iOS
Pri: 2
Type: Bug



Sign in to add a comment

Add integration tests for favicons

Project Member Reported by stkhapugin@chromium.org, Oct 28 2016 
To prevent https://bugs.chromium.org/p/chromium/issues/detail?id=658080 in the future, we need some tests that check that:

1. favicon appear at all
2. favicons appear specifically after scrolling

Since the issue was caused by iOS, our unit tests were useless - it's not us who broke the API contracts. Thus, we need integration tests.

Comment 1 by stkhapugin@chromium.org, Nov 3 2016

Labels: -Pri-3 Pri-2

Comment 2 by stkhapugin@chromium.org, Dec 8 2017

Cc: stkhapugin@chromium.org
Owner: martiw@chromium.org

Comment 3 by martiw@chromium.org, Feb 18 2018

Owner: ----
Status: Available (was: Assigned)
Switched team and releasing this bug.

Comment 4 by sczs@chromium.org, Sep 27

Owner: thegreenfrog@chromium.org
Status: Assigned (was: Available)
thegreenfrog@ is this a duplicate from the bug we created not long ago?

Comment 5 by thegreenfrog@chromium.org, Sep 27 

I created a bug for unittests for faviconloader, which is now used for all collections UI. I guess we can keep these separate, but they both exemplify the lack of tests for favicons.
►

Sign in to add a comment