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

Bug 378470 - refreshVisibleSection fails to update viewIndex properly. patch from onemen.one@gmail.com, r=me.

Browse Source 
git-svn-id: svn://10.0.0.236/trunk@224910 18797224-902f-48f8-a5cc-f745e15eee43
This commit is contained in:
mozilla.mano%sent.com 2007-04-23 23:30:11 +00:00
parent d59de8d930
commit f58b8fddb7
1 changed files with 3 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
mozilla/browser/components/places/content/treeView.js
View File

@ -318,7 +318,7 @@ PlacesTreeView.prototype = {
replaceCount-=1;
// Mark the removees as invisible
for (var i = startReplacement; i < replaceCount; i ++)
for (var i = 0; i < replaceCount; i ++)
this._visibleElements[startReplacement + i].viewIndex = -1;
// Building the new list will set the new elements' visible indices.
@ -735,7 +735,7 @@ PlacesTreeView.prototype = {
set collapseDuplicates(val) {
if (this._collapseDuplicates == val)
return; // no change;
return val; // no change;
this._collapseDuplicates = val;
if (this._tree && this._result)
@ -916,8 +916,7 @@ PlacesTreeView.prototype = {
// For consistency, we always return a favicon for non-containers,
// even if it is just the default one.
var icon = node.icon || PlacesUtils.favicons.defaultFavicon;
if (icon)
return icon.spec;
return icon ? icon.spec : "";
},
getProgressMode: function(aRow, aColumn) { },