16442 2007-12-14 15:10:31 -0800 navigation policy delegate gets called twice for each load 2007-12-14 16:08:16 -0800 1 1 1 Unclassified WebKit Page Loading 528+ (Nightly build) Mac OS X 10.4 RESOLVED FIXED P2 Normal --- 1 darin darin oldest_to_newest 64213 0 darin 2007-12-14 15:10:31 -0800 I noticed that the navigation policy delegate gets called twice for each load. One call is from FrameLoader and the other is from MainResourceLoader. 64214 1 17897 darin 2007-12-14 15:14:10 -0800 Created attachment 17897 patch 64224 2 darin 2007-12-14 16:08:16 -0800 Committed revision 28726. 17897 2007-12-14 15:14:10 -0800 2007-12-14 15:31:51 -0800 patch PolicyDelegatePatch.txt text/plain 3403 darin SW5kZXg6IFdlYkNvcmUvQ2hhbmdlTG9nCj09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIFdlYkNvcmUvQ2hhbmdlTG9n CShyZXZpc2lvbiAyODcyMykKKysrIFdlYkNvcmUvQ2hhbmdlTG9nCSh3b3JraW5nIGNvcHkpCkBA IC0xLDMgKzEsMTcgQEAKKzIwMDctMTItMTQgIERhcmluIEFkbGVyICA8ZGFyaW5AYXBwbGUuY29t PgorCisgICAgICAgIFJldmlld2VkIGJ5IE5PQk9EWSAoT09QUyEpLgorCisgICAgICAgIC0gZml4 IGh0dHA6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19idWcuY2dpP2lkPTE2NDQyCisgICAgICAgICAg bmF2aWdhdGlvbiBwb2xpY3kgZGVsZWdhdGUgZ2V0cyBjYWxsZWQgdHdpY2UgZm9yIGVhY2ggbG9h ZAorCisgICAgICAgICogbG9hZGVyL01haW5SZXNvdXJjZUxvYWRlci5jcHA6CisgICAgICAgIChX ZWJDb3JlOjpNYWluUmVzb3VyY2VMb2FkZXI6OndpbGxTZW5kUmVxdWVzdCk6IFJlbW92ZWQgY2Fs bCB0byBjaGVja05hdmlnYXRpb25Qb2xpY3kuCisgICAgICAgIFRoYXQncyBoYW5kbGVkIGJ5IEZy YW1lTG9hZGVyLgorCisgICAgICAgICogbG9hZGVyL01haW5SZXNvdXJjZUxvYWRlci5oOiBSZW1v dmVkIGNhbGxDb250aW51ZUFmdGVyTmF2aWdhdGlvblBvbGljeQorICAgICAgICBhbmQgY29udGlu dWVBZnRlck5hdmlnYXRpb25Qb2xpY3kuCisKIDIwMDctMTItMTQgIEFuZGVycyBDYXJsc3NvbiAg PGFuZGVyc2NhQGFwcGxlLmNvbT4KIAogICAgICAgICBSZXZpZXdlZCBieSBCcmFkeS4KSW5kZXg6 IFdlYkNvcmUvbG9hZGVyL01haW5SZXNvdXJjZUxvYWRlci5jcHAKPT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PQotLS0gV2Vi Q29yZS9sb2FkZXIvTWFpblJlc291cmNlTG9hZGVyLmNwcAkocmV2aXNpb24gMjg3MjMpCisrKyBX ZWJDb3JlL2xvYWRlci9NYWluUmVzb3VyY2VMb2FkZXIuY3BwCSh3b3JraW5nIGNvcHkpCkBAIC0x MDYsMTggKzEwNiw2IEBAIHZvaWQgTWFpblJlc291cmNlTG9hZGVyOjpzdG9wTG9hZGluZ0ZvclAK ICAgICBjYW5jZWwoaW50ZXJydXB0aW9uRm9yUG9saWN5Q2hhbmdlRXJyb3IoKSk7CiB9CiAKLXZv aWQgTWFpblJlc291cmNlTG9hZGVyOjpjYWxsQ29udGludWVBZnRlck5hdmlnYXRpb25Qb2xpY3ko dm9pZCogYXJndW1lbnQsIGNvbnN0IFJlc291cmNlUmVxdWVzdCYgcmVxdWVzdCwgUGFzc1JlZlB0 cjxGb3JtU3RhdGU+LCBib29sIHNob3VsZENvbnRpbnVlKQotewotICAgIHN0YXRpY19jYXN0PE1h aW5SZXNvdXJjZUxvYWRlcio+KGFyZ3VtZW50KS0+Y29udGludWVBZnRlck5hdmlnYXRpb25Qb2xp Y3kocmVxdWVzdCwgc2hvdWxkQ29udGludWUpOwotfQotCi12b2lkIE1haW5SZXNvdXJjZUxvYWRl cjo6Y29udGludWVBZnRlck5hdmlnYXRpb25Qb2xpY3koY29uc3QgUmVzb3VyY2VSZXF1ZXN0JiBy ZXF1ZXN0LCBib29sIHNob3VsZENvbnRpbnVlKQotewotICAgIGlmICghc2hvdWxkQ29udGludWUp Ci0gICAgICAgIHN0b3BMb2FkaW5nRm9yUG9saWN5Q2hhbmdlKCk7Ci0gICAgZGVyZWYoKTsgLy8g YmFsYW5jZXMgcmVmIGluIHdpbGxTZW5kUmVxdWVzdAotfQotCiBib29sIE1haW5SZXNvdXJjZUxv YWRlcjo6aXNQb3N0T3JSZWRpcmVjdEFmdGVyUG9zdChjb25zdCBSZXNvdXJjZVJlcXVlc3QmIG5l d1JlcXVlc3QsIGNvbnN0IFJlc291cmNlUmVzcG9uc2UmIHJlZGlyZWN0UmVzcG9uc2UpCiB7CiAg ICAgaWYgKG5ld1JlcXVlc3QuaHR0cE1ldGhvZCgpID09ICJQT1NUIikKQEAgLTE2OCw5ICsxNTYs NiBAQCB2b2lkIE1haW5SZXNvdXJjZUxvYWRlcjo6d2lsbFNlbmRSZXF1ZXN0CiAgICAgCiAgICAg Ly8gRG9uJ3Qgc2V0IHRoaXMgb24gdGhlIGZpcnN0IHJlcXVlc3QuIEl0IGlzIHNldCB3aGVuIHRo ZSBtYWluIGxvYWQgd2FzIHN0YXJ0ZWQuCiAgICAgbV9kb2N1bWVudExvYWRlci0+c2V0UmVxdWVz dChuZXdSZXF1ZXN0KTsKLSAgICAKLSAgICByZWYoKTsgLy8gYmFsYW5jZWQgYnkgZGVyZWYgaW4g Y29udGludWVBZnRlck5hdmlnYXRpb25Qb2xpY3kKLSAgICBmcmFtZUxvYWRlcigpLT5jaGVja05h dmlnYXRpb25Qb2xpY3kobmV3UmVxdWVzdCwgY2FsbENvbnRpbnVlQWZ0ZXJOYXZpZ2F0aW9uUG9s aWN5LCB0aGlzKTsKIH0KIAogc3RhdGljIGJvb2wgc2hvdWxkTG9hZEFzRW1wdHlEb2N1bWVudChj b25zdCBLVVJMJiB1cmwpCkluZGV4OiBXZWJDb3JlL2xvYWRlci9NYWluUmVzb3VyY2VMb2FkZXIu aAo9PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09Ci0tLSBXZWJDb3JlL2xvYWRlci9NYWluUmVzb3VyY2VMb2FkZXIuaAkocmV2 aXNpb24gMjg3MjMpCisrKyBXZWJDb3JlL2xvYWRlci9NYWluUmVzb3VyY2VMb2FkZXIuaAkod29y a2luZyBjb3B5KQpAQCAtMSw1ICsxLDUgQEAKIC8qCi0gKiBDb3B5cmlnaHQgKEMpIDIwMDUsIDIw MDYgQXBwbGUgQ29tcHV0ZXIsIEluYy4gIEFsbCByaWdodHMgcmVzZXJ2ZWQuCisgKiBDb3B5cmln aHQgKEMpIDIwMDUsIDIwMDYsIDIwMDcgQXBwbGUgSW5jLiBBbGwgcmlnaHRzIHJlc2VydmVkLgog ICoKICAqIFJlZGlzdHJpYnV0aW9uIGFuZCB1c2UgaW4gc291cmNlIGFuZCBiaW5hcnkgZm9ybXMs IHdpdGggb3Igd2l0aG91dAogICogbW9kaWZpY2F0aW9uLCBhcmUgcGVybWl0dGVkIHByb3ZpZGVk IHRoYXQgdGhlIGZvbGxvd2luZyBjb25kaXRpb25zCkBAIC03NCw5ICs3NCw2IEBAIG5hbWVzcGFj ZSBXZWJDb3JlIHsKICAgICAgICAgdm9pZCBzdG9wTG9hZGluZ0ZvclBvbGljeUNoYW5nZSgpOwog ICAgICAgICBib29sIGlzUG9zdE9yUmVkaXJlY3RBZnRlclBvc3QoY29uc3QgUmVzb3VyY2VSZXF1 ZXN0JiBuZXdSZXF1ZXN0LCBjb25zdCBSZXNvdXJjZVJlc3BvbnNlJiByZWRpcmVjdFJlc3BvbnNl KTsKIAotICAgICAgICBzdGF0aWMgdm9pZCBjYWxsQ29udGludWVBZnRlck5hdmlnYXRpb25Qb2xp Y3kodm9pZCosIGNvbnN0IFJlc291cmNlUmVxdWVzdCYsIFBhc3NSZWZQdHI8Rm9ybVN0YXRlPiwg Ym9vbCBzaG91bGRDb250aW51ZSk7Ci0gICAgICAgIHZvaWQgY29udGludWVBZnRlck5hdmlnYXRp b25Qb2xpY3koY29uc3QgUmVzb3VyY2VSZXF1ZXN0JiwgYm9vbCBzaG91bGRDb250aW51ZSk7Ci0K ICAgICAgICAgc3RhdGljIHZvaWQgY2FsbENvbnRpbnVlQWZ0ZXJDb250ZW50UG9saWN5KHZvaWQq LCBQb2xpY3lBY3Rpb24pOwogICAgICAgICB2b2lkIGNvbnRpbnVlQWZ0ZXJDb250ZW50UG9saWN5 KFBvbGljeUFjdGlvbik7CiAgICAgICAgIHZvaWQgY29udGludWVBZnRlckNvbnRlbnRQb2xpY3ko UG9saWN5QWN0aW9uLCBjb25zdCBSZXNvdXJjZVJlc3BvbnNlJik7Cg==