WebDialog.set_html fails under Safari 5.0.6

thomthom · 1 Sept 2011, 10:25

however, because you both use convoluted (ruby/js) html creation I can't work out how to specify a simple path.

Because it's not a simple path... it's created dynamically. I'll try to get to the bottom of it.

thomthom · 1 Sept 2011, 10:47

I'm still on 10.5.8, but safari 5.0.6 is the pre-release for the next safari and it stops Fredo's updater from checking here for updates, but I haven't had any local file errors from anything else, and sellectionToys works...

[attachment=2:a5f6zxlv]OSX 10.5.8 + Safari 5.0.png[/attachment:a5f6zxlv]

Any chance you can test something for me:
[attachment=0:a5f6zxlv]window.zip[/attachment:a5f6zxlv]

Replace the old window.html file, and have a look at the error console (in your screenshot) again.
It won't fix all loading errors, but it will give me a clue if it reduces some of the errors.

Extract into TT_Lib2/webdialog/

driven · 1 Sept 2011, 12:45

thomthom · 1 Sept 2011, 12:48

Cursed!

Just to verify, can you send me the source code of the dialog that opens? (Right click > View Source) ?

driven · 1 Sept 2011, 13:14

it's quite small, so here it is...

BTW. you can't view source in SU but you can right click on the html tag in Web Inspector and copy as html (the whole file)

<html xmlns="http://www.w3.org/1999/xhtml" xml;lang="en-gb" lang="en-gb"><head>
	<meta http-equiv="X-UA-Compatible" content="IE=8">
	<meta http-equiv="content-type" content="text/html; charset=utf-8">
	<meta http-equiv="MSThemeCompatible" content="Yes">
	<!--<meta http-equiv="X-UA-Compatible" content="IE=EmulateIE7" />-->
	<!--<meta http-equiv="X-UA-Compatible" content="IE=8"/>-->
	
	<base href="file;///Library/Application Support/Google SketchUp 8/SketchUp/Plugins/TT_Lib2/webdialog/">
	<link rel="stylesheet" type="text/css" media="screen" href="css/ui.css">
	<link rel="stylesheet" type="text/css" media="screen" href="file;///Library/Application Support/Google SketchUp 8/SketchUp/Plugins/TT_Lib2/webdialog/css/inputbox.css">
	<style type="text/css">html{overflow;hidden;}</style>
	<title>%TITLE%</title>
	<script type="text/javascript" src="js/jquery.js"></script>
	<script type="text/javascript" src="js/base.js"></script>
	<script type="text/javascript" src="file;///Library/Application Support/Google SketchUp 8/SketchUp/Plugins/TT_Lib2/webdialog/js/inputbox.js"></script>
	
	<!--
	<script type="text/javascript" src="https://getfirebug.com/firebug-lite.js"></script>
	-->
</head>
<body>



<!-- <a href="#">file;///Library/Application Support/Google SketchUp 8/SketchUp/Plugins/TT_Lib2/webdialog/</a> -->


</body></html>

thomthom · 1 Sept 2011, 13:27

hmm... Right. This looks like a nasty debug session coming up.

I wonder why on earth Safari would prevent loading of resources from file:// ...
Is it the file:/// protocol? Surely this would block viewing local webpages?

driven · 1 Sept 2011, 15:07

it seems to be something to do with the dynamic loading...

selection toys image paths work...

thomthom · 1 Sept 2011, 15:20

@driven said:

it seems to be something to do with the dynamic loading...

selection toys image paths work...

[attachment=0:1zv85099]selectionToys_path.png[/attachment:1zv85099]

This is so odd. I use webdialog.set_html to send it generated HTML. SU then makes a temp file out of that and loads it. So in theory it should not really make a difference... arrrgh... I'm getting flashbacks of IE-hell....

thomthom · 1 Sept 2011, 15:37

@driven said:

it seems to be something to do with the dynamic loading...

selection toys image paths work...

How do you enable the debug panel in Webdialogs under OSX?

fredo6 · 1 Sept 2011, 15:47

@driven said:

it seems to be something to do with the dynamic loading...

selection toys image paths work...

This is rather good news!
At least we may have a first clue that it can work and try identifying a possible workaround.

As Thomthom said, there should be some Html files in the /tmp directory of your Mac (at least while the dialog box is opened). That would be great to get one for Selection Toys and one for a plugin which does not show images.

Alternatively, we could design a small dialog box test plugin showing only one image, so that the code is easier to debug.

Fredo

PS: I guess Safari 5.1 only comes with Lion. Is it correct?

thomthom · 1 Sept 2011, 16:08

