Bug 43748

Summary:

[chromium] Blue line visible at the right hand side of the window with accelerated compositor

Product:

WebKit

Reporter:

Nat Duca <nduca>

Component:

WebCore Misc.

Assignee:

Nobody <webkit-unassigned>

Status:

RESOLVED FIXED

Severity:

Normal

CC:

abarth, dglazkov, eric, nduca, vangelis, webkit.review.bot

Priority:

Version:

528+ (Nightly build)

Hardware:

All

OS:

All

Attachments:

Description	Flags
Proposed fix	dglazkov: review-
Trying again, this time with no tabs :)	none

Nat Duca

Reported 2010-08-09 14:55:06 PDT

When running chrome with --enable-accelerated-compositing, evenly-sized windows will have a 1px wide blue line on the right or bottom of the screen, to the right of the scroll bar.

Attachments
Proposed fix (3.15 KB, patch) 2010-08-09 15:01 PDT, Nat Duca	dglazkov: review-	Details Formatted Diff Diff
Trying again, this time with no tabs :) (3.15 KB, patch) 2010-08-09 15:30 PDT, Nat Duca	no flags	Details Formatted Diff Diff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.

Nat Duca

Comment 1 2010-08-09 15:01:10 PDT

Created attachment 63932 [details] Proposed fix Proposed fix.

WebKit Review Bot

Comment 2 2010-08-09 15:05:26 PDT

Attachment 63932 [details] did not pass style-queue: Failed to run "['WebKitTools/Scripts/check-webkit-style']" exit_code: 1 WebCore/ChangeLog:6: Line contains tab character. [whitespace/tab] [5] Total errors found: 1 in 2 files If any of these errors are false positives, please file a bug against check-webkit-style.

Dimitri Glazkov (Google)

Comment 3 2010-08-09 15:17:52 PDT

Comment on attachment 63932 [details] Proposed fix Style elf says no. But the patch looks reasonable. Can haz no tabs in patch?

Nat Duca

Comment 4 2010-08-09 15:30:04 PDT

Created attachment 63940 [details] Trying again, this time with no tabs :)

Dimitri Glazkov (Google)

Comment 5 2010-08-09 15:40:31 PDT

Comment on attachment 63940 [details] Trying again, this time with no tabs :) ok. Don't forget to mark "commit-queue?" if you want the magic bot to land it for you.

Nat Duca

Comment 6 2010-08-09 15:45:23 PDT

Ah, will do next time. Thanks!

Eric Seidel (no email)

Comment 7 2010-08-09 17:25:34 PDT

Comment on attachment 63940 [details] Trying again, this time with no tabs :) Clearing flags on attachment: 63940 Committed r65020: <http://trac.webkit.org/changeset/65020>

Eric Seidel (no email)

Comment 8 2010-08-09 17:25:41 PDT

All reviewed patches have been landed. Closing bug.

WebKit Review Bot

Comment 9 2010-08-09 17:45:41 PDT

http://trac.webkit.org/changeset/65020 might have broken Qt Linux Release minimal The following changes are on the blame list: http://trac.webkit.org/changeset/65020 http://trac.webkit.org/changeset/65021

Note You need to log in before you can comment on or make changes to this bug.