Add DCHECK that Reload should never be called with ReloadType::NONE

content/public/browser/navigation_controller.h

 // Copyright (c) 2012 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.
 
 #ifndef CONTENT_PUBLIC_BROWSER_NAVIGATION_CONTROLLER_H_
 #define CONTENT_PUBLIC_BROWSER_NAVIGATION_CONTROLLER_H_
 
 #include <stdint.h>
 
 #include <map>
@@ skipping 338 matching lines @@
   // Navigates to the specified offset from the "current entry". Does nothing if
   // the offset is out of bounds.
   virtual void GoToOffset(int offset) = 0;
 
   // Reloads the current entry under the specified ReloadType.  If
   // |check_for_repost| is true and the current entry has POST data the user is
   // prompted to see if they really want to reload the page.  In nearly all
   // cases pass in true in production code, but would do false for testing, or
   // in cases where no user interface is available for prompting.  If a
   // transient entry is showing, initiates a new navigation to its URL.
+  // NOTE: |reload_type| should never be NONE.
   virtual void Reload(ReloadType reload_type, bool check_for_repost) = 0;
 
   // Removing of entries -------------------------------------------------------
 
   // Removes the entry at the specified |index|.  If the index is the last
   // committed index or the pending entry, this does nothing and returns false.
   // Otherwise this call discards any transient or pending entries.
   virtual bool RemoveEntryAtIndex(int index) = 0;
 
   // Random --------------------------------------------------------------------
@@ skipping 85 matching lines @@
 
  private:
   // This interface should only be implemented inside content.
   friend class NavigationControllerImpl;
   NavigationController() {}
 };
 
 }  // namespace content
 
 #endif  // CONTENT_PUBLIC_BROWSER_NAVIGATION_CONTROLLER_H_