Bug 188881

Summary: [CoordGraphics] Remove the remaining CoordinatedGraphicsLayerState cruft
Product: WebKit Reporter: Zan Dobersek <zan>
Component: New BugsAssignee: Zan Dobersek <zan>
Status: RESOLVED FIXED    
Severity: Normal CC: cgarcia, ews-watchlist, magomez, mcatanzaro, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
See Also: https://bugs.webkit.org/show_bug.cgi?id=188916
Bug Depends on:    
Bug Blocks: 187385    
Attachments:
Description Flags
Patch
none
Patch none

Description Zan Dobersek 2018-08-23 00:21:49 PDT
[CoordGraphics] Remove the remaining CoordinatedGraphicsLayerState cruft
Comment 1 Zan Dobersek 2018-08-23 00:33:11 PDT
Created attachment 347915 [details]
Patch
Comment 2 EWS Watchlist 2018-08-23 00:34:52 PDT
Attachment 347915 [details] did not pass style-queue:


ERROR: Source/WebCore/platform/graphics/texmap/coordinated/CoordinatedGraphicsLayer.h:114:  Inline functions should not be in classes annotated with WEBCORE_EXPORT. Remove the macro from the class and apply it to each appropriate method, or move the inline function definition out-of-line.  [build/webcore_export] [4]
Total errors found: 1 in 9 files


If any of these errors are false positives, please file a bug against check-webkit-style.
Comment 3 Zan Dobersek 2018-08-23 00:39:09 PDT
Created attachment 347916 [details]
Patch
Comment 4 EWS Watchlist 2018-08-23 00:40:44 PDT
Attachment 347916 [details] did not pass style-queue:


ERROR: Source/WebCore/platform/graphics/texmap/coordinated/CoordinatedGraphicsLayer.h:114:  Inline functions should not be in classes annotated with WEBCORE_EXPORT. Remove the macro from the class and apply it to each appropriate method, or move the inline function definition out-of-line.  [build/webcore_export] [4]
Total errors found: 1 in 9 files


If any of these errors are false positives, please file a bug against check-webkit-style.
Comment 5 Zan Dobersek 2018-08-23 05:34:39 PDT
Comment on attachment 347916 [details]
Patch

Clearing flags on attachment: 347916

Committed r235221: <https://trac.webkit.org/changeset/235221>
Comment 6 Zan Dobersek 2018-08-23 05:34:42 PDT
All reviewed patches have been landed.  Closing bug.
Comment 7 Radar WebKit Bug Importer 2018-08-23 05:35:23 PDT
<rdar://problem/43640432>
Comment 8 Michael Catanzaro 2018-08-23 07:52:34 PDT
I think you need to fix this:

(In reply to Build Bot from comment #4) 
> ERROR:
> Source/WebCore/platform/graphics/texmap/coordinated/CoordinatedGraphicsLayer.
> h:114:  Inline functions should not be in classes annotated with
> WEBCORE_EXPORT. Remove the macro from the class and apply it to each
> appropriate method, or move the inline function definition out-of-line. 
> [build/webcore_export] [4]
> Total errors found: 1 in 9 files
Comment 9 Zan Dobersek 2018-08-23 23:23:34 PDT
(In reply to Michael Catanzaro from comment #8)
> I think you need to fix this:
> 
> (In reply to Build Bot from comment #4) 
> > ERROR:
> > Source/WebCore/platform/graphics/texmap/coordinated/CoordinatedGraphicsLayer.
> > h:114:  Inline functions should not be in classes annotated with
> > WEBCORE_EXPORT. Remove the macro from the class and apply it to each
> > appropriate method, or move the inline function definition out-of-line. 
> > [build/webcore_export] [4]
> > Total errors found: 1 in 9 files

Bug #188916.