Sitepatching updates

tweaking the broken code until it works, one site at a time

jQuery.jsonp

By Ola P. Kleivenolak. Tuesday, December 6, 2011 12:45:00 PM

Edit: as Darko Pantić points out below, there was a bug in this patch. Thus a new file has been released with a fix for this.

Added PATCH-554: The jQuery jsonp plugin contains a workaround for Opera's historic lack of script.onerror support. Unfortunately the workaround relies on Opera's old script execution order as well - something that changed with the new HTML5-compatible parser. As a result a lot of content around the web breaks in Presto/2.9.220 onwards. This includes IMDB search suggestions, Hootsuite Twitter search, some Slideshare presentations etc. Even when the plugin is fixed, the propagation will take a while, so we add this generic patch to browser.js.

Removed PATCH-532: Google docs document list jumps back to top on scroll. Managed a last minute Core fix for this regression.

eBayFacebook contact list

Comments

Nimesh nimeshthakkar Monday, December 5, 2011 9:19:08 PM

IMDB search suggestions

up
Good to see this update

Opera Desktop 11.60 core 2.10.229, December 5, 2011. Active patches: 179

Darko Pantićpdarko Monday, December 5, 2011 11:03:05 PM

With new jQuery.jsonp patch (user) scripts break if they try to change scripts.

This will generate error fo every script. 
window.opera.addEventListener("BeforeScript",function (event) {
    event.element.text = event.element.text;
}, false);

// Uncaught exception: TypeError: Failed to call host setter
// Error thrown at line 592, column 3 in <anonymous function>(str) in browser.js:
//     return s.call(this);
// called from line 31, column 1 in <anonymous function>(event) in test.js:
//     event.element.text = event.element.text;

Using Opera 11.60 build 1184.

Hallvord R. M. Steenhallvors Tuesday, December 6, 2011 12:30:43 AM

Originally posted by pdarko:

With new jQuery.jsonp patch (user) scripts break if they try to change scripts.



Oops. Needs investigation (although I first need some sleep..). I think we're hitting some special and extremely obscure security limitations for user JS and getters/setters. There is probably a trivial workaround like setting textContent/doing appendChild instead of calling the setter or something like that..

Hallvord R. M. Steenhallvors Tuesday, December 6, 2011 9:55:35 AM

I'm wrong. Just a very stupid bug in the patch.. s.call(this) should be s.call(this, str). Perhaps I shouldn't write patches at the end of a working day :-p

Ola will get a new release out. Sorry if I've broken your user scripts meanwhile.

Charles SchlossChas4 Tuesday, December 6, 2011 1:49:03 PM

Target version and time stamp of the active browser.js file is Opera Desktop 11.60 core 2.10.229, December 6, 2011. Active patches: 179 .

Write a comment

New comments have been disabled for this post.