WebDialog.set_html fails under Safari 5.0.6

fredo6

To all,

Can we summarize what is the problem and what could be the workaround.

From what Driven has signaled, there are problems in both 5.0.3 and 5.0.6 Safari versions.

Also it would be good to hear from the Google SU Team, as they can perfectly implement a central solution that makes set_htmlwork.

Fredo

Dan Rathbun

@thomthom said:

I've not logged anything yet. Been bugging a couple to see if they could shed some light.

OK.. then your on "top of this"... I suppose "they'll" ask you to log a formal bug # if they don't do it themselves.

@Fredo.. agreed, and I hope they'll monitor this topic for ideas.

thomthom

@unknownuser said:

Can we summarize what is the problem and what could be the workaround.
From what Driven has signaled, there are problems in both 5.0.3 and 5.0.6 Safari versions.

I tried to summarize here: http://forums.sketchucation.com/viewtopic.php?f=180&t=39842&p=352375#p352375

Not sure about 5.0.3 - but there has been some discussions on how to format the URLs linking to local resources. We found that file:/// worked on both platforms.

thomthom

@dan rathbun said:

@thomthom said:

I've not logged anything yet. Been bugging a couple to see if they could shed some light.

OK.. then your on "top of this"... I suppose "they'll" ask you to log a formal bug # if they don't do it themselves.

Submitted!

fredo6

@thomthom said:

Not sure about 5.0.3 - but there has been some discussions on how to format the URLs linking to local resources. We found that file:/// worked on both platforms.

The issue with file:// on Mac/Safari was already there since the beginning.

So at the end, the workaround is just to use set_file on Mac

which works on Safari 5.06 (and to be confirmed on 5.03)
which also works on previous versions, though set_html still works in Safari up to 5.01

Is this correct?

Fredo

thomthom

I've used a mix of .set_html and .set_file in my plugins and have not had any problems until 5.0.6 came out.

thomthom

@driven said:

TT_SelectionToys works EXCEPT for 'cheat sheet' which shows as about:blank and loads nothing, not even error messages, so may be a different issue.

I believe it's a different issue as it's an older plugin. Lets look away from this one.

@driven said:

TT_CleanUp3 all now work with the latest patch EXCEPT temp file are not purged on close of dialog or SU.

Really? They should be erased when SketchUp is exited. hmm... Gonna look at it again on my Mac.

driven

on my computers, both running OSX 10.5.6.

under Safari 5.0.3

Fredo's WD all work correctly.
TT_SelectionToys works EXCEPT for 'cheat sheet' which shows as about:blank and loads nothing, not even error messages, so may be a different issue.
TT_CleanUp3 all works with original file (will test new one now) UPDATE: new file works

under Safari 5.0.6
Fredo's WD failed to load images and failed on check for updates . with the temp file test, it will now switch from about:blank if a manual reset is induced, It THEN works correctly.
TT_SelectionToys works EXCEPT [as above]
TT_CleanUp3 all now work with the latest patch EXCEPT temp file are not purged on close of dialog or SU.
temp files accumulate...

from my testing .set_file and .set_url seem to be interchangeable (with the corrected path), i.e. the same resource from the same source can be used with either method.

john

driven

@TT new patch works on Safari 5.0.3 as well, updated above... and temp files also don't purge.

thomthom

It appears that the Garbage Collector is no invoked when SketchUp exits under OSX - as oppose to under Windows. Gonna have to look for another way to cleanup.

thomthom

Cursed - this is more troublesome than I thought.
Under Windows I can clean up all temps by using AppObserver.onQuit - but under OSX it appear to never trigger.

driven

SketchUpUndo0.log is an on quit event, so it is possible. and I think Fredo's DebugLog class purges the old trace files in his tmpdir

/Library/Application Support/Google SketchUp 8/SketchUp/plugins/LIBFREDO6_Dir_40/Lib6PluginDialogs.rb line1047

thomthom

I got lots of Fredo tracelogs in my Window folder...

thomthom

@driven said:

SketchUpUndo0.log is an on quit event

Doesn't seem to trigger from the Ruby API...

thomthom

I'm finding no way to get anything to trigger when OSX exits. onDelete model triggers when you close a model, but not upon exit. It's like all Ruby callbacks are stopped when SU exits under OSX.

driven

sorry if I'm barking up the wrong tree, but this bit of code overwrites an existing file.
could something similar not work for .html's

	@sfile = "/Library/Application\ Support/Google\ SketchUp\ 8/SketchUp/plugins/audit_mac_ruby/audit.txt "
		@debug_file = File.new(@sfile, "w")

john

fredo6

@thomthom said:

I got lots of Fredo tracelogs in my Window folder...

Normally LibFredo6 keeps the last 10 or 15 log files (purge is done above).
This is to allow consulting traces from previous sessions.

Fredo

fredo6

@thomthom said:

Cursed - this is more troublesome than I thought.
Under Windows I can clean up all temps by using AppObserver.onQuit - but under OSX it appear to never trigger.

You should not worry too much with the temporary file.
You create it with a unique name and then delete it when you close the dialog box (I guess you have a hook in the on_close() event).
And if there are a few files remaining, the user or the system will purge them at some points.

The question I had on temp file is whether the html file should absolutely be in the ENV["TMPDIR"] folder or in any folder (like /tmp)

Fredo

driven

@unknownuser said:

@thomthom said:

The question I had on temp file is whether the html file should absolutely be in the ENV["TMPDIR"] folder or in any folder (like /tmp)

Fredo

I just did a test and

@tempfile = File.join( "/Library/Application Support/Google\ SketchUp\ 8/SketchUp/plugins/TT_Lib2/webdialog", filename )

runs the same as the former, but I can find it easier...

thomthom

I'm thinking I might not need temp files at all. I just need to send the HTML to the webdialog via .execute_script to a proxy HTML document that replaces the content.