Bug 193303

Summary: Remove the UIProcess components of the ResourceLoadStatistics code
Product: WebKit Reporter: Brent Fulgham <bfulgham>
Component: WebKit Misc.Assignee: Brent Fulgham <bfulgham>
Status: RESOLVED FIXED    
Severity: Normal CC: achristensen, bfulgham, cdumez, webkit-bug-importer, wilander
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Bug Depends on: 193297, 193556, 193659    
Bug Blocks: 193199, 193705    
Attachments:
Description Flags
Patch cdumez: review+, cdumez: commit-queue-

Brent Fulgham
Reported 2019-01-09 14:55:24 PST
Once Bug 193297 is landed, we should remove the old UIProcess code that used to do the same work.
Attachments
Patch (83.24 KB, patch)
2019-01-25 23:09 PST, Brent Fulgham 		cdumez: review+
cdumez: commit-queue-
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Radar WebKit Bug Importer
Comment 1 2019-01-09 14:55:45 PST
<rdar://problem/47160073>
Brent Fulgham
Comment 2 2019-01-25 23:09:25 PST
Created attachment 360229 [details] Patch
Chris Dumez
Comment 3 2019-01-28 08:52:44 PST
Comment on attachment 360229 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=360229&action=review r=me > Source/WebKit/NetworkProcess/Classifier/ResourceLoadStatisticsMemoryStore.cpp:2 > + * Copyright (C) 2017-2019Ã Apple Inc. All rights reserved. Bad character.
Brent Fulgham
Comment 4 2019-01-28 09:29:51 PST
Committed r240586: <https://trac.webkit.org/changeset/240586>
Brent Fulgham
Comment 5 2019-01-28 09:30:03 PST
(In reply to Chris Dumez from comment #3) > Comment on attachment 360229 [details] > Patch > > View in context: > https://bugs.webkit.org/attachment.cgi?id=360229&action=review > > r=me > > > Source/WebKit/NetworkProcess/Classifier/ResourceLoadStatisticsMemoryStore.cpp:2 > > + * Copyright (C) 2017-2019Ã Apple Inc. All rights reserved. > > Bad character. Fixed while landing.
Alex Christensen
Comment 6 2019-01-28 11:29:13 PST
Comment on attachment 360229 [details] Patch This is the most beautiful patch I've seen in a while.
Brent Fulgham
Comment 7 2019-01-28 11:34:03 PST
(In reply to Alex Christensen from comment #6) > Comment on attachment 360229 [details] > Patch > > This is the most beautiful patch I've seen in a while. lol. I notice I did manage to get an accidental whitespace change in. :-(
Note You need to log in before you can comment on or make changes to this bug.