Bug 173976

Summary: Slight clean up of WebResourceLoadStatisticsStore
Product: WebKit Reporter: Chris Dumez <cdumez>
Component: WebKit2Assignee: Chris Dumez <cdumez>
Status: RESOLVED FIXED    
Severity: Normal CC: bfulgham, commit-queue, ggaren, sam, wilander
Priority: P2    
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch
none
Patch
none
Patch none

Description Chris Dumez 2017-06-29 10:53:51 PDT
Slight clean up of WebResourceLoadStatisticsStore.
Comment 1 Chris Dumez 2017-06-29 10:58:36 PDT
Created attachment 314143 [details]
Patch
Comment 2 Chris Dumez 2017-06-29 11:13:05 PDT
Created attachment 314145 [details]
Patch
Comment 3 Brent Fulgham 2017-06-29 11:48:50 PDT
Comment on attachment 314145 [details]
Patch

r=me. I'm not sure why the WPE bot is red though. Doesn't seem like your changes should have affected it.
Comment 4 Chris Dumez 2017-06-29 11:49:59 PDT
(In reply to Brent Fulgham from comment #3)
> Comment on attachment 314145 [details]
> Patch
> 
> r=me. I'm not sure why the WPE bot is red though. Doesn't seem like your
> changes should have affected it.

ource/WebKit2/CMakeFiles/WebKit2.dir/UIProcess/WebResourceLoadStatisticsStore.cpp.o.d -o Source/WebKit2/CMakeFiles/WebKit2.dir/UIProcess/WebResourceLoadStatisticsStore.cpp.o -c /home/ews/igalia-wpe-ews/WebKit/Source/WebKit2/UIProcess/WebResourceLoadStatisticsStore.cpp
/home/ews/igalia-wpe-ews/WebKit/Source/WebKit2/UIProcess/WebResourceLoadStatisticsStore.cpp: In function 'const WTF::OptionSet<WebKit::WebsiteDataType>& WebKit::dataTypesToRemove()':
/home/ews/igalia-wpe-ews/WebKit/Source/WebKit2/UIProcess/WebResourceLoadStatisticsStore.cpp:62:103: error: variable 'WTF::NeverDestroyed<WTF::OptionSet<WebKit::WebsiteDataType> > dataTypes' has initializer but incomplete type

Will fix before landing.
Comment 5 Chris Dumez 2017-06-29 11:52:58 PDT
Created attachment 314148 [details]
Patch
Comment 6 WebKit Commit Bot 2017-06-29 12:51:09 PDT
Comment on attachment 314148 [details]
Patch

Clearing flags on attachment: 314148

Committed r218958: <http://trac.webkit.org/changeset/218958>
Comment 7 WebKit Commit Bot 2017-06-29 12:51:10 PDT
All reviewed patches have been landed.  Closing bug.