<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.1//EN"
"http://www.w3.org/TR/xhtml11/DTD/xhtml11.dtd">
<html xmlns="http://www.w3.org/1999/xhtml">
<head><meta http-equiv="content-type" content="text/html; charset=utf-8" />
<title>[185294] trunk/Source</title>
</head>
<body>
<style type="text/css"><!--
#msg dl.meta { border: 1px #006 solid; background: #369; padding: 6px; color: #fff; }
#msg dl.meta dt { float: left; width: 6em; font-weight: bold; }
#msg dt:after { content:':';}
#msg dl, #msg dt, #msg ul, #msg li, #header, #footer, #logmsg { font-family: verdana,arial,helvetica,sans-serif; font-size: 10pt; }
#msg dl a { font-weight: bold}
#msg dl a:link { color:#fc3; }
#msg dl a:active { color:#ff0; }
#msg dl a:visited { color:#cc6; }
h3 { font-family: verdana,arial,helvetica,sans-serif; font-size: 10pt; font-weight: bold; }
#msg pre { overflow: auto; background: #ffc; border: 1px #fa0 solid; padding: 6px; }
#logmsg { background: #ffc; border: 1px #fa0 solid; padding: 1em 1em 0 1em; }
#logmsg p, #logmsg pre, #logmsg blockquote { margin: 0 0 1em 0; }
#logmsg p, #logmsg li, #logmsg dt, #logmsg dd { line-height: 14pt; }
#logmsg h1, #logmsg h2, #logmsg h3, #logmsg h4, #logmsg h5, #logmsg h6 { margin: .5em 0; }
#logmsg h1:first-child, #logmsg h2:first-child, #logmsg h3:first-child, #logmsg h4:first-child, #logmsg h5:first-child, #logmsg h6:first-child { margin-top: 0; }
#logmsg ul, #logmsg ol { padding: 0; list-style-position: inside; margin: 0 0 0 1em; }
#logmsg ul { text-indent: -1em; padding-left: 1em; }#logmsg ol { text-indent: -1.5em; padding-left: 1.5em; }
#logmsg > ul, #logmsg > ol { margin: 0 0 1em 0; }
#logmsg pre { background: #eee; padding: 1em; }
#logmsg blockquote { border: 1px solid #fa0; border-left-width: 10px; padding: 1em 1em 0 1em; background: white;}
#logmsg dl { margin: 0; }
#logmsg dt { font-weight: bold; }
#logmsg dd { margin: 0; padding: 0 0 0.5em 0; }
#logmsg dd:before { content:'\00bb';}
#logmsg table { border-spacing: 0px; border-collapse: collapse; border-top: 4px solid #fa0; border-bottom: 1px solid #fa0; background: #fff; }
#logmsg table th { text-align: left; font-weight: normal; padding: 0.2em 0.5em; border-top: 1px dotted #fa0; }
#logmsg table td { text-align: right; border-top: 1px dotted #fa0; padding: 0.2em 0.5em; }
#logmsg table thead th { text-align: center; border-bottom: 1px solid #fa0; }
#logmsg table th.Corner { text-align: left; }
#logmsg hr { border: none 0; border-top: 2px dashed #fa0; height: 1px; }
#header, #footer { color: #fff; background: #636; border: 1px #300 solid; padding: 6px; }
#patch { width: 100%; }
#patch h4 {font-family: verdana,arial,helvetica,sans-serif;font-size:10pt;padding:8px;background:#369;color:#fff;margin:0;}
#patch .propset h4, #patch .binary h4 {margin:0;}
#patch pre {padding:0;line-height:1.2em;margin:0;}
#patch .diff {width:100%;background:#eee;padding: 0 0 10px 0;overflow:auto;}
#patch .propset .diff, #patch .binary .diff {padding:10px 0;}
#patch span {display:block;padding:0 10px;}
#patch .modfile, #patch .addfile, #patch .delfile, #patch .propset, #patch .binary, #patch .copfile {border:1px solid #ccc;margin:10px 0;}
#patch ins {background:#dfd;text-decoration:none;display:block;padding:0 10px;}
#patch del {background:#fdd;text-decoration:none;display:block;padding:0 10px;}
#patch .lines, .info {color:#888;background:#fff;}
--></style>
<div id="msg">
<dl class="meta">
<dt>Revision</dt> <dd><a href="http://trac.webkit.org/projects/webkit/changeset/185294">185294</a></dd>
<dt>Author</dt> <dd>andersca@apple.com</dd>
<dt>Date</dt> <dd>2015-06-06 14:22:18 -0700 (Sat, 06 Jun 2015)</dd>
</dl>
<h3>Log Message</h3>
<pre>Remove support from getting plug-in information from Carbon resources
https://bugs.webkit.org/show_bug.cgi?id=145730
Reviewed by Dan Bernstein.
Source/WebCore:
Remove a now unused function.
* platform/mac/WebCoreNSStringExtras.h:
* platform/mac/WebCoreNSStringExtras.mm:
(stringEncodingForResource): Deleted.
Source/WebKit/mac:
No current plug-in versions use this feature, and both Chrome and Mozilla for
Mac stopped looking at Carbon resources years ago.
* Misc/WebKitNSStringExtras.h:
* Misc/WebKitNSStringExtras.mm:
(+[NSString _web_encodingForResource:]): Deleted.
* Plugins/WebBasePluginPackage.mm:
* Plugins/WebNetscapePluginPackage.mm:
(-[WebNetscapePluginPackage _initWithPath:]):
(-[WebNetscapePluginPackage stringForStringListID:andIndex:]): Deleted.
(-[WebNetscapePluginPackage getPluginInfoFromResources]): Deleted.
Source/WebKit2:
No current plug-in versions use this feature, and both Chrome and Mozilla for
Mac stopped looking at Carbon resources years ago.
* Shared/Plugins/Netscape/mac/NetscapePluginModuleMac.mm:
(WebKit::NetscapePluginModule::getPluginInfo):
(WebKit::ResourceMap::ResourceMap): Deleted.
(WebKit::ResourceMap::~ResourceMap): Deleted.
(WebKit::ResourceMap::isValid): Deleted.
(WebKit::getStringListResource): Deleted.
(WebKit::getPluginInfoFromCarbonResources): Deleted.</pre>
<h3>Modified Paths</h3>
<ul>
<li><a href="#trunkSourceWebCoreChangeLog">trunk/Source/WebCore/ChangeLog</a></li>
<li><a href="#trunkSourceWebCoreplatformmacWebCoreNSStringExtrash">trunk/Source/WebCore/platform/mac/WebCoreNSStringExtras.h</a></li>
<li><a href="#trunkSourceWebCoreplatformmacWebCoreNSStringExtrasmm">trunk/Source/WebCore/platform/mac/WebCoreNSStringExtras.mm</a></li>
<li><a href="#trunkSourceWebKitmacChangeLog">trunk/Source/WebKit/mac/ChangeLog</a></li>
<li><a href="#trunkSourceWebKitmacMiscWebKitNSStringExtrash">trunk/Source/WebKit/mac/Misc/WebKitNSStringExtras.h</a></li>
<li><a href="#trunkSourceWebKitmacMiscWebKitNSStringExtrasmm">trunk/Source/WebKit/mac/Misc/WebKitNSStringExtras.mm</a></li>
<li><a href="#trunkSourceWebKitmacPluginsWebBasePluginPackagemm">trunk/Source/WebKit/mac/Plugins/WebBasePluginPackage.mm</a></li>
<li><a href="#trunkSourceWebKitmacPluginsWebNetscapePluginPackagemm">trunk/Source/WebKit/mac/Plugins/WebNetscapePluginPackage.mm</a></li>
<li><a href="#trunkSourceWebKit2ChangeLog">trunk/Source/WebKit2/ChangeLog</a></li>
<li><a href="#trunkSourceWebKit2SharedPluginsNetscapemacNetscapePluginModuleMacmm">trunk/Source/WebKit2/Shared/Plugins/Netscape/mac/NetscapePluginModuleMac.mm</a></li>
</ul>
</div>
<div id="patch">
<h3>Diff</h3>
<a id="trunkSourceWebCoreChangeLog"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/ChangeLog (185293 => 185294)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/ChangeLog        2015-06-06 20:01:09 UTC (rev 185293)
+++ trunk/Source/WebCore/ChangeLog        2015-06-06 21:22:18 UTC (rev 185294)
</span><span class="lines">@@ -1,3 +1,16 @@
</span><ins>+2015-06-06 Anders Carlsson <andersca@apple.com>
+
+ Remove support from getting plug-in information from Carbon resources
+ https://bugs.webkit.org/show_bug.cgi?id=145730
+
+ Reviewed by Dan Bernstein.
+
+ Remove a now unused function.
+
+ * platform/mac/WebCoreNSStringExtras.h:
+ * platform/mac/WebCoreNSStringExtras.mm:
+ (stringEncodingForResource): Deleted.
+
</ins><span class="cx"> 2015-06-06 Commit Queue <commit-queue@webkit.org>
</span><span class="cx">
</span><span class="cx"> Unreviewed, rolling out r185284.
</span></span></pre></div>
<a id="trunkSourceWebCoreplatformmacWebCoreNSStringExtrash"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/platform/mac/WebCoreNSStringExtras.h (185293 => 185294)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/platform/mac/WebCoreNSStringExtras.h        2015-06-06 20:01:09 UTC (rev 185293)
+++ trunk/Source/WebCore/platform/mac/WebCoreNSStringExtras.h        2015-06-06 21:22:18 UTC (rev 185294)
</span><span class="lines">@@ -44,9 +44,6 @@
</span><span class="cx"> WEBCORE_EXPORT BOOL hasCaseInsensitivePrefix(NSString *, NSString *prefix);
</span><span class="cx"> WEBCORE_EXPORT BOOL hasCaseInsensitiveSubstring(NSString *, NSString *substring);
</span><span class="cx"> WEBCORE_EXPORT NSString *filenameByFixingIllegalCharacters(NSString *);
</span><del>-#if !PLATFORM(IOS)
-WEBCORE_EXPORT CFStringEncoding stringEncodingForResource(Handle resource);
-#endif
</del><span class="cx">
</span><span class="cx"> WEBCORE_EXPORT NSString *preferredBundleLocalizationName();
</span><span class="cx"> NSString *canonicalLocaleName(NSString *);
</span></span></pre></div>
<a id="trunkSourceWebCoreplatformmacWebCoreNSStringExtrasmm"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/platform/mac/WebCoreNSStringExtras.mm (185293 => 185294)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/platform/mac/WebCoreNSStringExtras.mm        2015-06-06 20:01:09 UTC (rev 185293)
+++ trunk/Source/WebCore/platform/mac/WebCoreNSStringExtras.mm        2015-06-06 21:22:18 UTC (rev 185294)
</span><span class="lines">@@ -74,63 +74,6 @@
</span><span class="cx"> return filename;
</span><span class="cx"> }
</span><span class="cx">
</span><del>-#if !PLATFORM(IOS)
-
-#if COMPILER(CLANG)
-#pragma clang diagnostic push
-#pragma clang diagnostic ignored "-Wdeprecated-declarations"
-#endif
-
-CFStringEncoding stringEncodingForResource(Handle resource)
-{
- short resRef = HomeResFile(resource);
- if (ResError() != noErr)
- return NSMacOSRomanStringEncoding;
-
- // Get the FSRef for the current resource file
- FSRef fref;
- OSStatus error = FSGetForkCBInfo(resRef, 0, NULL, NULL, NULL, &fref, NULL);
- if (error != noErr)
- return NSMacOSRomanStringEncoding;
-
- RetainPtr<CFURLRef> url = adoptCF(CFURLCreateFromFSRef(NULL, &fref));
- if (!url)
- return NSMacOSRomanStringEncoding;
-
- NSString *path = [(NSURL *)url.get() path];
-
- // Get the lproj directory name
- path = [path stringByDeletingLastPathComponent];
- if (!stringIsCaseInsensitiveEqualToString([path pathExtension], @"lproj"))
- return NSMacOSRomanStringEncoding;
-
- NSString *directoryName = [[path stringByDeletingPathExtension] lastPathComponent];
- RetainPtr<CFStringRef> locale = adoptCF(CFLocaleCreateCanonicalLocaleIdentifierFromString(NULL, (CFStringRef)directoryName));
- if (!locale)
- return NSMacOSRomanStringEncoding;
-
- LangCode lang;
- RegionCode region;
- error = LocaleStringToLangAndRegionCodes([(NSString *)locale.get() UTF8String], &lang, &region);
- if (error != noErr)
- return NSMacOSRomanStringEncoding;
-
- TextEncoding encoding;
- error = UpgradeScriptInfoToTextEncoding(kTextScriptDontCare, lang, region, NULL, &encoding);
- if (error != noErr)
- return NSMacOSRomanStringEncoding;
-
- return encoding;
-}
-
-#if COMPILER(CLANG)
-#pragma clang diagnostic pop
-#endif
-
-#endif // !PLATFORM(IOS)
-
-
-
</del><span class="cx"> NSString *preferredBundleLocalizationName()
</span><span class="cx"> {
</span><span class="cx"> // FIXME: Any use of this function to pass localizations to another
</span></span></pre></div>
<a id="trunkSourceWebKitmacChangeLog"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebKit/mac/ChangeLog (185293 => 185294)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebKit/mac/ChangeLog        2015-06-06 20:01:09 UTC (rev 185293)
+++ trunk/Source/WebKit/mac/ChangeLog        2015-06-06 21:22:18 UTC (rev 185294)
</span><span class="lines">@@ -1,5 +1,24 @@
</span><span class="cx"> 2015-06-06 Anders Carlsson <andersca@apple.com>
</span><span class="cx">
</span><ins>+ Remove support from getting plug-in information from Carbon resources
+ https://bugs.webkit.org/show_bug.cgi?id=145730
+
+ Reviewed by Dan Bernstein.
+
+ No current plug-in versions use this feature, and both Chrome and Mozilla for
+ Mac stopped looking at Carbon resources years ago.
+
+ * Misc/WebKitNSStringExtras.h:
+ * Misc/WebKitNSStringExtras.mm:
+ (+[NSString _web_encodingForResource:]): Deleted.
+ * Plugins/WebBasePluginPackage.mm:
+ * Plugins/WebNetscapePluginPackage.mm:
+ (-[WebNetscapePluginPackage _initWithPath:]):
+ (-[WebNetscapePluginPackage stringForStringListID:andIndex:]): Deleted.
+ (-[WebNetscapePluginPackage getPluginInfoFromResources]): Deleted.
+
+2015-06-06 Anders Carlsson <andersca@apple.com>
+
</ins><span class="cx"> Remove now unused category methods
</span><span class="cx"> https://bugs.webkit.org/show_bug.cgi?id=145729
</span><span class="cx">
</span></span></pre></div>
<a id="trunkSourceWebKitmacMiscWebKitNSStringExtrash"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebKit/mac/Misc/WebKitNSStringExtras.h (185293 => 185294)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebKit/mac/Misc/WebKitNSStringExtras.h        2015-06-06 20:01:09 UTC (rev 185293)
+++ trunk/Source/WebKit/mac/Misc/WebKitNSStringExtras.h        2015-06-06 21:22:18 UTC (rev 185294)
</span><span class="lines">@@ -50,10 +50,6 @@
</span><span class="cx">
</span><span class="cx"> - (NSString *)_web_stringByStrippingReturnCharacters;
</span><span class="cx">
</span><del>-#if !TARGET_OS_IPHONE
-+ (NSStringEncoding)_web_encodingForResource:(Handle)resource;
-#endif
-
</del><span class="cx"> - (BOOL)_webkit_isCaseInsensitiveEqualToString:(NSString *)string;
</span><span class="cx"> - (BOOL)_webkit_hasCaseInsensitivePrefix:(NSString *)suffix;
</span><span class="cx"> - (BOOL)_webkit_hasCaseInsensitiveSuffix:(NSString *)suffix;
</span></span></pre></div>
<a id="trunkSourceWebKitmacMiscWebKitNSStringExtrasmm"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebKit/mac/Misc/WebKitNSStringExtras.mm (185293 => 185294)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebKit/mac/Misc/WebKitNSStringExtras.mm        2015-06-06 20:01:09 UTC (rev 185293)
+++ trunk/Source/WebKit/mac/Misc/WebKitNSStringExtras.mm        2015-06-06 21:22:18 UTC (rev 185294)
</span><span class="lines">@@ -170,13 +170,6 @@
</span><span class="cx"> return newString;
</span><span class="cx"> }
</span><span class="cx">
</span><del>-#if !PLATFORM(IOS)
-+ (NSStringEncoding)_web_encodingForResource:(Handle)resource
-{
- return CFStringConvertEncodingToNSStringEncoding(stringEncodingForResource(resource));
-}
-#endif
-
</del><span class="cx"> - (BOOL)_webkit_isCaseInsensitiveEqualToString:(NSString *)string
</span><span class="cx"> {
</span><span class="cx"> return stringIsCaseInsensitiveEqualToString(self, string);
</span></span></pre></div>
<a id="trunkSourceWebKitmacPluginsWebBasePluginPackagemm"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebKit/mac/Plugins/WebBasePluginPackage.mm (185293 => 185294)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebKit/mac/Plugins/WebBasePluginPackage.mm        2015-06-06 20:01:09 UTC (rev 185293)
+++ trunk/Source/WebKit/mac/Plugins/WebBasePluginPackage.mm        2015-06-06 21:22:18 UTC (rev 185294)
</span><span class="lines">@@ -33,7 +33,6 @@
</span><span class="cx"> #import "WebNetscapePluginPackage.h"
</span><span class="cx"> #import "WebPluginPackage.h"
</span><span class="cx"> #import "WebTypesInternal.h"
</span><del>-#import <WebCore/WebCoreNSStringExtras.h>
</del><span class="cx"> #import <WebCore/WebCoreObjCExtras.h>
</span><span class="cx"> #import <algorithm>
</span><span class="cx"> #import <mach-o/arch.h>
</span></span></pre></div>
<a id="trunkSourceWebKitmacPluginsWebNetscapePluginPackagemm"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebKit/mac/Plugins/WebNetscapePluginPackage.mm (185293 => 185294)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebKit/mac/Plugins/WebNetscapePluginPackage.mm        2015-06-06 20:01:09 UTC (rev 185293)
+++ trunk/Source/WebKit/mac/Plugins/WebNetscapePluginPackage.mm        2015-06-06 21:22:18 UTC (rev 185294)
</span><span class="lines">@@ -45,10 +45,6 @@
</span><span class="cx">
</span><span class="cx"> using namespace WebCore;
</span><span class="cx">
</span><del>-#define PluginNameOrDescriptionStringNumber 126
-#define MIMEDescriptionStringNumber 127
-#define MIMEListStringStringNumber 128
-
</del><span class="cx"> @interface WebNetscapePluginPackage (Internal)
</span><span class="cx"> - (void)_unloadWithShutdown:(BOOL)shutdown;
</span><span class="cx"> @end
</span><span class="lines">@@ -65,101 +61,6 @@
</span><span class="cx"> CFBundleCloseBundleResourceMap(cfBundle.get(), resRef);
</span><span class="cx"> }
</span><span class="cx">
</span><del>-#if COMPILER(CLANG)
-#pragma clang diagnostic push
-#pragma clang diagnostic ignored "-Wdeprecated-declarations"
-#endif
-
-- (NSString *)stringForStringListID:(SInt16)stringListID andIndex:(SInt16)index
-{
- // Get resource, and dereference the handle.
- Handle stringHandle = Get1Resource('STR#', stringListID);
- if (stringHandle == NULL) {
- return nil;
- }
- unsigned char *p = (unsigned char *)*stringHandle;
- if (!p)
- return nil;
-
- // Check the index against the length of the string list, then skip the length.
- if (index < 1 || index > *(SInt16 *)p)
- return nil;
- p += sizeof(SInt16);
-
- // Skip any strings that come before the one we are looking for.
- while (--index)
- p += 1 + *p;
-
- // Convert the one we found into an NSString.
- return [[[NSString alloc] initWithBytes:(p + 1) length:*p encoding:[NSString _web_encodingForResource:stringHandle]] autorelease];
-}
-
-- (BOOL)getPluginInfoFromResources
-{
- SInt16 resRef = [self openResourceFile];
- if (resRef == -1)
- return NO;
-
- UseResFile(resRef);
- if (ResError() != noErr)
- return NO;
-
- NSString *MIME, *extensionsList, *description;
- NSArray *extensions;
- unsigned i;
-
- for (i=1; 1; i+=2) {
- MIME = [[self stringForStringListID:MIMEListStringStringNumber
- andIndex:i] lowercaseString];
- if (!MIME)
- break;
-
- MimeClassInfo mimeClassInfo;
- mimeClassInfo.type = String(MIME).lower();
-
- extensionsList = [[self stringForStringListID:MIMEListStringStringNumber andIndex:i+1] lowercaseString];
- if (extensionsList) {
- extensions = [extensionsList componentsSeparatedByString:@","];
- for (NSUInteger j = 0; j < [extensions count]; ++j)
- mimeClassInfo.extensions.append((NSString *)[extensions objectAtIndex:j]);
- }
-
- description = [self stringForStringListID:MIMEDescriptionStringNumber
- andIndex:pluginInfo.mimes.size() + 1];
- mimeClassInfo.desc = description;
-
- pluginInfo.mimes.append(mimeClassInfo);
- }
-
- NSString *filename = [(NSString *)path lastPathComponent];
- pluginInfo.file = filename;
-
- description = [self stringForStringListID:PluginNameOrDescriptionStringNumber andIndex:1];
- if (!description)
- description = filename;
- pluginInfo.desc = description;
-
-
- NSString *theName = [self stringForStringListID:PluginNameOrDescriptionStringNumber andIndex:2];
- if (!theName)
- theName = filename;
- pluginInfo.name = theName;
-
- pluginInfo.isApplicationPlugin = false;
- pluginInfo.clientLoadPolicy = PluginLoadClientPolicyUndefined;
-#if PLATFORM(MAC)
- pluginInfo.bundleIdentifier = self.bundleIdentifier;
- pluginInfo.versionString = self.bundleVersion;
-#endif
-
- [self closeResourceFile:resRef];
-
- return YES;
-}
-#if COMPILER(CLANG)
-#pragma clang diagnostic pop
-#endif
-
</del><span class="cx"> - (BOOL)_initWithPath:(NSString *)pluginPath
</span><span class="cx"> {
</span><span class="cx"> resourceRef = -1;
</span><span class="lines">@@ -196,7 +97,7 @@
</span><span class="cx">
</span><span class="cx"> #endif
</span><span class="cx">
</span><del>- if (![self getPluginInfoFromPLists] && ![self getPluginInfoFromResources])
</del><ins>+ if (![self getPluginInfoFromPLists])
</ins><span class="cx"> return NO;
</span><span class="cx">
</span><span class="cx"> return YES;
</span></span></pre></div>
<a id="trunkSourceWebKit2ChangeLog"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebKit2/ChangeLog (185293 => 185294)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebKit2/ChangeLog        2015-06-06 20:01:09 UTC (rev 185293)
+++ trunk/Source/WebKit2/ChangeLog        2015-06-06 21:22:18 UTC (rev 185294)
</span><span class="lines">@@ -1,3 +1,21 @@
</span><ins>+2015-06-06 Anders Carlsson <andersca@apple.com>
+
+ Remove support from getting plug-in information from Carbon resources
+ https://bugs.webkit.org/show_bug.cgi?id=145730
+
+ Reviewed by Dan Bernstein.
+
+ No current plug-in versions use this feature, and both Chrome and Mozilla for
+ Mac stopped looking at Carbon resources years ago.
+
+ * Shared/Plugins/Netscape/mac/NetscapePluginModuleMac.mm:
+ (WebKit::NetscapePluginModule::getPluginInfo):
+ (WebKit::ResourceMap::ResourceMap): Deleted.
+ (WebKit::ResourceMap::~ResourceMap): Deleted.
+ (WebKit::ResourceMap::isValid): Deleted.
+ (WebKit::getStringListResource): Deleted.
+ (WebKit::getPluginInfoFromCarbonResources): Deleted.
+
</ins><span class="cx"> 2015-06-05 Anders Carlsson <andersca@apple.com>
</span><span class="cx">
</span><span class="cx"> Only allow plug-ins to create custom MIME property lists on older versions of OS X
</span></span></pre></div>
<a id="trunkSourceWebKit2SharedPluginsNetscapemacNetscapePluginModuleMacmm"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebKit2/Shared/Plugins/Netscape/mac/NetscapePluginModuleMac.mm (185293 => 185294)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebKit2/Shared/Plugins/Netscape/mac/NetscapePluginModuleMac.mm        2015-06-06 20:01:09 UTC (rev 185293)
+++ trunk/Source/WebKit2/Shared/Plugins/Netscape/mac/NetscapePluginModuleMac.mm        2015-06-06 21:22:18 UTC (rev 185294)
</span><span class="lines">@@ -30,7 +30,6 @@
</span><span class="cx">
</span><span class="cx"> #import "PluginProcessProxy.h"
</span><span class="cx"> #import "PluginSandboxProfile.h"
</span><del>-#import <WebCore/WebCoreNSStringExtras.h>
</del><span class="cx"> #import <wtf/HashSet.h>
</span><span class="cx"> #import <wtf/MainThread.h>
</span><span class="cx">
</span><span class="lines">@@ -218,139 +217,6 @@
</span><span class="cx"> return true;
</span><span class="cx"> }
</span><span class="cx">
</span><del>-#pragma clang diagnostic push
-#pragma clang diagnostic ignored "-Wdeprecated-declarations"
-
-class ResourceMap {
-public:
- explicit ResourceMap(CFBundleRef bundle)
- : m_bundle(bundle)
- , m_currentResourceFile(CurResFile())
- , m_bundleResourceMap(CFBundleOpenBundleResourceMap(m_bundle))
- {
- UseResFile(m_bundleResourceMap);
- }
-
- ~ResourceMap()
- {
- // Close the resource map.
- CFBundleCloseBundleResourceMap(m_bundle, m_bundleResourceMap);
-
- // And restore the old resource.
- UseResFile(m_currentResourceFile);
- }
-
- bool isValid() const { return m_bundleResourceMap != -1; }
-
-private:
- CFBundleRef m_bundle;
- ResFileRefNum m_currentResourceFile;
- ResFileRefNum m_bundleResourceMap;
-};
-
-static bool getStringListResource(ResID resourceID, Vector<String>& stringList) {
- Handle stringListHandle = Get1Resource('STR#', resourceID);
- if (!stringListHandle || !*stringListHandle)
- return false;
-
- // Get the string list size.
- Size stringListSize = GetHandleSize(stringListHandle);
- if (stringListSize < static_cast<Size>(sizeof(UInt16)))
- return false;
-
- CFStringEncoding stringEncoding = stringEncodingForResource(stringListHandle);
-
- unsigned char* ptr = reinterpret_cast<unsigned char*>(*stringListHandle);
- unsigned char* end = ptr + stringListSize;
-
- // Get the number of strings in the string list.
- UInt16 numStrings = *reinterpret_cast<UInt16*>(ptr);
- ptr += sizeof(UInt16);
-
- for (UInt16 i = 0; i < numStrings; ++i) {
- // We're past the end of the string, bail.
- if (ptr >= end)
- return false;
-
- // Get the string length.
- unsigned char stringLength = *ptr++;
-
- RetainPtr<CFStringRef> cfString = adoptCF(CFStringCreateWithBytesNoCopy(kCFAllocatorDefault, ptr, stringLength, stringEncoding, false, kCFAllocatorNull));
- if (!cfString.get())
- return false;
-
- stringList.append(cfString.get());
- ptr += stringLength;
- }
-
- if (ptr != end)
- return false;
-
- return true;
-}
-
-#pragma clang diagnostic pop
-
-static const ResID PluginNameOrDescriptionStringNumber = 126;
-static const ResID MIMEDescriptionStringNumber = 127;
-static const ResID MIMEListStringStringNumber = 128;
-
-static bool getPluginInfoFromCarbonResources(CFBundleRef bundle, PluginModuleInfo& plugin)
-{
- ASSERT(RunLoop::isMain());
-
- ResourceMap resourceMap(bundle);
- if (!resourceMap.isValid())
- return false;
-
- // Get the description and name string list.
- Vector<String> descriptionAndName;
- if (!getStringListResource(PluginNameOrDescriptionStringNumber, descriptionAndName))
- return false;
-
- // Get the MIME types and extensions string list. This list needs to be a multiple of two.
- Vector<String> mimeTypesAndExtensions;
- if (!getStringListResource(MIMEListStringStringNumber, mimeTypesAndExtensions))
- return false;
-
- if (mimeTypesAndExtensions.size() % 2)
- return false;
-
- // Now get the MIME type descriptions string list. This string list needs to be the same length as the number of MIME types.
- Vector<String> mimeTypeDescriptions;
- if (!getStringListResource(MIMEDescriptionStringNumber, mimeTypeDescriptions))
- return false;
-
- // Add all MIME types.
- for (size_t i = 0; i < mimeTypesAndExtensions.size() / 2; ++i) {
- MimeClassInfo mimeClassInfo;
-
- const String& mimeType = mimeTypesAndExtensions[i * 2];
- String description;
- if (i < mimeTypeDescriptions.size())
- description = mimeTypeDescriptions[i];
-
- mimeClassInfo.type = mimeType.lower();
- mimeClassInfo.desc = description;
-
- Vector<String> extensions;
- mimeTypesAndExtensions[i * 2 + 1].split(',', extensions);
-
- for (size_t i = 0; i < extensions.size(); ++i)
- mimeClassInfo.extensions.append(extensions[i].lower());
-
- plugin.info.mimes.append(mimeClassInfo);
- }
-
- // Set the description and name if they exist.
- if (descriptionAndName.size() > 0)
- plugin.info.desc = descriptionAndName[0];
- if (descriptionAndName.size() > 1)
- plugin.info.name = descriptionAndName[1];
-
- return true;
-}
-
</del><span class="cx"> bool NetscapePluginModule::getPluginInfo(const String& pluginPath, PluginModuleInfo& plugin)
</span><span class="cx"> {
</span><span class="cx"> RetainPtr<CFURLRef> bundleURL = adoptCF(CFURLCreateWithFileSystemPath(kCFAllocatorDefault, pluginPath.createCFString().get(), kCFURLPOSIXPathStyle, false));
</span><span class="lines">@@ -387,9 +253,7 @@
</span><span class="cx"> plugin.preferencePanePath = static_cast<CFStringRef>(preferencePathTypeRef);
</span><span class="cx"> }
</span><span class="cx">
</span><del>- // Check that there's valid info for this plug-in.
- if (!getPluginInfoFromPropertyLists(bundle.get(), plugin) &&
- !getPluginInfoFromCarbonResources(bundle.get(), plugin))
</del><ins>+ if (!getPluginInfoFromPropertyLists(bundle.get(), plugin))
</ins><span class="cx"> return false;
</span><span class="cx">
</span><span class="cx"> plugin.hasSandboxProfile = pluginHasSandboxProfile(plugin.bundleIdentifier);
</span></span></pre>
</div>
</div>
</body>
</html>