Magic-Mirror/gecko-dev
1
0
Fork 0
mirror of https://github.com/mozilla/gecko-dev.git synced 2024-10-11 12:25:53 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

follow-up patch for bug 117026 "bookmark properties dialog very slow to load" patch=alfredkayser@nl.ibm.com, r=p_ch@verizon.net, sr=bzbarsky@mit.edu

Browse Source
This commit is contained in:
darin%meer.net 2003-10-24 18:24:13 +00:00
parent 83d3ceaf43
commit bed23ed055
2 changed files with 15 additions and 34 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

40
xpfe/components/bookmarks/resources/bm-props.js
View File

@ -42,9 +42,6 @@ var gFields;
// that they are associated with.
var gProperties;
// All the bookmarks datasource
var gBookmarks;
// The ID of the current shown bookmark
var gBookmarkID;
@ -63,7 +60,6 @@ function Init()
NC_NS + "ShortcutURL",
NC_NS + "Description"];
gBookmarks = RDF.GetDataSource("rdf:bookmarks");
gBookmarkID = window.arguments[0];
var i;
@ -74,7 +70,7 @@ function Init()
for (i = 0; i < gFields.length; ++i) {
var field = document.getElementById(gFields[i]);
var value = gBookmarks.GetTarget(resource, RDF.GetResource(gProperties[i]), true);
var value = BMDS.GetTarget(resource, RDF.GetResource(gProperties[i]), true);
if (value)
value = value.QueryInterface(Components.interfaces.nsIRDFLiteral).Value;
@ -90,7 +86,7 @@ function Init()
propsWindow.setAttribute("title", title);
// if its a container, disable some things
var isContainerFlag = RDFCU.IsContainer(gBookmarks, resource);
var isContainerFlag = RDFCU.IsContainer(BMDS, resource);
if (!isContainerFlag) {
// XXX To do: the "RDFCU.IsContainer" call above only works for RDF sequences;
// if its not a RDF sequence, we should to more checking to see if
@ -110,7 +106,7 @@ function Init()
}
var showScheduling = false;
var url = gBookmarks.GetTarget(resource, RDF.GetResource(gProperties[1]), true);
var url = BMDS.GetTarget(resource, RDF.GetResource(gProperties[1]), true);
if (url) {
url = url.QueryInterface(Components.interfaces.nsIRDFLiteral).Value;
if (url.substr(0, 7).toLowerCase() == "http://" ||
@ -125,7 +121,7 @@ function Init()
} else {
// check bookmark schedule
var scheduleArc = RDF.GetResource("http://home.netscape.com/WEB-rdf#Schedule");
value = gBookmarks.GetTarget(resource, scheduleArc, true);
value = BMDS.GetTarget(resource, scheduleArc, true);
if (value) {
value = value.QueryInterface(Components.interfaces.nsIRDFLiteral).Value;
@ -222,9 +218,8 @@ function Commit()
// Get the new value as a literal, using 'null' if the value is empty.
var newvalue = field.value;
var oldvalue = gBookmarks.GetTarget(RDF.GetResource(gBookmarkID),
RDF.GetResource(gProperties[i]),
true);
var oldvalue = BMDS.GetTarget(RDF.GetResource(gBookmarkID),
RDF.GetResource(gProperties[i]), true);
if (oldvalue)
oldvalue = oldvalue.QueryInterface(Components.interfaces.nsIRDFLiteral);
@ -255,8 +250,8 @@ function Commit()
var schedulingHidden = scheduling.getAttribute("hidden");
if (schedulingHidden != "true") {
var scheduleRes = "http://home.netscape.com/WEB-rdf#Schedule";
oldvalue = gBookmarks.GetTarget(RDF.GetResource(gBookmarkID),
RDF.GetResource(scheduleRes), true);
oldvalue = BMDS.GetTarget(RDF.GetResource(gBookmarkID),
RDF.GetResource(scheduleRes), true);
newvalue = "";
var dayRangeNode = document.getElementById("dayRange");
var dayRange = dayRangeNode.selectedItem.getAttribute("value");
@ -308,7 +303,7 @@ function Commit()
}
if (changed) {
var remote = gBookmarks.QueryInterface(Components.interfaces.nsIRDFRemoteDataSource);
var remote = BMDS.QueryInterface(Components.interfaces.nsIRDFRemoteDataSource);
if (remote)
remote.Flush();
}
@ -324,21 +319,16 @@ function updateAttribute(prop, oldvalue, newvalue)
if (prop && (oldvalue || newvalue) && oldvalue != newvalue) {
if (oldvalue && !newvalue) {
gBookmarks.Unassert(RDF.GetResource(gBookmarkID),
RDF.GetResource(prop),
oldvalue);
BMDS.Unassert(RDF.GetResource(gBookmarkID),
RDF.GetResource(prop), oldvalue);
}
else if (!oldvalue && newvalue) {
gBookmarks.Assert(RDF.GetResource(gBookmarkID),
RDF.GetResource(prop),
newvalue,
true);
BMDS.Assert(RDF.GetResource(gBookmarkID),
RDF.GetResource(prop), newvalue, true);
}
else /* if (oldvalue && newvalue) */ {
gBookmarks.Change(RDF.GetResource(gBookmarkID),
RDF.GetResource(prop),
oldvalue,
newvalue);
BMDS.Change(RDF.GetResource(gBookmarkID),
RDF.GetResource(prop), oldvalue, newvalue);
}
changed = true;

9
xpfe/components/bookmarks/resources/locale/en-US/bookmarks.properties
View File

@ -18,15 +18,6 @@
# Contributor(s):
#
description_Bookmark = %brandShortName% can remember web page locations for you. Type the page name and location in the fields below, then click OK. Select the page from the Bookmarks menu or your Bookmarks Sidebar tab to visit the page.
description_Folder = %brandShortName% can organize your bookmarks into specific folders. Type the folder name and an optional comment below, then click OK.
description_FolderGroup = %brandShortName% can bookmark a group of web pages. Type a group name and an optional comment in the field below, then click OK. Select the group from the Bookmark menu or your Bookmarks Sidebar tab to open each page in a separate Navigator tab.
description_PersonalToolbarFolder = Folders and bookmarks in this folder appear on the Personal Toolbar.
description_NewBookmarkFolder = Newly created bookmarks will be stored here by default.
description_NewSearchFolder = Search results will be stored here by default.
description_NewBookmarkAndSearchFolder = Newly created bookmarks and search results will be stored here by default.
description_BookmarkSeparator = %brandShortName% can organize your bookmarks by putting separators between them. Type an optional separator name below, then click OK.
cmd_bm_open = Open
cmd_bm_expandfolder = Expand
cmd_bm_collapsefolder = Collapse