New issue
Advanced search Search tips

Issue 883971 link

Starred by 1 user

Issue metadata

Status: WontFix
Owner:
Closed: Dec 6
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 3
Type: Bug


Show other hotlists

Hotlists containing this issue:
Launcher-Tech-Debt


Sign in to add a comment

Clean up check in app_list_item_list.cc

Project Member Reported by weidongg@chromium.org, Sep 13

Issue description

app_list_item_list.cc is filled with CHECKs that wouldn't be necessary. We should change the FindItemIndex to return an iterator. 
 
Labels: OS-Chrome
Owner: mmourgos@chromium.org
Status: Assigned (was: Untriaged)
Owner: weidongg@chromium.org
Status: WontFix (was: Assigned)
hmm, if using iterator, we still need to convert to index before making calls to observer interfaces. This would increase code complexity, so I would mark this as wontfix.

Sign in to add a comment