Cleanup in FullscreenController.

ios/chrome/browser/ui/fullscreen_controller.mm

 // Copyright 2013 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.
 
 #import "ios/chrome/browser/ui/fullscreen_controller.h"
 
 #include <cmath>
 
 #include "base/logging.h"
 #include "base/mac/objc_property_releaser.h"
@@ skipping 126 matching lines @@
 // events triggered by the keyboard appearing to go through.
 - (void)keyboardStart:(NSNotification*)notification;
 - (void)keyboardEnd:(NSNotification*)notification;
 // Called before and after an action that disables full screen. The version
 // resetting the timer will ensure that the header stay on screen for a little
 // while.
 - (void)incrementFullScreenLock;
 - (void)decrementFullScreenLock;
 // Called when the application is about to be the foreground application.
 - (void)applicationWillEnterForeground:(NSNotification*)notification;
-// TODO(shreyasv): Make the following methods act on a WebViewScrollView proxy
-// instead of taking in a UIScrollView directly.
 // Called from -webViewScrollViewDidScroll: Returns YES if the scroll should be
 // ignored.
 - (BOOL)shouldIgnoreScroll:(CRWWebViewScrollViewProxy*)webViewScrollViewProxy;
 // Processes a scroll event triggered by a user action.
 - (void)userTriggeredWebViewScrollViewDidScroll:
     (CRWWebViewScrollViewProxy*)webViewScrollViewProxy;
 // Processes a scroll event triggered by code (these could be initiated via
 // Javascript, find in page or simply the keyboard sliding in and out).
 - (void)codeTriggeredWebViewScrollViewDidScroll:
     (CRWWebViewScrollViewProxy*)webViewScrollViewProxy;
@@ skipping 128 matching lines @@
                    name:kTabStripDragEnded
                  object:nil];
     [center addObserver:self
                selector:@selector(incrementFullScreenLock)
                    name:ios_internal::kSideSwipeWillStartNotification
                  object:nil];
     [center addObserver:self
                selector:@selector(decrementFullScreenLock)
                    name:ios_internal::kSideSwipeDidStopNotification
                  object:nil];
-    // TODO(jbbegue): Evaluate using a listener instead of a notification
-    // crbug/451373.
+    // TODO(crbug.com/451373): Evaluate using listener instead of notifications.

[rohitrao (ping after 24h), 2017/01/10 17:24:02]

"Listeners"?

     [center addObserver:self
                selector:@selector(overscrollActionsWillStart)
                    name:kOverscrollActionsWillStart
                  object:nil];
     [center addObserver:self
                selector:@selector(overscrollActionsDidEnd)
                    name:kOverscrollActionsDidEnd
                  object:nil];
     [self moveHeaderToRestingPosition:YES];
   }
@@ skipping 510 matching lines @@
   overscrollActionsInProgress_ = NO;
 }
 
 #pragma mark - Used for testing
 
 + (void)setEnabledForTests:(BOOL)enabled {
   gEnabledForTests = enabled;
 }
 
 @end