311065 - Live Bookmarks FindTextInNode shouldn't find whitespace-only nodes

Status: RESOLVED DUPLICATE of bug 302133

(Firefox :: Bookmarks & History, defect)

Description

[JongAm Park]

User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.8b5) Gecko/20051003 Firefox/1.4.1
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.8b5) Gecko/20051003 Firefox/1.4.1

After you add one of the links at http://www.hani.co.kr/rss/home_rss-01.html,
which are RSS addresses, you can't see article titles written in Korean in the
live bookmark.

Reproducible: Always

Steps to Reproduce:
1. Add one of the link to your live bookmark.
   Because there is no "Add to Live bookmark", you need to copy existing live
bookmark and change its feed URL to the one you see on the
http://www.hani.co.kr/rss/home_rss-01.html page.

2. Check if the article titles are displayed in the live bookmark.
3.

Actual Results:  
"Black square with a circle" characters are displayed.

Expected Results:  
Article titles in Korean

Probably there can be syntax errors in the RSS pages, but many web sites use
that kind of syntax. So, Firefox should handle it.

Comment 1

[Phil Ringnalda (:philor)]

A not-very-edge-case that bug 253911 didn't handle - I'm surprised not to find
it already reported.

<title>
<![CDATA[The title]]>
</title>

gives us a text-node with a newline, a cdata-section-node with "The title", and
a text-node with a newline, but FindTextInNode is happy enough to stop with the
newline.

Comment 2

[Phil Ringnalda (:philor)]

Attachment: testcase

Comment 3

[Phil Ringnalda (:philor)]

Fixed for Firefox2/Places by bug 321036, would have been fixed by bug 302133 except the non-Places code wasn't ever going to make it to a release... duplicate's close enough, I guess.

*** This bug has been marked as a duplicate of 302133 ***