Bug 141104 – Access MemoryCache singleton using MemoryCache::singleton()

• Component: Page Loading
• Status: RESOLVED FIXED

[Chris Dumez]

Access MemoryCache singleton using MemoryCache::singleton() static member function, instead of a free function, as per the recent coding style discussion on WebKit-dev.

[Chris Dumez]

Created attachment 245757 [details]
Patch

[Chris Dumez]

The win-ews does not seem to give a compilation error. Not sure why it failed.

[Chris Dumez]

ping review?

[Andreas Kling]

Comment on attachment 245757 [details]
Patch

r=me

[WebKit Commit Bot]

Comment on attachment 245757 [details]
Patch

Clearing flags on attachment: 245757

Committed r179489: <http://trac.webkit.org/changeset/179489>

[WebKit Commit Bot]

All reviewed patches have been landed.  Closing bug.

[Brent Fulgham]

This broke the Windows build, as was reported in <https://webkit-queues.appspot.com/results/5739814385090560> two days ago (at the same time as the other EWS bots reported their status).

Fix landed in <http://trac.webkit.org/changeset/179498>.

[Chris Dumez]

(In reply to comment #7)
> This broke the Windows build, as was reported in
> <https://webkit-queues.appspot.com/results/5739814385090560> two days ago
> (at the same time as the other EWS bots reported their status).
> 
> Fix landed in <http://trac.webkit.org/changeset/179498>.

Sorry Brent. I fails to see the missing symbol error in the output and thus did not understand why the win-ews was failing. BTW, I don't know if we can do something about this I always have trouble finding the build errors on the win-ews stdio. Most bots would give out the errors at the end but win-ews doesn't. I usually search for "error C" on win-ews, but this apparently only works for build errors, not linking ones.

[Brent Fulgham]

(In reply to comment #8)
> (In reply to comment #7)
> > This broke the Windows build, as was reported in
> > <https://webkit-queues.appspot.com/results/5739814385090560> two days ago
> > (at the same time as the other EWS bots reported their status).
> > 
> > Fix landed in <http://trac.webkit.org/changeset/179498>.
> 
> Sorry Brent. I fails to see the missing symbol error in the output and thus
> did not understand why the win-ews was failing. BTW, I don't know if we can
> do something about this I always have trouble finding the build errors on
> the win-ews stdio. Most bots would give out the errors at the end but
> win-ews doesn't. I usually search for "error C" on win-ews, but this
> apparently only works for build errors, not linking ones.

Yes -- I think these show up as "error LNK0123", so I search for "error L" if I don't find something under "error C".

Darin has encountered a case where neither error is shown in the EWS log because of the massive amounts of output produced.

We have a filtering script that is used on our internal bots, but it hasn't been migrated to the OpenSource or EWS bots for reasons that I don't fully understand.

[Chris Dumez]

(In reply to comment #9)
> (In reply to comment #8)
> > (In reply to comment #7)
> > > This broke the Windows build, as was reported in
> > > <https://webkit-queues.appspot.com/results/5739814385090560> two days ago
> > > (at the same time as the other EWS bots reported their status).
> > > 
> > > Fix landed in <http://trac.webkit.org/changeset/179498>.
> > 
> > Sorry Brent. I fails to see the missing symbol error in the output and thus
> > did not understand why the win-ews was failing. BTW, I don't know if we can
> > do something about this I always have trouble finding the build errors on
> > the win-ews stdio. Most bots would give out the errors at the end but
> > win-ews doesn't. I usually search for "error C" on win-ews, but this
> > apparently only works for build errors, not linking ones.
> 
> Yes -- I think these show up as "error LNK0123", so I search for "error L"
> if I don't find something under "error C".

Good to know the "error L" one. I'll do this next time if "error C" doesn't work. Thanks.

BTW, if the windows-ews is reliable enough these days, we could probably make the commit queue depend on it succeeding to build. Currently, I think the CQ stops only if mac-ews fails.