161219 2016-08-25 16:26:21 -0700 Regression(r204605): WebCore::checkExistingName() logs error messages on iOS 2016-08-25 20:45:08 -0700 1 1 1 Unclassified WebKit DOM WebKit Nightly Build Unspecified Unspecified ASSIGNED P2 Normal --- 160931 1 cdumez cdumez ap cdumez oldest_to_newest 1223309 0 cdumez 2016-08-25 16:26:21 -0700 WebCore::checkExistingName() logs error messages on iOS after r204605: ERROR: alias x-mac-cyrillic maps to x-mac-cyrillic already, but someone is trying to make it map to macos-7_3-10.2 /Volumes/Data/Safari/OpenSource/Source/WebCore/platform/text/TextEncodingRegistry.cpp(138) : void WebCore::checkExistingName(const char *, const char *) ERROR: alias windows-10007 maps to x-mac-cyrillic already, but someone is trying to make it map to macos-7_3-10.2 /Volumes/Data/Safari/OpenSource/Source/WebCore/platform/text/TextEncodingRegistry.cpp(138) : void WebCore::checkExistingName(const char *, const char *) ERROR: alias mac-cyrillic maps to x-mac-cyrillic already, but someone is trying to make it map to macos-7_3-10.2 /Volumes/Data/Safari/OpenSource/Source/WebCore/platform/text/TextEncodingRegistry.cpp(138) : void WebCore::checkExistingName(const char *, const char *) ERROR: alias maccy maps to x-mac-cyrillic already, but someone is trying to make it map to macos-7_3-10.2 /Volumes/Data/Safari/OpenSource/Source/WebCore/platform/text/TextEncodingRegistry.cpp(138) : void WebCore::checkExistingName(const char *, const char *) ERROR: alias x-MacCyrillic maps to x-mac-cyrillic already, but someone is trying to make it map to macos-7_3-10.2 /Volumes/Data/Safari/OpenSource/Source/WebCore/platform/text/TextEncodingRegistry.cpp(138) : void WebCore::checkExistingName(const char *, const char *) ERROR: alias x-MacUkraine maps to x-mac-cyrillic already, but someone is trying to make it map to macos-7_3-10.2 /Volumes/Data/Safari/OpenSource/Source/WebCore/platform/text/TextEncodingRegistry.cpp(138) : void WebCore::checkExistingName(const char *, const char *) ERROR: alias x-mac-greek maps to x-mac-greek already, but someone is trying to make it map to macos-6_2-10.4 /Volumes/Data/Safari/OpenSource/Source/WebCore/platform/text/TextEncodingRegistry.cpp(138) : void WebCore::checkExistingName(const char *, const char *) ERROR: alias windows-10006 maps to x-mac-greek already, but someone is trying to make it map to macos-6_2-10.4 /Volumes/Data/Safari/OpenSource/Source/WebCore/platform/text/TextEncodingRegistry.cpp(138) : void WebCore::checkExistingName(const char *, const char *) ERROR: alias macgr maps to x-mac-greek already, but someone is trying to make it map to macos-6_2-10.4 /Volumes/Data/Safari/OpenSource/Source/WebCore/platform/text/TextEncodingRegistry.cpp(138) : void WebCore::checkExistingName(const char *, const char *) ERROR: alias x-MacGreek maps to x-mac-greek already, but someone is trying to make it map to macos-6_2-10.4 /Volumes/Data/Safari/OpenSource/Source/WebCore/platform/text/TextEncodingRegistry.cpp(138) : void WebCore::checkExistingName(const char *, const char *) ERROR: alias x-mac-centraleurroman maps to x-mac-centraleurroman already, but someone is trying to make it map to macos-29-10.2 /Volumes/Data/Safari/OpenSource/Source/WebCore/platform/text/TextEncodingRegistry.cpp(138) : void WebCore::checkExistingName(const char *, const char *) ERROR: alias windows-10029 maps to x-mac-centraleurroman already, but someone is trying to make it map to macos-29-10.2 /Volumes/Data/Safari/OpenSource/Source/WebCore/platform/text/TextEncodingRegistry.cpp(138) : void WebCore::checkExistingName(const char *, const char *) ERROR: alias x-mac-ce maps to x-mac-centraleurroman already, but someone is trying to make it map to macos-29-10.2 /Volumes/Data/Safari/OpenSource/Source/WebCore/platform/text/TextEncodingRegistry.cpp(138) : void WebCore::checkExistingName(const char *, const char *) ERROR: alias macce maps to x-mac-centraleurroman already, but someone is trying to make it map to macos-29-10.2 /Volumes/Data/Safari/OpenSource/Source/WebCore/platform/text/TextEncodingRegistry.cpp(138) : void WebCore::checkExistingName(const char *, const char *) ERROR: alias maccentraleurope maps to x-mac-centraleurroman already, but someone is trying to make it map to macos-29-10.2 /Volumes/Data/Safari/OpenSource/Source/WebCore/platform/text/TextEncodingRegistry.cpp(138) : void WebCore::checkExistingName(const char *, const char *) ERROR: alias x-MacCentralEurope maps to x-mac-centraleurroman already, but someone is trying to make it map to macos-29-10.2 /Volumes/Data/Safari/OpenSource/Source/WebCore/platform/text/TextEncodingRegistry.cpp(138) : void WebCore::checkExistingName(const char *, const char *) ERROR: alias x-mac-turkish maps to x-mac-turkish already, but someone is trying to make it map to macos-35-10.2 /Volumes/Data/Safari/OpenSource/Source/WebCore/platform/text/TextEncodingRegistry.cpp(138) : void WebCore::checkExistingName(const char *, const char *) ERROR: alias windows-10081 maps to x-mac-turkish already, but someone is trying to make it map to macos-35-10.2 /Volumes/Data/Safari/OpenSource/Source/WebCore/platform/text/TextEncodingRegistry.cpp(138) : void WebCore::checkExistingName(const char *, const char *) ERROR: alias mactr maps to x-mac-turkish already, but someone is trying to make it map to macos-35-10.2 /Volumes/Data/Safari/OpenSource/Source/WebCore/platform/text/TextEncodingRegistry.cpp(138) : void WebCore::checkExistingName(const char *, const char *) ERROR: alias x-MacTurkish maps to x-mac-turkish already, but someone is trying to make it map to macos-35-10.2 /Volumes/Data/Safari/OpenSource/Source/WebCore/platform/text/TextEncodingRegistry.cpp(138) : void WebCore::checkExistingName(const char *, const char *) 1223319 1 287043 cdumez 2016-08-25 16:35:42 -0700 Created attachment 287043 Patch 1223397 2 287043 ap 2016-08-25 20:44:58 -0700 Comment on attachment 287043 Patch View in context: https://bugs.webkit.org/attachment.cgi?id=287043&action=review > Source/WebCore/ChangeLog:12 > + I also dropped to 2 remaining ones (macos-6-10.2, softbank-sjis) because: The latter is scary, I'm pretty sure that it is the important on iOS - although I do not know if it gets used in WebKit specifically. 287043 2016-08-25 16:35:42 -0700 2016-08-25 16:35:42 -0700 Patch 161219_iOS_codecs.patch text/plain 3915 cdumez ZGlmZiAtLWdpdCBhL1NvdXJjZS9XZWJDb3JlL0NoYW5nZUxvZyBiL1NvdXJjZS9XZWJDb3JlL0No YW5nZUxvZwppbmRleCA2OGUxN2U5Li40ZjgyYjZlIDEwMDY0NAotLS0gYS9Tb3VyY2UvV2ViQ29y ZS9DaGFuZ2VMb2cKKysrIGIvU291cmNlL1dlYkNvcmUvQ2hhbmdlTG9nCkBAIC0xLDUgKzEsMjUg QEAKIDIwMTYtMDgtMjUgIENocmlzIER1bWV6ICA8Y2R1bWV6QGFwcGxlLmNvbT4KIAorICAgICAg ICBSZWdyZXNzaW9uKHIyMDQ2MDUpOiBXZWJDb3JlOjpjaGVja0V4aXN0aW5nTmFtZSgpIGxvZ3Mg ZXJyb3IgbWVzc2FnZXMgb24gaU9TCisgICAgICAgIGh0dHBzOi8vYnVncy53ZWJraXQub3JnL3No b3dfYnVnLmNnaT9pZD0xNjEyMTkKKworICAgICAgICBSZXZpZXdlZCBieSBOT0JPRFkgKE9PUFMh KS4KKworICAgICAgICBUaGUgZm9sbG93aW5nIGNvZGVjcyBhbHJlYWR5IGdldCByZWdpc3RlcmVk IGFmdGVyIHIyMDQ2MDUgc28gdGhlcmUgaXMKKyAgICAgICAgbm8gbmVlZCB0byByZWdpc3RlciB0 aGVtIGFnYWluIG9uIGlPUzoKKyAgICAgICAgbWFjb3MtN18zLTEwLjIsIG1hY29zLTZfMi0xMC40 LCBtYWNvcy0yOS0xMC4yLCBtYWNvcy0zNS0xMC4yCisKKyAgICAgICAgSSBhbHNvIGRyb3BwZWQg dG8gMiByZW1haW5pbmcgb25lcyAobWFjb3MtNi0xMC4yLCBzb2Z0YmFuay1zamlzKSBiZWNhdXNl OgorICAgICAgICAtIEkgY2Fubm90IGZpbmQgYW55IHJlZmVyZW5jZSB0byB0aGVtIGluIHRoZSBJ Q1UgZG9jdW1lbnRhdGlvbgorICAgICAgICAtIFRoZXkgYXJlIG5vdCBwcmVzZW50IGluIHRoZSBJ Q1UgbGlicmFyeSBJIGhhdmUgb24gbXkgc3lzdGVtCisgICAgICAgIC0gVGhleSBkbyBub3QgZ2V0 IHJlZ2lzdGVyZWQgYnkgVGV4dENvZGVjTWFjIG9uIG15IHN5c3RlbQorCisgICAgICAgICogcGxh dGZvcm0vdGV4dC9UZXh0Q29kZWNJQ1UuY3BwOgorICAgICAgICAoV2ViQ29yZTo6VGV4dENvZGVj SUNVOjpyZWdpc3RlckVuY29kaW5nTmFtZXMpOgorICAgICAgICAoV2ViQ29yZTo6VGV4dENvZGVj SUNVOjpyZWdpc3RlckNvZGVjcyk6CisKKzIwMTYtMDgtMjUgIENocmlzIER1bWV6ICA8Y2R1bWV6 QGFwcGxlLmNvbT4KKwogICAgICAgICBSZWdyZXNzaW9uKHIyMDM2MjMpOiBCcmVha3MgQXBwIFN0 b3JlIGFwcGxpY2F0aW9uCiAgICAgICAgIGh0dHBzOi8vYnVncy53ZWJraXQub3JnL3Nob3dfYnVn LmNnaT9pZD0xNjEyMDYKICAgICAgICAgPHJkYXI6Ly9wcm9ibGVtLzI4MDE1MDYwPgpkaWZmIC0t Z2l0IGEvU291cmNlL1dlYkNvcmUvcGxhdGZvcm0vdGV4dC9UZXh0Q29kZWNJQ1UuY3BwIGIvU291 cmNlL1dlYkNvcmUvcGxhdGZvcm0vdGV4dC9UZXh0Q29kZWNJQ1UuY3BwCmluZGV4IDQwMjEyZmUu LmNmNDI3ZTUgMTAwNjQ0Ci0tLSBhL1NvdXJjZS9XZWJDb3JlL3BsYXRmb3JtL3RleHQvVGV4dENv ZGVjSUNVLmNwcAorKysgYi9Tb3VyY2UvV2ViQ29yZS9wbGF0Zm9ybS90ZXh0L1RleHRDb2RlY0lD VS5jcHAKQEAgLTQzLDE4ICs0Myw2IEBAIG5hbWVzcGFjZSBXZWJDb3JlIHsKIAogY29uc3Qgc2l6 ZV90IENvbnZlcnNpb25CdWZmZXJTaXplID0gMTYzODQ7CiAKLSNpZiBQTEFURk9STShJT1MpCi1z dGF0aWMgY29uc3QgY2hhciogdGV4dENvZGVjTWFjQWxpYXNlc1tdID0gewotICAgICJtYWNvcy03 XzMtMTAuMiIsIC8vIHhtYWNjeXJpbGxpYywgbWFjY3lyaWxsaWMKLSAgICAibWFjb3MtNl8yLTEw LjQiLCAvLyB4bWFjZ3JlZWsKLSAgICAibWFjb3MtNi0xMC4yIiwgICAvLyBtYWNncmVlawotICAg ICJtYWNvcy0yOS0xMC4yIiwgIC8vIHhtYWNjZW50cmFsZXVycm9tYW4sIG1hY2NlbnRyYWxldXJy b21hbgotICAgICJtYWNvcy0zNS0xMC4yIiwgIC8vIHhtYWN0dXJraXNoLCBtYWN0dXJraXNoCi0g ICAgInNvZnRiYW5rLXNqaXMiLCAgLy8gc29mdGJhbmtzamlzCi0gICAgbnVsbHB0cgotfTsKLSNl bmRpZgotCiBJQ1VDb252ZXJ0ZXJXcmFwcGVyOjp+SUNVQ29udmVydGVyV3JhcHBlcigpCiB7CiAg ICAgaWYgKGNvbnZlcnRlcikKQEAgLTE4MCwyOSArMTY4LDYgQEAgdm9pZCBUZXh0Q29kZWNJQ1U6 OnJlZ2lzdGVyRW5jb2RpbmdOYW1lcyhFbmNvZGluZ05hbWVSZWdpc3RyYXIgcmVnaXN0cmFyKQog ICAgICAgICBmb3IgKHNpemVfdCBpID0gMDsgaSA8IGVuY29kaW5nTmFtZS5hbGlhc0NvdW50OyAr K2kpCiAgICAgICAgICAgICByZWdpc3RyYXIoZW5jb2RpbmdOYW1lLmFsaWFzZXNbaV0sIGVuY29k aW5nTmFtZS5uYW1lKTsKICAgICB9Ci0KLSNpZiBQTEFURk9STShJT1MpCi0gICAgLy8gQS5CLiBh ZGRpbmcgYSBmZXcgbW9yZSBNYWMgZW5jb2RpbmdzIG1pc3NpbmcgJ2NhdXNlIHdlIGRvbid0IGhh dmUgVGV4dENvZGVjTWFjIHJpZ2h0IG5vdwotICAgIC8vIGx1Y2tpbHksIHRoZXkgYXJlIHN1cHBv cnRlZCBpbiBJQ1UsIGp1c3QgbmVlZCB0byBhbGlhcyB0aGVtLgotICAgIC8vIHRoaXMgaGFuZGxl cyBlbmNvZGluZ3MgdGhhdCBPUyBYIHVzZXMgVEVDIChUZXh0Q29kZWNNYWMpCi0gICAgLy8gPGh0 dHA6Ly9wdWJsaWIuYm91bGRlci5pYm0uY29tL2luZm9jZW50ZXIvd21iaGVscC92NnIwbTAvaW5k ZXguanNwP3RvcGljPS9jb20uaWJtLmV0b29scy5tZnQuZWIuZG9jL2FjMDA0MDhfLmh0bT4KLSAg ICBpbnQzMl90IGkgPSAwOwotICAgIGZvciAoY29uc3QgY2hhciogbWFjQWxpYXMgPSB0ZXh0Q29k ZWNNYWNBbGlhc2VzW2ldOyBtYWNBbGlhczsgbWFjQWxpYXMgPSB0ZXh0Q29kZWNNYWNBbGlhc2Vz WysraV0pIHsKLSAgICAgICAgcmVnaXN0cmFyKG1hY0FsaWFzLCBtYWNBbGlhcyk7Ci0KLSAgICAg ICAgVUVycm9yQ29kZSBlcnJvciA9IFVfWkVST19FUlJPUjsKLSAgICAgICAgdWludDE2X3QgbnVt QWxpYXNlcyA9IHVjbnZfY291bnRBbGlhc2VzKG1hY0FsaWFzLCAmZXJyb3IpOwotICAgICAgICBB U1NFUlQoVV9TVUNDRVNTKGVycm9yKSk7Ci0gICAgICAgIGlmIChVX1NVQ0NFU1MoZXJyb3IpKQot ICAgICAgICAgICAgZm9yICh1aW50MTZfdCBqID0gMDsgaiA8IG51bUFsaWFzZXM7ICsraikgewot ICAgICAgICAgICAgICAgIGVycm9yID0gVV9aRVJPX0VSUk9SOwotICAgICAgICAgICAgICAgIGNv bnN0IGNoYXIqIGFsaWFzID0gdWNudl9nZXRBbGlhcyhtYWNBbGlhcywgaiwgJmVycm9yKTsKLSAg ICAgICAgICAgICAgICBBU1NFUlQoVV9TVUNDRVNTKGVycm9yKSk7Ci0gICAgICAgICAgICAgICAg aWYgKFVfU1VDQ0VTUyhlcnJvcikgJiYgc3RyY21wKGFsaWFzLCBtYWNBbGlhcykpCi0gICAgICAg ICAgICAgICAgICAgIHJlZ2lzdHJhcihhbGlhcywgbWFjQWxpYXMpOwotICAgICAgICAgICAgfQot ICAgIH0KLSNlbmRpZgogfQogCiB2b2lkIFRleHRDb2RlY0lDVTo6cmVnaXN0ZXJDb2RlY3MoVGV4 dENvZGVjUmVnaXN0cmFyIHJlZ2lzdHJhcikKQEAgLTI0MCwxMyArMjA1LDYgQEAgdm9pZCBUZXh0 Q29kZWNJQ1U6OnJlZ2lzdGVyQ29kZWNzKFRleHRDb2RlY1JlZ2lzdHJhciByZWdpc3RyYXIpCiAg ICAgICAgIEFTU0VSVChVX1NVQ0NFU1MoZXJyb3IpKTsKICAgICAgICAgcmVnaXN0cmFyKGVuY29k aW5nTmFtZS5uYW1lLCBjcmVhdGUsIGNhbm9uaWNhbENvbnZlcnRlck5hbWUpOwogICAgIH0KLQot I2lmIFBMQVRGT1JNKElPUykKLSAgICAvLyBTZWUgY29tbWVudCBhYm92ZSBpbiByZWdpc3RlckVu Y29kaW5nTmFtZXMoKS4KLSAgICBpbnQzMl90IGkgPSAwOwotICAgIGZvciAoY29uc3QgY2hhciog YWxpYXMgPSB0ZXh0Q29kZWNNYWNBbGlhc2VzW2ldOyBhbGlhczsgYWxpYXMgPSB0ZXh0Q29kZWNN YWNBbGlhc2VzWysraV0pCi0gICAgICAgIHJlZ2lzdHJhcihhbGlhcywgY3JlYXRlLCAwKTsKLSNl bmRpZgogfQogCiBUZXh0Q29kZWNJQ1U6OlRleHRDb2RlY0lDVShjb25zdCBjaGFyKiBlbmNvZGlu ZywgY29uc3QgY2hhciogY2Fub25pY2FsQ29udmVydGVyTmFtZSkK