Bommels05/Mozilla
Bommels05/Mozilla
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Bug 341837 microsummary service should impose limit on length of microsummary, r=myk

Browse Source 
git-svn-id: svn://10.0.0.236/trunk@203515 18797224-902f-48f8-a5cc-f745e15eee43
This commit is contained in:
jminta%gmail.com 2006-07-24 18:01:04 +00:00
parent 2ea181699d
commit bea24ef4ea
1 changed files with 16 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

21
mozilla/browser/components/microsummaries/src/nsMicrosummaryService.js.in
View File

@ -73,6 +73,8 @@ const FIELD_CONTENT_TYPE = NC_NS + "ContentType";
const FIELD_BOOKMARK_URL = NC_NS + "URL";
#endif
const MAX_SUMMARY_LENGTH = 4096;
function MicrosummaryService() {}
MicrosummaryService.prototype = {
@ -1282,11 +1284,16 @@ MicrosummaryGenerator.prototype = {
},
generateMicrosummary: function MSD_generateMicrosummary(pageContent) {
if (this.content)
return this.content.replace(/^\s+|\s+$/g, "");
else if (this.template)
if (this.content) {
var content = this.content.replace(/^\s+|\s+$/g, "");
if (content.length > MAX_SUMMARY_LENGTH)
content = content.substring(0, MAX_SUMMARY_LENGTH);
return content;
} else if (this.template) {
// Note that _processTemplate truncates the summary for us, so we don't
// need to do anything more here
return this._processTemplate(pageContent);
else
} else
throw("generateMicrosummary called on uninitialized microsummary generator");
},
@ -1328,7 +1335,11 @@ MicrosummaryGenerator.prototype = {
// XXX When we support HTML microsummaries we'll need to do something
// more sophisticated than just returning the text content of the fragment.
return fragment.textContent;
var content = fragment.textContent;
if (content.length > MAX_SUMMARY_LENGTH)
content = content.substring(0, MAX_SUMMARY_LENGTH);
return content;
}
};