Classification of TODOs in Panel code.

chrome/browser/ui/panels/panel.cc

 // Copyright (c) 2011 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/ui/panels/panel.h"
 
 #include "base/logging.h"
 #include "chrome/browser/extensions/extension_prefs.h"
 #include "chrome/browser/extensions/extension_service.h"
 #include "chrome/browser/profiles/profile.h"
@@ skipping 154 matching lines @@
 }
 
 // Close() may be called multiple times if the browser window is not ready to
 // close on the first attempt.
 void Panel::Close() {
   native_panel_->ClosePanel();
 
 // 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
 }
 
 void Panel::Activate() {
   // Make sure the panel is expanded when activated programmatically,
   // so the user input does not go into collapsed window.
   SetExpansionState(Panel::EXPANDED);
   native_panel_->ActivatePanel();
@@ skipping 14 matching lines @@
 gfx::NativeWindow Panel::GetNativeHandle() {
   return native_panel_->GetNativePanelHandle();
 }
 
 BrowserWindowTesting* Panel::GetBrowserWindowTesting() {
   NOTIMPLEMENTED();
   return NULL;
 }
 
 StatusBubble* Panel::GetStatusBubble() {
-  // TODO(jennb): Implement.
+  // TODO(jennb): Implement. http://crbug.com/102723
   return NULL;
 }
 
 void Panel::ToolbarSizeChanged(bool is_animating){
   NOTIMPLEMENTED();
 }
 
 void Panel::UpdateTitleBar() {
   native_panel_->UpdatePanelTitleBar();
 }
 
 void Panel::BookmarkBarStateChanged(
     BookmarkBar::AnimateChangeType change_type) {
   NOTIMPLEMENTED();
 }
 
 void Panel::UpdateDevTools() {
   NOTIMPLEMENTED();
 }
 
 void Panel::UpdateLoadingAnimations(bool should_animate) {
   native_panel_->UpdatePanelLoadingAnimations(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.
 }
 
 gfx::Rect Panel::GetRestoredBounds() const {
   gfx::Rect bounds = native_panel_->GetPanelBounds();
   bounds.set_y(bounds.y() + bounds.height() - restored_height_);
   bounds.set_height(restored_height_);
   return bounds;
 }
 
 gfx::Rect Panel::GetBounds() const {
   return native_panel_->GetPanelBounds();
 }
 
 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'.
   return false;
 }
 
 bool Panel::IsMinimized() const {
   return expansion_state_ != EXPANDED;
 }
 
 void Panel::EnterFullscreen(
       const GURL& url, FullscreenExitBubbleType type) {
   NOTIMPLEMENTED();
@@ skipping 21 matching lines @@
 LocationBar* Panel::GetLocationBar() const {
   // Panels do not have a location bar.
   return NULL;
 }
 
 void Panel::SetFocusToLocationBar(bool select_all) {
   // Panels do not have a location bar.
 }
 
 void Panel::UpdateReloadStopState(bool is_loading, bool force) {
-  // TODO(jennb): Implement.
+  // Panels don't have stop/reload indicator.
 }
 
 void Panel::UpdateToolbar(TabContentsWrapper* contents,
                           bool should_restore_state) {
   // Panels do not have a toolbar.
 }
 
 void Panel::FocusToolbar() {
   // Panels do not have a toolbar.
 }
@@ skipping 286 matching lines @@
   RequestRenderViewHostToDisableScrollbars(GetRenderViewHost());
 }
 
 Browser* Panel::browser() const {
   return native_panel_->GetPanelBrowser();
 }
 
 void Panel::DestroyBrowser() {
   native_panel_->DestroyPanelBrowser();
 }