Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(115)

Issue 503693002: Remove the search icon from the search box in the experimental app list. (Closed)

Created:
6 years, 4 months ago by calamity
Modified:
6 years, 3 months ago
Reviewers:
tapted
CC:
chromium-reviews, chrome-apps-syd-reviews_chromium.org, tfarina
Project:
chromium
Visibility:
Public.

Description

Remove the search icon from the search box in the experimental app list. This CL removes the magnifying glass icon from the experimental app list. The normal app list should be unaffected BUG=406222 Committed: https://crrev.com/88de91114cd40aebdbc548bef10827d62f012ff4 Cr-Commit-Position: refs/heads/master@{#292569}

Patch Set 1 : #

Patch Set 2 : #

Patch Set 3 : rebase #

Unified diffs Side-by-side diffs Delta from patch set Stats (+8 lines, -4 lines) Patch
M ui/app_list/views/search_box_view.cc View 1 2 2 chunks +8 lines, -4 lines 0 comments Download

Messages

Total messages: 8 (0 generated)
calamity
calamity@chromium.org changed reviewers: + tapted@chromium.org
6 years, 3 months ago (2014-08-28 02:52:33 UTC) #1
calamity
Screenie on bug.
6 years, 3 months ago (2014-08-28 02:52:33 UTC) #2
tapted
lgtm
6 years, 3 months ago (2014-08-28 03:04:24 UTC) #3
calamity
Patchset #4 (id:100001) has been deleted
6 years, 3 months ago (2014-08-28 04:51:01 UTC) #4
calamity
The CQ bit was checked by calamity@chromium.org
6 years, 3 months ago (2014-08-29 03:08:42 UTC) #5
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/calamity@chromium.org/503693002/80001
6 years, 3 months ago (2014-08-29 03:09:27 UTC) #6
commit-bot: I haz the power
Committed patchset #3 (id:80001) as 76e5237b70844cdfa4f2310e7609b9107cf1a47d
6 years, 3 months ago (2014-08-29 04:04:34 UTC) #7
commit-bot: I haz the power
6 years, 3 months ago (2014-09-10 03:06:11 UTC) #8
Message was sent while issue was closed.
Patchset 3 (id:??) landed as
https://crrev.com/88de91114cd40aebdbc548bef10827d62f012ff4
Cr-Commit-Position: refs/heads/master@{#292569}

Powered by Google App Engine
This is Rietveld 408576698