New issue
Advanced search Search tips

Issue 668318 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Nov 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: iOS
Pri: 1
Type: Bug



Sign in to add a comment

Going huge deltas from JavaScript freezes the app for a couple of seconds

Project Member Reported by eugene...@chromium.org, Nov 24 2016

Issue description

This script locks up the UI:
window.history.go(1000000000);
 
Labels: Merge-Request-56
Status: Fixed (was: Started)

Comment 3 by dimu@chromium.org, Nov 30 2016

Labels: -Merge-Request-56 Merge-Approved-56 Hotlist-Merge-Approved
Your change meets the bar and is auto-approved for M56 (branch: 2924)
Project Member

Comment 4 by bugdroid1@chromium.org, Dec 2 2016

Labels: -merge-approved-56 merge-merged-2924
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/d03344d142ac9fb00f061e5ee823fb6b57794f4e

commit d03344d142ac9fb00f061e5ee823fb6b57794f4e
Author: eugenebut <eugenebut@chromium.org>
Date: Fri Dec 02 22:35:44 2016

[ios] Improved GetIndexForOffset performance to large offsets.

The difference from trunk CL is allowing integer overflows, which is not
a big issue in practice.

BUG= 668318 
Trunk CL: https://codereview.chromium.org/2526093002/
NOTRY=true
NOPRESUBMIT=true

Review-Url: https://codereview.chromium.org/2540373003
Cr-Commit-Position: refs/branch-heads/2924@{#306}
Cr-Branched-From: 3a87aecc31cd1ffe751dd72c04e5a96a1fc8108a-refs/heads/master@{#433059}

[modify] https://crrev.com/d03344d142ac9fb00f061e5ee823fb6b57794f4e/ios/web/navigation/crw_session_controller.mm

Sign in to add a comment