Bug 141022 – Enable WebKit disk cache on OS X

• Status: RESOLVED
• Resolution: FIXED
• Component: Page Loading

[Antti Koivisto]

Flip the switch

[Antti Koivisto]

Created attachment 245598 [details]
patch

[Build Bot]

Comment on attachment 245598 [details]
patch

Attachment 245598 [details] did not pass mac-wk2-ews (mac-wk2):
Output: http://webkit-queues.appspot.com/results/5312927921864704

Number of test failures exceeded the failure limit.

[Build Bot]

Created attachment 245603 [details]
Archive of layout-test-results from ews106 for mac-mavericks-wk2

The attached test failures were seen while running run-webkit-tests on the mac-wk2-ews.
Bot: ews106  Port: mac-mavericks-wk2  Platform: Mac OS X 10.9.5

[Darin Adler]

Comment on attachment 245598 [details]
patch

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

30 tests are failing

> Source/WebKit2/config.h:89
> +#if ENABLE(NETWORK_PROCESS)
> +#if PLATFORM(MAC)
> +#define ENABLE_NETWORK_CACHE 1
> +#endif
>  #endif

I believe what we want is:

    #if ENABLE(NETWORK_PROCESS) && PLATFORM(MAC)
    #define ENABLE_NETWORK_CACHE 1
    #else
    #define ENABLE_NETWORK_CACHE 0
    #endif

[Antti Koivisto]

Hmm, looks like Maverics is unhappy with some selector and crashes the network process:

2015-01-28 19:33:04.851 com.apple.WebKit.Networking.Development[37196:c0b] *** Terminating app due to uncaught exception 'NSInvalidArgumentException', reason: '-[OS_dispatch_data _xref_dispose]: unrecognized selector sent to instance 0x7fa02cd01ef0'
*** First throw call stack:
(
	0   CoreFoundation                      0x00007fff89d1625c __exceptionPreprocess + 172
	1   libobjc.A.dylib                     0x00007fff87f34e75 objc_exception_throw + 43
	2   CoreFoundation                      0x00007fff89d1912d -[NSObject(NSObject) doesNotRecognizeSelector:] + 205
	3   CoreFoundation                      0x00007fff89c74272 ___forwarding___ + 1010
	4   CoreFoundation                      0x00007fff89c73df8 _CF_forwarding_prep_0 + 120
	5   WebKit                              0x000000010ef81497 ___ZN6WebKit19NetworkCacheStorage5storeERKNS_15NetworkCacheKeyERKNS0_5EntryENSt3__18functionIFvbEEE_block_invoke + 547
	6   libdispatch.dylib                   0x00007fff900771bb _dispatch_call_block_and_release + 12
	7   libdispatch.dylib                   0x00007fff9007428d _dispatch_client_callout + 8
	8   libdispatch.dylib                   0x00007fff900787e3 _dispatch_async_redirect_invoke + 154
	9   libdispatch.dylib                   0x00007fff9007428d _dispatch_client_callout + 8
	10  libdispatch.dylib                   0x00007fff90076082 _dispatch_root_queue_drain + 326
	11  libdispatch.dylib                   0x00007fff90077177 _dispatch_worker_thread2 + 40
	12  libsystem_pthread.dylib             0x00007fff8b94cef8 _pthread_wqthread + 314
	13  libsystem_pthread.dylib             0x00007fff8b94ffb9 start_wqthread + 13

[Antti Koivisto]

Created attachment 245732 [details]
try again

[Build Bot]

Comment on attachment 245732 [details]
try again

Attachment 245732 [details] did not pass mac-wk2-ews (mac-wk2):
Output: http://webkit-queues.appspot.com/results/6508317715202048

Number of test failures exceeded the failure limit.

[Build Bot]

Created attachment 245738 [details]
Archive of layout-test-results from ews107 for mac-mavericks-wk2

The attached test failures were seen while running run-webkit-tests on the mac-wk2-ews.
Bot: ews107  Port: mac-mavericks-wk2  Platform: Mac OS X 10.9.5

[Antti Koivisto]

Hmm, the bot is still seeing these -[OS_dispatch_data _xref_dispose]: unrecognized selector sent to instance

[Antti Koivisto]

Created attachment 245779 [details]
try again

[Build Bot]

Comment on attachment 245779 [details]
try again

Attachment 245779 [details] did not pass mac-wk2-ews (mac-wk2):
Output: http://webkit-queues.appspot.com/results/5103764088291328

Number of test failures exceeded the failure limit.

[Build Bot]

Created attachment 245780 [details]
Archive of layout-test-results from ews105 for mac-mavericks-wk2

The attached test failures were seen while running run-webkit-tests on the mac-wk2-ews.
Bot: ews105  Port: mac-mavericks-wk2  Platform: Mac OS X 10.9.5

[Antti Koivisto]

Created attachment 245782 [details]
test

[WebKit Commit Bot]

Attachment 245782 [details] did not pass style-queue:


ERROR: Source/WebKit2/NetworkProcess/cache/NetworkCacheStorage.h:59:  Should be indented on a separate line, with the colon or comma first on that line.  [whitespace/indent] [4]
ERROR: Source/WebKit2/NetworkProcess/cache/NetworkCacheStorage.h:72:  Should be indented on a separate line, with the colon or comma first on that line.  [whitespace/indent] [4]
Total errors found: 2 in 5 files


If any of these errors are false positives, please file a bug against check-webkit-style.

[Antti Koivisto]

Created attachment 245783 [details]
test2

[Antti Koivisto]

Ok, replacing OSObjectPtr with DispatchPtr works. The fixed OSObjectPtr apparently still doesn't work on Mavericks setup used in bots.

[Antti Koivisto]

Created attachment 245794 [details]
patch

[Antti Koivisto]

Reverted r179447 for reason:

Forgot

Committed r179448: <http://trac.webkit.org/changeset/179448>

[Antti Koivisto]

https://webkit.org/b/141022

[Antti Koivisto]

https://trac.webkit.org/r179449