Fredo, you have similar problems? With dialogs that use .set_html instead of .set_file ?

thomthom · 1 Sept 2011, 16:39

I just did a test, where instead of using my generated HTML with .set_html I wrote it to a temp file and used .set_file - everything worked fine again.

I really do not understand why it makes a difference, because what I just did as a workaround is exactly what SketchUp does internally with .set_html ...

fredo6 · 1 Sept 2011, 17:00

@thomthom said:

Fredo, you have similar problems? With dialogs that use .set_html instead of .set_file ?

I don't have Safari at hand (and anyway not Safari 5), so it is difficult to say anything.

While it is easy to write the HTML to a temp file and load it via set_file, the problem I see is for the refresh (I use set_html when the web dialog is opened). Maybe it will work as well.

Fredo

thomthom · 1 Sept 2011, 17:12

@unknownuser said:

While it is easy to write the HTML to a temp file and load it via set_file, the problem I see is for the refresh (I use set_html when the web dialog is opened). Maybe it will work as well.

So you also use .set_html ?
I'll be looking into it closer to see if I can make it work with that method, otherwise I have to switch to making a temp file.

As you mention, if you delete the temp file while the window is open you break Refresh.
If you observe the tmep folder when you use .set_html you'll see that SU keeps the temp HTML file around until you call .set_html again for that window object or until SU closes, where it cleans up all tmep files.

Making a cleanup operation from Ruby won't be so easy I think...

thomthom · 1 Sept 2011, 17:17

I tried this:


w=UI;;WebDialog.new "Hello"
#<UI;;WebDialog;0x5ff2fa8>

w.set_html "Hello"
nil

GC.start
nil

w=nil
nil

GC.start
nil

When I invoke CG.start the temp file is erased.
I wonder if it's possible to be notified in Ruby when an object is garbage collected...

thomthom · 1 Sept 2011, 17:20

Maybe this: http://www.ruby-doc.org/core-1.8.6/classes/ObjectSpace.html#M001111 ?

With a gotcha: http://www.mikeperham.com/2010/02/24/the-trouble-with-ruby-finalizers/

fredo6 · 1 Sept 2011, 17:30

Before going too far and change the method of building web dialogs, let's try to find out why images cannot be loaded on Safari 5, whereas

it works fine on previous versions of Safari (but we need the file:// prefix)
it works fine on IE, any version (prefix file:// not required).

So far, the problem seems to be related to the security of Safari or Lion, since the message is that access to local resources is not allowed. And what we just found out is that Safari would react differently is the web dialog is loaded via set_html vs. set_file.

The question is why? By the way, did you try with loading a HTML file from the /tmp directory with set_file?

Fredo

PS: It is true that the browser designers primarily think about 'connected' web pages, with a real web servers, not using HTML and JS as a simple way to build GUI locally on the PC, without any remote connection and web site. Added to that Apple is known to pay relatively little attention to backward compatibility.

Fredo

thomthom · 1 Sept 2011, 18:22

@unknownuser said:

So far, the problem seems to be related to the security of Safari or Lion, since the message is that access to local resources is not allowed. And what we just found out is that Safari would react differently is the web dialog is loaded via set_html vs. set_file.

Strange new find:
w.set_html '<script>alert( window.location )</script>'

Under Windows:
This presents a messagebox with a string to a temp file.

Under OSX with Safari 5.0.6 the messagebox yields "about:blank"
I wonder if that was the case in previous version. Now I can't test...

@unknownuser said:

The question is why? By the way, did you try with loading a HTML file from the /tmp directory with set_file?

That's the thing - I can't find what temp location SU writes to now as window.location reports nothing...

ENV['TMPDIR'] yields /var/folders/jw/jwKHMuVxGEi4OILNAxFfgU+++TI/-Tmp-/
I can't even find that on the disk...

thomthom · 1 Sept 2011, 18:40

Found one of my older posts - I'd forgotten about.
http://forums.sketchucation.com/viewtopic.php?f=180&p=306204#p306122

Similar, expect that I made it work with file:// - now that seems to have been closed down as well.

driven · 1 Sept 2011, 21:04

@thomthom said:

ENV['TMPDIR'] yields /var/folders/jw/jwKHMuVxGEi4OILNAxFfgU+++TI/-Tmp-/ :?
I can't even find that on the disk...

the path to the -Tmp-
Macintosh HD/private/var/etc...

private is a hidden folder

the easiest way to expose hidden folders is HiddenFiles widget softpedia has it

I've got both versions of Safari if you want me to test things..

@fredo, I'll pm you after I test on 5.0.6