[WK1 Mac] Fix repaints of fixed-background elements in layer-backed WebViews
Created attachment 290980 [details] Patch
Attachment 290980 [details] did not pass style-queue: ERROR: Source/WebKit/mac/WebView/WebClipView.h:0: Use #pragma once header guard. [build/header_guard] [5] Total errors found: 1 in 4 files If any of these errors are false positives, please file a bug against check-webkit-style.
Created attachment 290981 [details] Patch
Attachment 290981 [details] did not pass style-queue: ERROR: Source/WebKit/mac/WebView/WebClipView.h:0: Use #pragma once header guard. [build/header_guard] [5] Total errors found: 1 in 4 files If any of these errors are false positives, please file a bug against check-webkit-style.
Comment on attachment 290980 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=290980&action=review > Source/WebKit/mac/WebView/WebClipView.mm:51 > @interface NSClipView (WebNSClipViewDetails) Nothing in the () maybe. > Source/WebKit/mac/WebView/WebHTMLView.mm:4115 > - if (_private->inScrollPositionChanged) { > + if (_private->inScrollPositionChanged && [(WebClipView *)[self superview] currentScrollIsBlit]) { Maybe check to be safe?
Created attachment 290986 [details] Patch
Attachment 290986 [details] did not pass style-queue: ERROR: Source/WebKit/mac/WebView/WebHTMLView.mm:4122: Multi line control clauses should use braces. [whitespace/braces] [4] ERROR: Source/WebKit/mac/WebView/WebClipView.h:0: Use #pragma once header guard. [build/header_guard] [5] Total errors found: 2 in 4 files If any of these errors are false positives, please file a bug against check-webkit-style.
Comment on attachment 290986 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=290986&action=review > Source/WebKit/mac/WebView/WebHTMLView.mm:4122 > - if (_private->inScrollPositionChanged) { > + if (_private->inScrollPositionChanged Do not love this wrapping.
https://trac.webkit.org/r206945