[Edbrowse-dev] [patch] TidyEscapeScripts

[Edbrowse-dev] [patch] TidyEscapeScripts

[Kevin Carhart]

Thanks for the remarks about frames.  I'll revisit that shortly
but let's get tidy issue 348 out of our hair.  Geoff said that his
suggestion for how to implement this is what is shown in the following
patch.  It sets the new tidy option to 'no' by default in edbrowse,
meaning don't parse scripts by default, meaning do make the change by
default.  Whereas when you compile tidy, the new option is set to 
'yes' by default, meaning, leave things as they have been for 
continuity.  So the feature is there but latent until you make the
setting.

Thanks
Kevin

>From c4b58703036dd77db3ce14562ff89e10383b0664 Mon Sep 17 00:00:00 2001
From: Kevin Carhart <kevin@carhart.net>
Date: Wed, 23 Mar 2016 23:12:45 -0700
Subject: [PATCH] sets new tidy opt 'EscapeScripts' to 'no' by default

---
 src/html-tidy.c | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/src/html-tidy.c b/src/html-tidy.c
index 7c086ad..d121c21 100644
--- a/src/html-tidy.c
+++ b/src/html-tidy.c
@@ -119,6 +119,10 @@ void html2nodes(const char *htmltext, bool startpage)
 		tidyOptSetInt(tdoc, TidyBodyOnly, yes);
 	tidySetReportFilter(tdoc, tidyErrorHandler);
 //    tidySetReportFilter(tdoc, tidyReportFilter);
+	
+	// the following tidyOptSetBool implements 
+	// a fix for https://github.com/htacg/tidy-html5/issues/348 
+	tidyOptSetBool( tdoc, TidyEscapeScripts, no );
 
 	tidySetCharEncoding(tdoc, (cons_utf8 ? "utf8" : "latin1"));
 
-- 
1.8.3.2

[Edbrowse-dev] [patch] TidyEscapeScripts

[Karl Dahlke]

Pushed, plus another change to jsrt to verify that tidy 348 is indeed fixed.
We now respect
foo.innerHTML = "<A rhef=blah>click here</a>";
just like any other browser.
Hmm - don't know if the same fix handles </a in a style script.
Or the <textarea> almost script.
Well anyways it looks like a big step forward.
Thanks to the tidy team.

Karl Dahlke