Disable overscroll on Windows machines w/o touchscreen

content/browser/frame_host/navigator_impl.cc

Index: content/browser/frame_host/navigator_impl.cc
diff --git a/content/browser/frame_host/navigator_impl.cc b/content/browser/frame_host/navigator_impl.cc
index 384bceeafbe37a8ccaab4546a517d86c8df019d1..ce11890db3fccc967b889fc5d179fd3618a0d1bf 100644
--- a/content/browser/frame_host/navigator_impl.cc
+++ b/content/browser/frame_host/navigator_impl.cc
@@ -459,7 +459,7 @@ void NavigatorImpl::DidNavigate(
       // calling RenderFrameHostManager::DidNavigateMainFrame, because that can
       // change WebContents::GetRenderViewHost to return the new host, instead
       // of the one that may have just been swapped out.
-      if (delegate_->CanOverscrollContent()) {
+      if (controller_->ShouldTakeScreenshotOnNavigation()) {

[sadrul, 2014/09/29 20:39:24]

Can we change the Browser::CanOverscrollContent() override instead?

[mfomitchev, 2014/09/29 20:41:39]

This was my first intention, but then I thought it might be better to leave the
overscroll gesture enabled and just disable the screenshot taking - please see
my first comment on the review.

         // Don't take screenshots if we are staying on the same page. We want
         // in-page navigations to be super fast, and taking a screenshot
         // currently blocks GPU for a longer time than we are willing to