Add bug #s and remove NOTIMPLEMENTED()s for things we don't need to implement in RWHVA

content/browser/renderer_host/render_widget_host_view_aura.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 "content/browser/renderer_host/render_widget_host_view_aura.h"
 
 #include "base/logging.h"
 #include "content/browser/renderer_host/backing_store_skia.h"
 #include "content/browser/renderer_host/render_widget_host.h"
 #include "content/browser/renderer_host/web_input_event_aura.h"
@@ skipping 195 matching lines @@
 }
 
 void RenderWidgetHostViewAura::SetIsLoading(bool is_loading) {
   is_loading_ = is_loading;
   UpdateCursorIfOverSelf();
 }
 
 void RenderWidgetHostViewAura::TextInputStateChanged(
     ui::TextInputType type,
     bool can_compose_inline) {
+  // http://crbug.com/102569
   NOTIMPLEMENTED();
 }
 
 void RenderWidgetHostViewAura::ImeCancelComposition() {
+  // http://crbug.com/102569
   NOTIMPLEMENTED();
 }
 
 void RenderWidgetHostViewAura::DidUpdateBackingStore(
     const gfx::Rect& scroll_rect, int scroll_dx, int scroll_dy,
     const std::vector<gfx::Rect>& copy_rects) {
   if (!window_->IsVisible())
     return;
 
   if (!scroll_rect.IsEmpty())
@@ skipping 95 matching lines @@
   results->depthPerComponent = 8;
 }
 
 gfx::Rect RenderWidgetHostViewAura::GetRootWindowBounds() {
   return aura::Desktop::GetInstance()->bounds();
 }
 #endif
 
 void RenderWidgetHostViewAura::SetVisuallyDeemphasized(const SkColor* color,
                                                        bool animate) {
+  // http://crbug.com/102568
   NOTIMPLEMENTED();
 }
 
 void RenderWidgetHostViewAura::UnhandledWheelEvent(
     const WebKit::WebMouseWheelEvent& event) {
-  NOTIMPLEMENTED();
+  // Not needed. Mac-only.
 }
 
 void RenderWidgetHostViewAura::SetHasHorizontalScrollbar(
     bool has_horizontal_scrollbar) {
-  NOTIMPLEMENTED();
+  // Not needed. Mac-only.
 }
 
 void RenderWidgetHostViewAura::SetScrollOffsetPinning(
     bool is_pinned_to_left, bool is_pinned_to_right) {
-  NOTIMPLEMENTED();
+  // Not needed. Mac-only.
 }
 
 #if defined(OS_WIN)
 void RenderWidgetHostViewAura::WillWmDestroy() {
   // Nothing to do.
 }
 
 void RenderWidgetHostViewAura::ShowCompositorHostWindow(bool show) {
   // Nothing to do.
 }
@@ skipping 11 matching lines @@
   // TODO(oshima): The original implementation was broken as
   // GtkNativeViewManager doesn't know about NativeWidgetGtk. Figure
   // out if this makes sense without compositor. If it does, then find
   // out the right way to handle.
   NOTIMPLEMENTED();
   return gfx::kNullPluginWindow;
 }
 #endif
 
 bool RenderWidgetHostViewAura::LockMouse() {
+  // http://crbug.com/102563
   NOTIMPLEMENTED();
   return false;
 }
 
 void RenderWidgetHostViewAura::UnlockMouse() {
+  // http://crbug.com/102563
   NOTIMPLEMENTED();
   host_->LostMouseLock();
 }
 
 ////////////////////////////////////////////////////////////////////////////////
 // RenderWidgetHostViewAura, aura::WindowDelegate implementation:
 
 void RenderWidgetHostViewAura::OnBoundsChanged(const gfx::Rect& old_bounds,
                                                const gfx::Rect& new_bounds) {
   // We don't care about this one, we are always sized via SetSize() or
@@ skipping 13 matching lines @@
   if (is_fullscreen_ && event->key_code() == ui::VKEY_ESCAPE) {
     host_->Shutdown();
   } else {
     NativeWebKeyboardEvent webkit_event(event);
     host_->ForwardKeyboardEvent(webkit_event);
   }
   return true;
 }
 
 gfx::NativeCursor RenderWidgetHostViewAura::GetCursor(const gfx::Point& point) {
-  // TODO(beng): talk to beng before implementing this.
-  //NOTIMPLEMENTED();
+  // http://crbug.com/102562
+  // NOTIMPLEMENTED();
   return gfx::kNullCursor;
 }
 
 int RenderWidgetHostViewAura::GetNonClientComponent(
     const gfx::Point& point) const {
   return HTCLIENT;
 }
 
 bool RenderWidgetHostViewAura::OnMouseEvent(aura::MouseEvent* event) {
   if (event->type() == ui::ET_MOUSEWHEEL)
@@ skipping 70 matching lines @@
   }
   on_compositing_ended_callbacks_.clear();
   compositor->RemoveObserver(this);
 }
 #endif
 
 ////////////////////////////////////////////////////////////////////////////////
 // RenderWidgetHostViewAura, private:
 
 void RenderWidgetHostViewAura::UpdateCursorIfOverSelf() {
-  //NOTIMPLEMENTED();
+  // http://crbug.com/102562
+  // NOTIMPLEMENTED();
   // TODO(beng): See RenderWidgetHostViewWin.
 }