diff options
author | mukai@chromium.org <mukai@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2013-03-20 21:09:40 +0000 |
---|---|---|
committer | mukai@chromium.org <mukai@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2013-03-20 21:09:40 +0000 |
commit | 53d90b1b59f79b025b3dc2fe7b0188979efb61ea (patch) | |
tree | 226aad1da7253a18699a78e312afa55221e40c37 /tools | |
parent | dff794d33644871dee662e20ea57faae8693b8a6 (diff) | |
download | chromium_src-53d90b1b59f79b025b3dc2fe7b0188979efb61ea.zip chromium_src-53d90b1b59f79b025b3dc2fe7b0188979efb61ea.tar.gz chromium_src-53d90b1b59f79b025b3dc2fe7b0188979efb61ea.tar.bz2 |
Revert 189384 "Revert 189289 "Fix notification settings dialog v..."
Actually this is not the real cause of the failure. See crbug.com/222500
for the details.
> Revert 189289 "Fix notification settings dialog visuals."
>
> This CL may have caused persistent failures in interactive_ui_tests.
> The errors can be seen here:
> http://build.chromium.org/p/chromium.win/builders/Interactive%20Tests%20%28dbg%29/builds/35517/steps/interactive_ui_tests/logs/ManyMessageCenterNotifications
>
> While the CL doesn't appear to be directly related, it's being reverted to see
> if this clears up the problem.
>
> > Fix notification settings dialog visuals.
> >
> > This CL does the following things:
> > - do not shrink the contents of the scroll view, instead enlarge the
> > window if there is a scroll bar.
> > - set an empty insets for separator, otherwise a scrollbar appears
> > when the total height of the contents is exactly same as the expected
> > window height.
> > - fix the length of the separator.
> >
> > BUG=161094
> >
> > Review URL: https://codereview.chromium.org/12566037
>
> TBR=mukai@chromium.org
> Review URL: https://codereview.chromium.org/12812024
TBR=xusydoc@chromium.org
Review URL: https://codereview.chromium.org/12964005
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@189402 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'tools')
0 files changed, 0 insertions, 0 deletions