Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(287)

Issue 1883041: Adding parseDomForCharsetDetection interface method in GadgetHtmlParser

Can't Edit
Can't Publish+Mail
Start Review
Created:
15 years, 6 months ago by gagan.goku
Modified:
15 years, 6 months ago
Reviewers:
cool-shindig-committers
Base URL:
http://svn.apache.org/repos/asf/shindig/trunk
Visibility:
Public.

Description

Adding a parseDomForCharsetDetection method separate from parseDom method. The objective is to parse out some portion of the initial html to be able to parse the <meta http-equiv> tag for charset encoding detection. This method can do an approximate job of parsing the content as it is only going to be called when we want to detect the charset encoding of the document. FYI: For more robust charset detection, we should think of implementing the encoding sniff algo described here: http://www.whatwg.org/specs/web-apps/current-work/multipage/urls.html#content-type http://www.whatwg.org/specs/web-apps/current-work/multipage/parsing.html

Patch Set 1 #

Patch Set 2 : 'adding_more_documentation' #

Patch Set 3 : svn up #

Unified diffs Side-by-side diffs Delta from patch set Stats (+18 lines, -0 lines) Patch
M java/gadgets/src/main/java/org/apache/shindig/gadgets/parse/GadgetHtmlParser.java View 1 2 1 chunk +18 lines, -0 lines 0 comments Download

Messages

Total messages: 1
gagan.goku
15 years, 6 months ago (2010-07-20 06:14:49 UTC) #1

          
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b