Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(580)

Issue 11821007: Merge 139017. Requested by yusufo@google.com (Closed)

Created:
7 years, 11 months ago by leviw_travelin_and_unemployed
Modified:
7 years, 11 months ago
Reviewers:
commit-queue
CC:
chromium-reviews
Base URL:
http://svn.webkit.org/repository/webkit/branches/chromium/1364/
Visibility:
Public.

Description

Merge 139017. Requested by yusufo@google.com > Don't use double tap zoom if minimum and maximum page scale is the same > https://bugs.webkit.org/show_bug.cgi?id=106272 > > Patch by Yusuf Ozuysal <yusufo@google.com>; on 2013-01-07 > Reviewed by James Robinson. > > We should only try to computed relevant scale and scrollOffset and start the animation > if the page is zoomable. This now creates an undesired animation for mobile pages which > are not zoomable. > > * src/WebViewImpl.cpp: > (WebKit::WebViewImpl::handleGestureEvent): TBR=commit-queue@webkit.org Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=139115

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+1 line, -1 line) Patch
M Source/WebKit/chromium/src/WebViewImpl.cpp View 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 1 (0 generated)
leviw_travelin_and_unemployed
7 years, 11 months ago (2013-01-08 22:45:54 UTC) #1

          

Powered by Google App Engine
This is Rietveld 408576698