Bug 188703

Summary: Start bringing up Unified Sources in WebKit2
Product: WebKit Reporter: Tim Horton <thorton>
Component: New BugsAssignee: Tim Horton <thorton>
Status: RESOLVED FIXED    
Severity: Normal CC: commit-queue, ggaren, keith_miller, simon.fraser, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch
none
Patch
none
Patch
none
Patch none

Tim Horton
Reported 2018-08-17 11:48:16 PDT
Start bringing up Unified Sources in WebKit2
Attachments
Patch (563.33 KB, patch)
2018-08-17 11:49 PDT, Tim Horton 		no flags
DetailsFormatted DiffDiff
Patch (342.67 KB, patch)
2018-08-17 12:17 PDT, Tim Horton 		no flags
DetailsFormatted DiffDiff
Patch (364.58 KB, patch)
2018-08-17 15:43 PDT, Tim Horton 		no flags
DetailsFormatted DiffDiff
Patch (364.89 KB, patch)
2018-08-17 16:15 PDT, Tim Horton 		no flags
DetailsFormatted DiffDiff
Show Obsolete (3) View All Add attachment proposed patch, testcase, etc.
Tim Horton
Comment 1 2018-08-17 11:49:13 PDT
Created attachment 347375 [details] Patch
Simon Fraser (smfr)
Comment 2 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.
Tim Horton
Comment 3 2018-08-17 12:17:33 PDT
Created attachment 347377 [details] Patch
Simon Fraser (smfr)
Comment 4 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.
Tim Horton
Comment 5 2018-08-17 15:43:06 PDT
Created attachment 347400 [details] Patch
Tim Horton
Comment 6 2018-08-17 16:15:36 PDT
Created attachment 347409 [details] Patch
WebKit Commit Bot
Comment 7 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>
WebKit Commit Bot
Comment 8 2018-08-17 18:31:24 PDT
All reviewed patches have been landed. Closing bug.
Radar WebKit Bug Importer
Comment 9 2018-08-17 18:33:31 PDT
<rdar://problem/43448839>
Keith Miller
Comment 10 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.
Note You need to log in before you can comment on or make changes to this bug.