35202 2010-02-20 18:42:25 -0800 Calling Java method which accepts Object always passes a null argument 2010-02-20 22:02:07 -0800 1 1 1 Unclassified WebKit Java 528+ (Nightly build) Mac All RESOLVED FIXED InRadar P2 Normal --- 1 ap ap oldest_to_newest 192226 0 ap 2010-02-20 18:42:25 -0800 Bug 9761 said that converting from JS to Java with target type java.lang.Object should produce a String object. This is actually more general, other types also need to be converted to appropriate objects. <rdar://problem/4856597> 192229 1 ap 2010-02-20 18:42:54 -0800 *** Bug 35203 has been marked as a duplicate of this bug. *** 192230 2 49139 ap 2010-02-20 18:45:04 -0800 Created attachment 49139 proposed fix 192233 3 49139 mjs 2010-02-20 20:56:37 -0800 Comment on attachment 49139 proposed fix r=me 192236 4 ap 2010-02-20 22:02:07 -0800 Committed <http://trac.webkit.org/changeset/55059>. 49139 2010-02-20 18:45:04 -0800 2010-02-20 20:56:36 -0800 proposed fix JavaObject.txt text/plain 5487 ap SW5kZXg6IFdlYkNvcmUvQ2hhbmdlTG9nCj09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIFdlYkNvcmUvQ2hhbmdlTG9n CShyZXZpc2lvbiA1NTA1NykKKysrIFdlYkNvcmUvQ2hhbmdlTG9nCSh3b3JraW5nIGNvcHkpCkBA IC0xLDMgKzEsMTMgQEAKKzIwMTAtMDItMjAgIEFsZXhleSBQcm9za3VyeWFrb3YgIDxhcEBhcHBs ZS5jb20+CisKKyAgICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BTISkuCisKKyAgICAgICAg aHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19idWcuY2dpP2lkPTM1MjAyCisgICAgICAgIDxy ZGFyOi8vcHJvYmxlbS80ODU2NTk3PiBDYWxsaW5nIEphdmEgbWV0aG9kIHdoaWNoIGFjY2VwdHMg T2JqZWN0IGFsd2F5cyBwYXNzZXMgYSBudWxsIGFyZ3VtZW50CisKKyAgICAgICAgKiBicmlkZ2Uv am5pL2pzYy9KTklVdGlsaXR5UHJpdmF0ZS5jcHA6IChKU0M6OkJpbmRpbmdzOjpjb252ZXJ0VmFs dWVUb0pWYWx1ZSk6IEFkZCBjYXNlcyBmb3IKKyAgICAgICAgb3RoZXIgSlMgdHlwZXMuCisKIDIw MTAtMDItMjAgIE5vYW0gUm9zZW50aGFsICA8bm9hbS5yb3NlbnRoYWxAbm9raWEuY29tPgogCiAg ICAgICAgIFJldmlld2VkIGJ5IExhc3psbyBHb21ib3MuCkluZGV4OiBXZWJDb3JlL2JyaWRnZS9q bmkvanNjL0pOSVV0aWxpdHlQcml2YXRlLmNwcAo9PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBXZWJDb3JlL2JyaWRn ZS9qbmkvanNjL0pOSVV0aWxpdHlQcml2YXRlLmNwcAkocmV2aXNpb24gNTUwNTcpCisrKyBXZWJD b3JlL2JyaWRnZS9qbmkvanNjL0pOSVV0aWxpdHlQcml2YXRlLmNwcAkod29ya2luZyBjb3B5KQpA QCAtMTczLDEyICsxNzMsMTQgQEAganZhbHVlIGNvbnZlcnRWYWx1ZVRvSlZhbHVlKEV4ZWNTdGF0 ZSogZQogICAgIEpTTG9jayBsb2NrKFNpbGVuY2VBc3NlcnRpb25zT25seSk7CiAKICAgICBqdmFs dWUgcmVzdWx0OworICAgIGJ6ZXJvKCZyZXN1bHQsIHNpemVvZihqdmFsdWUpKTsKIAogICAgIHN3 aXRjaCAoam5pVHlwZSkgewogICAgIGNhc2UgYXJyYXlfdHlwZToKICAgICBjYXNlIG9iamVjdF90 eXBlOgogICAgICAgICB7Ci0gICAgICAgICAgICByZXN1bHQubCA9IChqb2JqZWN0KTA7CisgICAg ICAgICAgICAvLyBGSVhNRTogSmF2YUpTT2JqZWN0Ojpjb252ZXJ0VmFsdWVUb0pPYmplY3QgZnVu Y3Rpb25hbGl0eSBpcyBhbG1vc3QgZXhhY3RseSB0aGUgc2FtZSwKKyAgICAgICAgICAgIC8vIHRo ZXNlIGZ1bmN0aW9ucyBzaG91bGQgdXNlIGNvbW1vbiBjb2RlLgogCiAgICAgICAgICAgICAvLyBG aXJzdCBzZWUgaWYgd2UgaGF2ZSBhIEphdmEgaW5zdGFuY2UuCiAgICAgICAgICAgICBpZiAodmFs dWUuaXNPYmplY3QoKSkgewpAQCAtMTk5LDI2ICsyMDEsNDQgQEAganZhbHVlIGNvbnZlcnRWYWx1 ZVRvSlZhbHVlKEV4ZWNTdGF0ZSogZQogICAgICAgICAgICAgICAgIH0KICAgICAgICAgICAgIH0K IAotICAgICAgICAgICAgaWYgKCFyZXN1bHQubCkgewotICAgICAgICAgICAgICAgIC8vIENvbnZl cnQgdmFsdWUgdG8gYSBzdHJpbmcgaWYgdGhlIHRhcmdldCB0eXBlIGlzIGEgamF2YS5sYW5nLnN0 cmluZywgYW5kIHdlJ3JlIG5vdAotICAgICAgICAgICAgICAgIC8vIGNvbnZlcnRpbmcgZnJvbSBh IG51bGwuCi0gICAgICAgICAgICAgICAgaWYgKCFzdHJjbXAoamF2YUNsYXNzTmFtZSwgImphdmEu bGFuZy5TdHJpbmciKSkgewotICAgICAgICAgICAgICAgICAgICBpZiAoIXZhbHVlLmlzTnVsbCgp KSB7Ci0gICAgICAgICAgICAgICAgICAgICAgICBVU3RyaW5nIHN0cmluZ1ZhbHVlID0gdmFsdWUu dG9TdHJpbmcoZXhlYyk7Ci0gICAgICAgICAgICAgICAgICAgICAgICBKTklFbnYqIGVudiA9IGdl dEpOSUVudigpOwotICAgICAgICAgICAgICAgICAgICAgICAgam9iamVjdCBqYXZhU3RyaW5nID0g ZW52LT5mdW5jdGlvbnMtPk5ld1N0cmluZyhlbnYsIChjb25zdCBqY2hhciopc3RyaW5nVmFsdWUu ZGF0YSgpLCBzdHJpbmdWYWx1ZS5zaXplKCkpOwotICAgICAgICAgICAgICAgICAgICAgICAgcmVz dWx0LmwgPSBqYXZhU3RyaW5nOwotICAgICAgICAgICAgICAgICAgICB9Ci0gICAgICAgICAgICAg ICAgLy8gSlMgc3RyaW5ncyBhcmUgY29udmVydGVkIHRvIEphdmEgc3RyaW5ncyBpZiBhcmd1bWVu dCB0eXBlIGlzIE9iamVjdC4KLSAgICAgICAgICAgICAgICB9IGVsc2UgaWYgKCFzdHJjbXAoamF2 YUNsYXNzTmFtZSwgImphdmEubGFuZy5PYmplY3QiKSkgewotICAgICAgICAgICAgICAgICAgICBp ZiAodmFsdWUuaXNTdHJpbmcoKSkgewotICAgICAgICAgICAgICAgICAgICAgICAgVVN0cmluZyBz dHJpbmdWYWx1ZSA9IGFzU3RyaW5nKHZhbHVlKS0+dmFsdWUoZXhlYyk7Ci0gICAgICAgICAgICAg ICAgICAgICAgICBKTklFbnYqIGVudiA9IGdldEpOSUVudigpOwotICAgICAgICAgICAgICAgICAg ICAgICAgam9iamVjdCBqYXZhU3RyaW5nID0gZW52LT5mdW5jdGlvbnMtPk5ld1N0cmluZyhlbnYs IChjb25zdCBqY2hhciopc3RyaW5nVmFsdWUuZGF0YSgpLCBzdHJpbmdWYWx1ZS5zaXplKCkpOwot ICAgICAgICAgICAgICAgICAgICAgICAgcmVzdWx0LmwgPSBqYXZhU3RyaW5nOwotICAgICAgICAg ICAgICAgICAgICB9Ci0gICAgICAgICAgICAgICAgfSBlbHNlCi0gICAgICAgICAgICAgICAgICAg IGJ6ZXJvKCZyZXN1bHQsIHNpemVvZihqdmFsdWUpKTsgLy8gSGFuZGxlIGl0IHRoZSBzYW1lIGFz IGEgdm9pZCBjYXNlCisgICAgICAgICAgICAvLyBDcmVhdGUgYW4gYXBwcm9wcmlhdGUgSmF2YSBv YmplY3QgaWYgdGFyZ2V0IHR5cGUgaXMgamF2YS5sYW5nLk9iamVjdC4KKyAgICAgICAgICAgIGlm ICghcmVzdWx0LmwgJiYgIXN0cmNtcChqYXZhQ2xhc3NOYW1lLCAiamF2YS5sYW5nLk9iamVjdCIp KSB7CisgICAgICAgICAgICAgICAgaWYgKHZhbHVlLmlzU3RyaW5nKCkpIHsKKyAgICAgICAgICAg ICAgICAgICAgVVN0cmluZyBzdHJpbmdWYWx1ZSA9IGFzU3RyaW5nKHZhbHVlKS0+dmFsdWUoZXhl Yyk7CisgICAgICAgICAgICAgICAgICAgIEpOSUVudiogZW52ID0gZ2V0Sk5JRW52KCk7CisgICAg ICAgICAgICAgICAgICAgIGpvYmplY3QgamF2YVN0cmluZyA9IGVudi0+ZnVuY3Rpb25zLT5OZXdT dHJpbmcoZW52LCAoY29uc3QgamNoYXIqKXN0cmluZ1ZhbHVlLmRhdGEoKSwgc3RyaW5nVmFsdWUu c2l6ZSgpKTsKKyAgICAgICAgICAgICAgICAgICAgcmVzdWx0LmwgPSBqYXZhU3RyaW5nOworICAg ICAgICAgICAgICAgIH0gZWxzZSBpZiAodmFsdWUuaXNOdW1iZXIoKSkgeworICAgICAgICAgICAg ICAgICAgICBkb3VibGUgZG91YmxlVmFsdWUgPSB2YWx1ZS51bmNoZWNrZWRHZXROdW1iZXIoKTsK KyAgICAgICAgICAgICAgICAgICAgSk5JRW52KiBlbnYgPSBnZXRKTklFbnYoKTsKKyAgICAgICAg ICAgICAgICAgICAgamNsYXNzIGNsYXp6ID0gZW52LT5GaW5kQ2xhc3MoImphdmEvbGFuZy9Eb3Vi bGUiKTsKKyAgICAgICAgICAgICAgICAgICAgam1ldGhvZElEIGNvbnN0cnVjdG9yID0gZW52LT5H ZXRNZXRob2RJRChjbGF6eiwgIjxpbml0PiIsICIoRClWIik7CisgICAgICAgICAgICAgICAgICAg IGpvYmplY3QgamF2YURvdWJsZSA9IGVudi0+ZnVuY3Rpb25zLT5OZXdPYmplY3QoZW52LCBjbGF6 eiwgY29uc3RydWN0b3IsIGRvdWJsZVZhbHVlKTsKKyAgICAgICAgICAgICAgICAgICAgcmVzdWx0 LmwgPSBqYXZhRG91YmxlOworICAgICAgICAgICAgICAgIH0gZWxzZSBpZiAodmFsdWUuaXNCb29s ZWFuKCkpIHsKKyAgICAgICAgICAgICAgICAgICAgYm9vbCBib29sVmFsdWUgPSB2YWx1ZS5nZXRC b29sZWFuKCk7CisgICAgICAgICAgICAgICAgICAgIEpOSUVudiogZW52ID0gZ2V0Sk5JRW52KCk7 CisgICAgICAgICAgICAgICAgICAgIGpjbGFzcyBjbGF6eiA9IGVudi0+RmluZENsYXNzKCJqYXZh L2xhbmcvQm9vbGVhbiIpOworICAgICAgICAgICAgICAgICAgICBqbWV0aG9kSUQgY29uc3RydWN0 b3IgPSBlbnYtPkdldE1ldGhvZElEKGNsYXp6LCAiPGluaXQ+IiwgIihaKVYiKTsKKyAgICAgICAg ICAgICAgICAgICAgam9iamVjdCBqYXZhQm9vbGVhbiA9IGVudi0+ZnVuY3Rpb25zLT5OZXdPYmpl Y3QoZW52LCBjbGF6eiwgY29uc3RydWN0b3IsIGJvb2xWYWx1ZSk7CisgICAgICAgICAgICAgICAg ICAgIHJlc3VsdC5sID0gamF2YUJvb2xlYW47CisgICAgICAgICAgICAgICAgfSBlbHNlIGlmICh2 YWx1ZS5pc1VuZGVmaW5lZCgpKSB7CisgICAgICAgICAgICAgICAgICAgIFVTdHJpbmcgc3RyaW5n VmFsdWUgPSAidW5kZWZpbmVkIjsKKyAgICAgICAgICAgICAgICAgICAgSk5JRW52KiBlbnYgPSBn ZXRKTklFbnYoKTsKKyAgICAgICAgICAgICAgICAgICAgam9iamVjdCBqYXZhU3RyaW5nID0gZW52 LT5mdW5jdGlvbnMtPk5ld1N0cmluZyhlbnYsIChjb25zdCBqY2hhciopc3RyaW5nVmFsdWUuZGF0 YSgpLCBzdHJpbmdWYWx1ZS5zaXplKCkpOworICAgICAgICAgICAgICAgICAgICByZXN1bHQubCA9 IGphdmFTdHJpbmc7CisgICAgICAgICAgICAgICAgfQorICAgICAgICAgICAgfQorCisgICAgICAg ICAgICAvLyBDb252ZXJ0IHZhbHVlIHRvIGEgc3RyaW5nIGlmIHRoZSB0YXJnZXQgdHlwZSBpcyBh IGphdmEubGFuZy5TdHJpbmcsIGFuZCB3ZSdyZSBub3QKKyAgICAgICAgICAgIC8vIGNvbnZlcnRp bmcgZnJvbSBhIG51bGwuCisgICAgICAgICAgICBpZiAoIXJlc3VsdC5sICYmICFzdHJjbXAoamF2 YUNsYXNzTmFtZSwgImphdmEubGFuZy5TdHJpbmciKSkgeworICAgICAgICAgICAgICAgIGlmICgh dmFsdWUuaXNOdWxsKCkpIHsKKyAgICAgICAgICAgICAgICAgICAgVVN0cmluZyBzdHJpbmdWYWx1 ZSA9IHZhbHVlLnRvU3RyaW5nKGV4ZWMpOworICAgICAgICAgICAgICAgICAgICBKTklFbnYqIGVu diA9IGdldEpOSUVudigpOworICAgICAgICAgICAgICAgICAgICBqb2JqZWN0IGphdmFTdHJpbmcg PSBlbnYtPmZ1bmN0aW9ucy0+TmV3U3RyaW5nKGVudiwgKGNvbnN0IGpjaGFyKilzdHJpbmdWYWx1 ZS5kYXRhKCksIHN0cmluZ1ZhbHVlLnNpemUoKSk7CisgICAgICAgICAgICAgICAgICAgIHJlc3Vs dC5sID0gamF2YVN0cmluZzsKKyAgICAgICAgICAgICAgICB9CiAgICAgICAgICAgICB9CiAgICAg ICAgIH0KICAgICAgICAgYnJlYWs7CkBAIC0yNzEsMTQgKzI5MSw4IEBAIGp2YWx1ZSBjb252ZXJ0 VmFsdWVUb0pWYWx1ZShFeGVjU3RhdGUqIGUKICAgICAgICAgfQogICAgICAgICBicmVhazsKIAot ICAgICAgICBicmVhazsKLQogICAgIGNhc2UgaW52YWxpZF90eXBlOgotICAgIGRlZmF1bHQ6CiAg ICAgY2FzZSB2b2lkX3R5cGU6Ci0gICAgICAgIHsKLSAgICAgICAgICAgIGJ6ZXJvKCZyZXN1bHQs IHNpemVvZihqdmFsdWUpKTsKLSAgICAgICAgfQogICAgICAgICBicmVhazsKICAgICB9CiAgICAg cmV0dXJuIHJlc3VsdDsK