Bug 188703 - Start bringing up Unified Sources in WebKit2
Summary: Start bringing up Unified Sources in WebKit2
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: New Bugs (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Tim Horton
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2018-08-17 11:48 PDT by Tim Horton
Modified: 2018-08-18 03:52 PDT (History)
5 users (show)

See Also:


Attachments
Patch (563.33 KB, patch)
2018-08-17 11:49 PDT, Tim Horton
no flags Details | Formatted Diff | Diff
Patch (342.67 KB, patch)
2018-08-17 12:17 PDT, Tim Horton
no flags Details | Formatted Diff | Diff
Patch (364.58 KB, patch)
2018-08-17 15:43 PDT, Tim Horton
no flags Details | Formatted Diff | Diff
Patch (364.89 KB, patch)
2018-08-17 16:15 PDT, Tim Horton
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Tim Horton 2018-08-17 11:48:16 PDT
Start bringing up Unified Sources in WebKit2
Comment 1 Tim Horton 2018-08-17 11:49:13 PDT
Created attachment 347375 [details]
Patch
Comment 2 Simon Fraser (smfr) 2018-08-17 11:55:15 PDT
Comment on attachment 347375 [details]
Patch

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

> Source/WebKit/WebKit.xcodeproj/project.pbxproj:1182
> +		2D11B9922126A286006F8878 /* UnifiedSource517.cpp in Sources */ = {isa = PBXBuildFile; fileRef = 2D7DEDD121269E4D00B9F73C /* UnifiedSource517.cpp */; };
> +		2D11B9932126A286006F8878 /* UnifiedSource518.cpp in Sources */ = {isa = PBXBuildFile; fileRef = 2D7DEC5D21269E4C00B9F73C /* UnifiedSource518.cpp */; };
> +		2D11B9942126A286006F8878 /* UnifiedSource519.cpp in Sources */ = {isa = PBXBuildFile; fileRef = 2D7DEC7D21269E4C00B9F73C /* UnifiedSource519.cpp */; };
> +		2D11B9952126A286006F8878 /* UnifiedSource520.cpp in Sources */ = {isa = PBXBuildFile; fileRef = 2D7DEC0621269E4B00B9F73C /* UnifiedSource520.cpp */; };
> +		2D11B9962126A286006F8878 /* UnifiedSource521.cpp in Sources */ = {isa = PBXBuildFile; fileRef = 2D7DEBE521269E4B00B9F73C /* UnifiedSource521.cpp */; };
> +		2D11B9972126A286006F8878 /* UnifiedSource522.cpp in Sources */ = {isa = PBXBuildFile; fileRef = 2D7DEC0C21269E4B00B9F73C /* UnifiedSource522.cpp */; };
> +		2D11B9982126A286006F8878 /* UnifiedSource523.cpp in Sources */ = {isa = PBXBuildFile; fileRef = 2D7DEC2E21269E4B00B9F73C /* UnifiedSource523.cpp */; };
> +		2D11B9992126A286006F8878 /* UnifiedSource524.cpp in Sources */ = {isa = PBXBuildFile; fileRef = 2D7DEC5C21269E4C00B9F73C /* UnifiedSource524.cpp */; };
> +		2D11B99A2126A286006F8878 /* UnifiedSource525.cpp in Sources */ = {isa = PBXBuildFile; fileRef = 2D7DEC7F21269E4C00B9F73C /* UnifiedSource525.cpp */; };
> +		2D11B99B2126A286006F8878 /* UnifiedSource526.cpp in Sources */ = {isa = PBXBuildFile; fileRef = 2D7DEC5721269E4B00B9F73C /* UnifiedSource526.cpp */; };
> +		2D11B99C2126A286006F8878 /* UnifiedSource527.cpp in Sources */ = {isa = PBXBuildFile; fileRef = 2D7DEC3021269E4B00B9F73C /* UnifiedSource527.cpp */; };
> +		2D11B99D2126A286006F8878 /* UnifiedSource528.cpp in Sources */ = {isa = PBXBuildFile; fileRef = 2D7DEDAD21269E4D00B9F73C /* UnifiedSource528.cpp */; };
> +		2D11B99E2126A286006F8878 /* UnifiedSource529.cpp in Sources */ = {isa = PBXBuildFile; fileRef = 2D7DEDCD21269E4D00B9F73C /* UnifiedSource529.cpp */; };
> +		2D11B99F2126A286006F8878 /* UnifiedSource530.cpp in Sources */ = {isa = PBXBuildFile; fileRef = 2D7DEC5921269E4B00B9F73C /* UnifiedSource530.cpp */; };

Tooooo many. Please guesstimate.
Comment 3 Tim Horton 2018-08-17 12:17:33 PDT
Created attachment 347377 [details]
Patch
Comment 4 Simon Fraser (smfr) 2018-08-17 13:16:40 PDT
Comment on attachment 347377 [details]
Patch

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

> Source/WebKit/WebKit.xcodeproj/project.pbxproj:11500
> +				2D11B7512126A282006F8878 /* UnifiedSource1-mm.mm in Sources */,

I wish these were called UnifiedSourceObjC-1.mm or something.
Comment 5 Tim Horton 2018-08-17 15:43:06 PDT
Created attachment 347400 [details]
Patch
Comment 6 Tim Horton 2018-08-17 16:15:36 PDT
Created attachment 347409 [details]
Patch
Comment 7 WebKit Commit Bot 2018-08-17 18:31:23 PDT
Comment on attachment 347409 [details]
Patch

Clearing flags on attachment: 347409

Committed r235006: <https://trac.webkit.org/changeset/235006>
Comment 8 WebKit Commit Bot 2018-08-17 18:31:24 PDT
All reviewed patches have been landed.  Closing bug.
Comment 9 Radar WebKit Bug Importer 2018-08-17 18:33:31 PDT
<rdar://problem/43448839>
Comment 10 Keith Miller 2018-08-18 03:52:23 PDT
Comment on attachment 347409 [details]
Patch

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

> Source/WebKit/Shared/RemoteLayerTree/RemoteLayerBackingStore.mm:-292
> -    FloatSize scaledSize = m_size;

When I ran into functions that heavily referenced types from JSC in WebCore I would just do:

using namespace JSC; 

on the first line of the function.