From 45f54ffb1f0c2738dbce6d644e505129d2290770 Mon Sep 17 00:00:00 2001
From: Brett Slatkin <bslatkin@gmail.com>
Date: Sun, 20 Jan 2013 11:13:20 -0800
Subject: [PATCH] newui: Cleaning up TODOs in comments

---
 server/camlistored/newui/blob_item.js           | 6 ++++--
 server/camlistored/newui/blob_item_container.js | 3 ++-
 server/camlistored/newui/create_item.js         | 3 ++-
 server/camlistored/newui/index.js               | 2 +-
 server/camlistored/newui/server_connection.js   | 6 ++++--
 server/camlistored/newui/toolbar.js             | 2 +-
 6 files changed, 14 insertions(+), 8 deletions(-)

diff --git a/server/camlistored/newui/blob_item.js b/server/camlistored/newui/blob_item.js
index 4a2bd4bd3..b7c6c7e22 100644
--- a/server/camlistored/newui/blob_item.js
+++ b/server/camlistored/newui/blob_item.js
@@ -1,5 +1,7 @@
 /**
- * @fileoverview TODO
+ * @fileoverview An item showing in a blob item container; represents a blob
+ * that has already been uploaded in the system, or acts as a placeholder
+ * for a new blob.
  *
  */
 goog.provide('camlistore.BlobItem');
@@ -64,7 +66,7 @@ goog.inherits(camlistore.BlobItem, goog.ui.Control);
 
 
 /**
- * TODO: Handle more permanode types.
+ * TODO(bslatkin): Handle more permanode types.
  *
  * @param {string} blobRef string BlobRef to resolve.
  * @param {camlistore.ServerType.IndexerMetaBag} metaBag Metadata bag to use
diff --git a/server/camlistored/newui/blob_item_container.js b/server/camlistored/newui/blob_item_container.js
index d7dbcabaf..ef84f2d84 100644
--- a/server/camlistored/newui/blob_item_container.js
+++ b/server/camlistored/newui/blob_item_container.js
@@ -1,5 +1,6 @@
 /**
- * @fileoverview TODO
+ * @fileoverview Contains a set of BlobItems. Knows how to fetch items from
+ * the server side. Is preconfigured with common queries like "recent" blobs.
  *
  */
 goog.provide('camlistore.BlobItemContainer');
diff --git a/server/camlistored/newui/create_item.js b/server/camlistored/newui/create_item.js
index 42a461f71..23e3db4a0 100644
--- a/server/camlistored/newui/create_item.js
+++ b/server/camlistored/newui/create_item.js
@@ -1,5 +1,6 @@
 /**
- * @fileoverview TODO
+ * @fileoverview Placeholder BlobItem in a BlobItemContainer that lets the
+ * user upload new blobs to the server.
  *
  */
 goog.provide('camlistore.CreateItem');
diff --git a/server/camlistored/newui/index.js b/server/camlistored/newui/index.js
index 300a0c383..d1687efdf 100644
--- a/server/camlistored/newui/index.js
+++ b/server/camlistored/newui/index.js
@@ -1,5 +1,5 @@
 /**
- * @fileoverview TODO
+ * @fileoverview Entry point for the blob browser UI.
  *
  */
 goog.provide('camlistore.IndexPage');
diff --git a/server/camlistored/newui/server_connection.js b/server/camlistored/newui/server_connection.js
index 9e472a582..c54b7b465 100644
--- a/server/camlistored/newui/server_connection.js
+++ b/server/camlistored/newui/server_connection.js
@@ -1,5 +1,7 @@
 /**
- * @fileoverview TODO
+ * @fileoverview Connection to the blob server and API for the RPCs it
+ * provides. All blob index UI code should use this connection to contact
+ * the server.
  *
  */
 goog.provide('camlistore.ServerConnection');
@@ -79,7 +81,7 @@ camlistore.ServerConnection.prototype.getRecentlyUpdatedPermanodesDone_ =
  * @param {Function=} opt_fail Optional fail callback.
  */
 camlistore.ServerConnection.prototype.createPermanode = function(success, opt_fail) {
-  // TODO: stop depending on camli.js.  For now, cheating:
+  // TODO(bradfitz): stop depending on camli.js.  For now, cheating:
   camliCreateNewPermanode({
       success: success,
       fail: opt_fail
diff --git a/server/camlistored/newui/toolbar.js b/server/camlistored/newui/toolbar.js
index a6773547d..a9c95eac7 100644
--- a/server/camlistored/newui/toolbar.js
+++ b/server/camlistored/newui/toolbar.js
@@ -1,5 +1,5 @@
 /**
- * @fileoverview TODO
+ * @fileoverview Toolbar for manipulating the display of the blob index page.
  *
  */
 goog.provide('camlistore.Toolbar');