Bug 181764 - Deprecate Application Cache
Summary: Deprecate Application Cache
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: New Bugs (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Alex Christensen
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2018-01-17 14:06 PST by Alex Christensen
Modified: 2018-01-17 16:22 PST (History)
4 users (show)

See Also:


Attachments
Patch (1.32 KB, patch)
2018-01-17 14:17 PST, Alex Christensen
ggaren: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Alex Christensen 2018-01-17 14:06:34 PST
Deprecate Application Cache
Comment 1 Alex Christensen 2018-01-17 14:17:05 PST
Created attachment 331540 [details]
Patch
Comment 2 Geoffrey Garen 2018-01-17 14:19:24 PST
Comment on attachment 331540 [details]
Patch

r=me
Comment 3 Alex Christensen 2018-01-17 14:34:51 PST
http://trac.webkit.org/r227086
Comment 4 Radar WebKit Bug Importer 2018-01-17 14:36:48 PST
<rdar://problem/36596556>
Comment 5 Joseph Pecoraro 2018-01-17 16:22:26 PST
I think we should include a warning to the console on pages that make use of ApplicationCache.

We did this in the past for things like webkitSubtle:

>    if (!m_webkitSubtle) {
>        m_webkitSubtle = WebKitSubtleCrypto::create(*downcast<Document>(scriptExecutionContext()));
>        scriptExecutionContext()->addConsoleMessage(MessageSource::Other, MessageLevel::Warning, ASCIILiteral("WebKitSubtleCrypto is deprecated. Please use SubtleCrypto instead."));
>    }
Comment 6 Alex Christensen 2018-01-17 16:22:58 PST
Let's do it