After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 672192 - Support the Opera Web Browser
Support the Opera Web Browser
Status: RESOLVED FIXED
Product: website
Classification: Infrastructure
Component: extensions.gnome.org
beta
Other Linux
: Normal normal
: ---
Assigned To: Shell extensions maintainer(s)
Shell extensions maintainer(s)
Depends on:
Blocks:
 
 
Reported: 2012-03-15 21:03 UTC by connerleecml
Modified: 2012-05-29 22:57 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
opera screenshot gnome extensions website (102.49 KB, image/png)
2012-05-14 19:01 UTC, connerleecml
  Details
browser-plugin: Fix Opera support (1.07 KB, patch)
2012-05-14 21:46 UTC, Jasper St. Pierre (not reading bugmail)
committed Details | Review

Description connerleecml 2012-03-15 21:03:54 UTC
I am using Opera 11.61 and I can not use the gnome shell extensions website. I realize the web browser is proprietary but I hope this can be fixed. If I run opera from terminal and go the website, I get no output. If I click an extension I do not get the on and off switch.
Comment 1 Jasper St. Pierre (not reading bugmail) 2012-03-15 21:07:30 UTC
What version of GNOME Shell are you using?
Comment 2 connerleecml 2012-03-15 22:24:31 UTC
gnome-shell 3.3.90
Comment 3 Jasper St. Pierre (not reading bugmail) 2012-03-15 23:16:02 UTC
Fixed in git.
Comment 4 connerleecml 2012-05-14 18:53:14 UTC
Did this fix make it into gnome 3.4, because if so, it is still not working for me. GNOME Shell 3.4.1, Opera 11.64. The plugin is definitely enabled.
Comment 5 Jasper St. Pierre (not reading bugmail) 2012-05-14 18:55:39 UTC
It should have. Is there anything in ~/.xsession-errors, or if you start Opera from the command line?
Comment 6 connerleecml 2012-05-14 19:01:05 UTC
Created attachment 214021 [details]
opera screenshot gnome extensions website
Comment 7 Jasper St. Pierre (not reading bugmail) 2012-05-14 19:05:32 UTC
Open up the JS console and enter "window.SweetTooth.apiVersion". What does it say?
Comment 8 connerleecml 2012-05-14 19:16:04 UTC
Not sure I went to the right console. I went to opera dragon fly and clicked console and entered it. 


Unhandled error: cannot convert 'window.SweetTooth' to object
Comment 9 Jasper St. Pierre (not reading bugmail) 2012-05-14 19:18:55 UTC
Oh boy. What does "window.SweetTooth" print out? null? undefined? Make sure that you're on the extensions page when you go to the console.
Comment 10 connerleecml 2012-05-14 19:23:32 UTC
<embed type="application/x-gnome-shell-integration" style="position: absolute; left: -1000em; top: -1000em"/>
Comment 11 Jasper St. Pierre (not reading bugmail) 2012-05-14 20:04:16 UTC
OK, I went and grabbed the latest copy of Opera. I had grabbed whatever copy I had before, and it was fixed, and now it's broken with the new one. Considering Opera never calls our NP_Initialize function (even though it calls NP_GetValue/NP_GetMIMEDescription) it looks like this is an Opera bug. I'm trying to reach the developers on IRC right now.
Comment 12 Jasper St. Pierre (not reading bugmail) 2012-05-14 21:46:07 UTC
Created attachment 214036 [details] [review]
browser-plugin: Fix Opera support

Opera now tries to call NP_SetWindow without trying to check
if it's a NULL pointer or not.


This is the browser-plugin fix. Unfortunately, the website still has
issues because we try to insert the plugin before the DOM has loaded.
It's not easy to fix nicely quite yet, but I'm working on it.
Comment 13 drago01 2012-05-29 22:17:48 UTC
Review of attachment 214036 [details] [review]:

Looks fine, not sure whether Opera's behavior is correct or not but this fix shouldn't hurt.
Comment 14 Jasper St. Pierre (not reading bugmail) 2012-05-29 22:57:32 UTC
Attachment 214036 [details] pushed as 5e4edac - browser-plugin: Fix Opera support


Yeah, I talked to the Opera team, and they said they're not going to change it.