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

Issue 2324613002: Bug Fix: Desktop Picker Window Shows Unnecessary Scrollbar [M54] (Closed)

Created:
4 years, 3 months ago by qiangchen
Modified:
4 years, 3 months ago
Reviewers:
CC:
chromium-reviews, tfarina
Target Ref:
refs/pending/branch-heads/2840
Project:
chromium
Visibility:
Public.

Description

Bug Fix: Desktop Picker Window Shows Unnecessary Scrollbar [M54] When the number of source items just exactly fill the container, we displayed the scroll bar. In this CL, we fixed the issue, and not displayed scroll bar in that case. BUG=636338 TBR=msw@chromium.org,sky@chromium.org NOPRESUBMIT=TRUE NOTRY=TRUE Review-Url: https://codereview.chromium.org/2304743002 Cr-Commit-Position: refs/heads/master@{#416763} (cherry picked from commit 6d191b6ae63639d0c55046ef4bc909726d25a30c)

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+8 lines, -0 lines) Patch
M chrome/browser/ui/views/desktop_capture/desktop_media_list_view.cc View 1 chunk +8 lines, -0 lines 0 comments Download

Messages

Total messages: 5 (3 generated)
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.org/2324613002/1
4 years, 3 months ago (2016-09-07 22:44:04 UTC) #3
commit-bot: I haz the power
4 years, 3 months ago (2016-09-07 22:48:07 UTC) #5
Message was sent while issue was closed.
Committed patchset #1 (id:1)

Powered by Google App Engine
This is Rietveld 408576698