Bug 77670

Summary:

The overhang area layer should have a linen background

Product:

WebKit

Reporter:

Anders Carlsson <andersca>

Component:

New Bugs

Assignee:

Anders Carlsson <andersca>

Status:

RESOLVED FIXED

Severity:

Normal

Priority:

Version:

528+ (Nightly build)

Hardware:

Unspecified

OS:

Unspecified

Attachments:

Description	Flags
Patch	kling: review+

Description Anders Carlsson 2012-02-02 12:52:24 PST

The overhang area layer should have a linen background

Comment 1 Anders Carlsson 2012-02-02 12:54:23 PST

Created attachment 125166 [details]
Patch

Comment 2 Andreas Kling 2012-02-02 13:10:46 PST

Comment on attachment 125166 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=125166&action=review

> Source/WebCore/page/scrolling/mac/ScrollingCoordinatorMac.mm:94
> +#if !DISABLE_SCROLL_POSITION_CLAMPING

Double negatives aren't not harder to read.

> Source/WebCore/platform/mac/ScrollbarThemeMac.h:85
> +    virtual void setUpOverhangAreasLayerContents(GraphicsLayer*) override;

override -> OVERRIDE

> Source/WebCore/platform/mac/ScrollbarThemeMac.mm:441
> +

Unnecessary new whitespace.

Comment 3 Anders Carlsson 2012-02-02 13:19:06 PST

Committed r106588: <http://trac.webkit.org/changeset/106588>