Bug 146327

Summary: Get rid of ScrollbarThemeClient now that it's unused
Product: WebKit Reporter: Anders Carlsson <andersca>
Component: New BugsAssignee: Anders Carlsson <andersca>
Status: RESOLVED FIXED    
Severity: Normal CC: bdakin, buildbot, commit-queue, esprehn+autocc, glenn, kondapallykalyan, ossy, rniwa
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Bug Depends on: 146342    
Bug Blocks:    
Attachments:
Description Flags
Patch
none
Archive of layout-test-results from ews106 for mac-mavericks-wk2
none
Patch
none
Archive of layout-test-results from ews105 for mac-mavericks-wk2
none
Patch bdakin: review+

Description Anders Carlsson 2015-06-25 15:45:00 PDT
Get rid of ScrollbarThemeClient now that it's unused
Comment 1 Anders Carlsson 2015-06-25 15:46:17 PDT
Created attachment 255581 [details]
Patch
Comment 2 Build Bot 2015-06-25 16:34:51 PDT
Comment on attachment 255581 [details]
Patch

Attachment 255581 [details] did not pass mac-wk2-ews (mac-wk2):
Output: http://webkit-queues.appspot.com/results/5075377353392128

New failing tests:
scrollbars/custom-scrollbars-paint-outside-iframe.html
fast/repaint/fixed-right-in-page-scale.html
fast/repaint/fixed-move-after-keyboard-scroll.html
scrollbars/scrollbar-iframe-click-does-not-blur-content.html
fast/css/scrollbar-dynamic-style-change.html
fast/frames/unload-reparent-sibling-frame.html
scrollbars/scrollbar-scrollbarparts-repaint-crash.html
fast/events/scale-and-scroll-iframe-body.html
fast/events/scale-and-scroll-window.html
scrollbars/scrollbar-drag-thumb-with-large-content.html
Comment 3 Build Bot 2015-06-25 16:34:54 PDT
Created attachment 255595 [details]
Archive of layout-test-results from ews106 for mac-mavericks-wk2

The attached test failures were seen while running run-webkit-tests on the mac-wk2-ews.
Bot: ews106  Port: mac-mavericks-wk2  Platform: Mac OS X 10.9.5
Comment 4 Csaba Osztrogonác 2015-06-26 03:53:25 PDT
(In reply to comment #1)
> Created attachment 255581 [details]
> Patch

I fixed the EFL build in bug146342, please don't 
land this patchbefore bug146342. Thanks.
Comment 5 Anders Carlsson 2015-06-26 10:57:37 PDT
Created attachment 255648 [details]
Patch
Comment 6 Build Bot 2015-06-26 11:45:47 PDT
Comment on attachment 255648 [details]
Patch

Attachment 255648 [details] did not pass mac-wk2-ews (mac-wk2):
Output: http://webkit-queues.appspot.com/results/5647686845857792

New failing tests:
scrollbars/custom-scrollbars-paint-outside-iframe.html
fast/repaint/fixed-right-in-page-scale.html
fast/repaint/fixed-move-after-keyboard-scroll.html
scrollbars/scrollbar-iframe-click-does-not-blur-content.html
fast/css/scrollbar-dynamic-style-change.html
fast/frames/unload-reparent-sibling-frame.html
scrollbars/scrollbar-scrollbarparts-repaint-crash.html
fast/events/scale-and-scroll-iframe-body.html
fast/events/scale-and-scroll-window.html
scrollbars/scrollbar-drag-thumb-with-large-content.html
Comment 7 Build Bot 2015-06-26 11:45:49 PDT
Created attachment 255654 [details]
Archive of layout-test-results from ews105 for mac-mavericks-wk2

The attached test failures were seen while running run-webkit-tests on the mac-wk2-ews.
Bot: ews105  Port: mac-mavericks-wk2  Platform: Mac OS X 10.9.5
Comment 8 Anders Carlsson 2015-06-26 12:29:08 PDT
Created attachment 255658 [details]
Patch
Comment 9 Anders Carlsson 2015-06-26 13:43:28 PDT
Committed r186007: <http://trac.webkit.org/changeset/186007>