capture_mode: Associate some TODOs with bug number.

Bug: none
Test: none
Change-Id: I28a0739e54c36ee10f81fd5b59cd1cbdba0019d4
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2730848
Commit-Queue: Sammie Quon <sammiequon@chromium.org>
Auto-Submit: Sammie Quon <sammiequon@chromium.org>
Reviewed-by: Ahmed Fakhry <afakhry@chromium.org>
Cr-Commit-Position: refs/heads/master@{#859862}

ash/capture_mode/capture_mode_session_focus_cycler.h

@@ -27,16 +27,16 @@ class CaptureModeSession;
 // CaptureModeSessionFocusCycler handles the special focus transitions which
 // happen between the capture session UI items. These include the capture bar
 // buttons, the selection region UI and the capture button.
-// TODO(sammiequon): The selection region UI are drawn directly on a layer. We
-// simulate focus by drawing focus rings on the same layer, but this is not
-// compatible with accessibility. Investigate using AxVirtualView or making the
-// dots actual Views.
+// TODO(crbug.com/1182456): The selection region UI are drawn directly on a
+// layer. We simulate focus by drawing focus rings on the same layer, but this
+// is not compatible with accessibility. Investigate using AxVirtualView or
+// making the dots actual Views.
 class ASH_EXPORT CaptureModeSessionFocusCycler : public views::WidgetObserver {
  public:
   // The different groups which can receive focus during a capture mode session.
   // A group may have multiple items which can receive focus.
-  // TODO(sammiequon): Investigate removing the groups concept and having one
-  // flat list.
+  // TODO(crbug.com/1182456): Investigate removing the groups concept and having
+  // one flat list.
   enum class FocusGroup {
     kNone = 0,
     // The buttons to select the capture type and source on the capture bar.
@@ -83,8 +83,8 @@ class ASH_EXPORT CaptureModeSessionFocusCycler : public views::WidgetObserver {
     void ClickView();
 
    private:
-    // TODO(sammiequon): This can result in multiple of these objects thinking
-    // they have focus if CaptureModeSessionFocusCycler does not call
+    // TODO(crbug.com/1182456): This can result in multiple of these objects
+    // thinking they have focus if CaptureModeSessionFocusCycler does not call
     // PseudoFocus or PseudoBlur properly. Investigate if there is a better
     // approach.
     bool has_focus_ = false;

ash/capture_mode/capture_mode_unittests.cc

@@ -129,6 +129,8 @@ void ClickNotification(base::Optional<int> button_index) {
 
 // Moves the mouse and updates the cursor's display manually to imitate what a
 // real mouse move event does in shell.
+// TODO(crbug.com/990589): Unit tests should be able to simulate mouse input
+// without having to call |CursorManager::SetDisplay|.
 void MoveMouseToAndUpdateCursorDisplay(
     const gfx::Point& point,
     ui::test::EventGenerator* event_generator) {

ash/wm/toplevel_window_event_handler_unittest.cc

@@ -1097,6 +1097,8 @@ TEST_F(ToplevelWindowEventHandlerTest, DragSnappedWindowToExternalDisplay) {
   // To determine the state, WindowWorkspaceResizer determines the display by
   // checking the CursorManager for the correct display. EventGenerator does not
   // update the display in the CursorManager, so we manually do it here.
+  // TODO(crbug.com/990589): Unit tests should be able to simulate mouse input
+  // without having to call |CursorManager::SetDisplay|.
   const gfx::Point drag_location = gfx::Point(472, -462);
   Shell::Get()->cursor_manager()->SetDisplay(
       display::Screen::GetScreen()->GetDisplayNearestPoint(drag_location));