WebDialog.set_html fails under Safari 5.0.6

thomthom · 4 Sept 2011, 06:59

@dan rathbun said:

Can these meta tags be injected to prevent caching:
%(#8000BF)[<META HTTP-EQUIV="Pragma" CONTENT="no-cache"> <META HTTP-EQUIV="Expires" CONTENT="-1">]

ref: How to prevent caching in Internet Explorer

I did consider sending a refresh comment to JS, but I figured I'd follow the pattern of how .set_html works on Windows, which is creating a temp file with a unique name.

Dan Rathbun · 4 Sept 2011, 07:41

@dan rathbun said:

Has anyone logged this with Google ?? I'd think they'd wish to release patches if Safari 5.0.6 will break WebDialogs on all Mac versions.

TT ?

thomthom · 4 Sept 2011, 08:15

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

thomthom · 4 Sept 2011, 08:21

Driven: I implemented a patch. Update to TT_Lib 2.5.5
http://forums.sketchucation.com/viewtopic.php?f=323&t=30503

fredo6 · 4 Sept 2011, 09:11

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 · 4 Sept 2011, 09:12

@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 · 4 Sept 2011, 09:21

@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 · 4 Sept 2011, 09:39

@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 · 4 Sept 2011, 15:01

@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 · 4 Sept 2011, 15:09

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 · 4 Sept 2011, 15:37

@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 · 4 Sept 2011, 15:53

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.

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 · 4 Sept 2011, 15:54

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

thomthom · 4 Sept 2011, 17:19

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 · 4 Sept 2011, 17:41

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 · 4 Sept 2011, 18:17

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 · 4 Sept 2011, 18:27

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

thomthom · 4 Sept 2011, 18:28

@driven said:

SketchUpUndo0.log is an on quit event

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

thomthom · 4 Sept 2011, 18:36

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 · 4 Sept 2011, 18:59

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