Classification of TODOs in Panel code.

chrome/browser/ui/panels/panel.cc

Index: chrome/browser/ui/panels/panel.cc
diff --git a/chrome/browser/ui/panels/panel.cc b/chrome/browser/ui/panels/panel.cc
index 9f4f1a2baba07f17a8626cbb758c239e77d9245c..99c4ae2ddd5b22a162ec2b9adbfce0f10ca1b980 100644
--- a/chrome/browser/ui/panels/panel.cc
+++ b/chrome/browser/ui/panels/panel.cc
@@ -172,7 +172,7 @@ void Panel::Close() {
 // TODO(dimich): Only implemented fully async on Mac. Need to update other
 // platforms. The panel should be removed from PanelManager when and if it
 // was actually closed. The closing can be cancelled because of onbeforeunload
-// handler on the web page.
+// handler on the web page. http://crbug.com/102720
 #if !defined(OS_MACOSX)
   manager()->Remove(this);
 #endif
@@ -207,7 +207,7 @@ BrowserWindowTesting* Panel::GetBrowserWindowTesting() {
 }
 
 StatusBubble* Panel::GetStatusBubble() {
-  // TODO(jennb): Implement.
+  // TODO(jennb): Implement. http://crbug.com/102723
   return NULL;
 }
 
@@ -233,7 +233,8 @@ void Panel::UpdateLoadingAnimations(bool should_animate) {
 }
 
 void Panel::SetStarredState(bool is_starred) {
-  // TODO(jennb): Figure out if this applies to Panels.
+  // Since panels are typically not bookmarked extension UI windows, they don't
+  // have starred state.

[Dmitry Titov, 2011/11/02 22:15:01]

I took a liberty to 'resolve' it based on current understanding of Panels. Feel
free to object.

 }
 
 gfx::Rect Panel::GetRestoredBounds() const {
@@ -248,7 +249,7 @@ gfx::Rect Panel::GetBounds() const {
 }
 
 bool Panel::IsMaximized() const {
-  // TODO(jennb): Figure out how this applies to Panels. Means isZoomed.
+  // Size of panels is managed by PanelManager, they are never 'zoomed'.

[Dmitry Titov, 2011/11/02 22:15:01]

I took a liberty to 'resolve' it based on current understanding of Panels. Feel
free to object.

   return false;
 }
 
@@ -290,7 +291,7 @@ void Panel::SetFocusToLocationBar(bool select_all) {
 }
 
 void Panel::UpdateReloadStopState(bool is_loading, bool force) {
-  // TODO(jennb): Implement.
+  // Panels don't have stop/reload indicator.

[Dmitry Titov, 2011/11/02 22:15:01]

I took a liberty to 'resolve' it based on current understanding of Panels. Feel
free to object.

 }
 
 void Panel::UpdateToolbar(TabContentsWrapper* contents,