New issue
Advanced search Search tips

Issue 825898 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Mar 2018
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: iOS
Pri: 2
Type: Bug
Q2



Sign in to add a comment

Tapping on page control doesn't update new tab button

Project Member Reported by edchin@chromium.org, Mar 26 2018

Issue description

Tapping on page control doesn't update new tab button. 

Also, tapping on page control, then swiping on scrollView
messes up some internal state such that tapping on page 
control is disabled. 

Thanks to michaeldo@ for discovering.
 

Comment 1 by edchin@chromium.org, Mar 26 2018

Description: Show this description
Project Member

Comment 2 by bugdroid1@chromium.org, Mar 27 2018

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/b213eb0d4ad27dc3972762d8d07c881766b66638

commit b213eb0d4ad27dc3972762d8d07c881766b66638
Author: edchin <edchin@chromium.org>
Date: Tue Mar 27 03:20:07 2018

[ios] Synchronize tab grid page control with scroll view

Previously there was a bug where there was a mismatch of internal states
between the page control and the scroll view.
It is important that they both are synchronized with the same current
page.
This CL fixes the lack of synchronization.

TBR=marq@chromium.org
Bug:  825898 

Cq-Include-Trybots: master.tryserver.chromium.mac:ios-simulator-cronet;master.tryserver.chromium.mac:ios-simulator-full-configs
Change-Id: I284460d9581a05f780057ef53715484254978859
Reviewed-on: https://chromium-review.googlesource.com/981302
Commit-Queue: edchin <edchin@chromium.org>
Reviewed-by: edchin <edchin@chromium.org>
Reviewed-by: Sergio Collazos <sczs@chromium.org>
Cr-Commit-Position: refs/heads/master@{#545953}
[modify] https://crrev.com/b213eb0d4ad27dc3972762d8d07c881766b66638/ios/chrome/browser/ui/tab_grid/tab_grid_page_control.mm
[modify] https://crrev.com/b213eb0d4ad27dc3972762d8d07c881766b66638/ios/chrome/browser/ui/tab_grid/tab_grid_view_controller.mm

Comment 3 by edchin@chromium.org, Mar 28 2018

NextAction: 2018-03-28
Status: Fixed (was: Assigned)
The NextAction date has arrived: 2018-03-28

Comment 5 by cmasso@google.com, Mar 28 2018

NextAction: ----
Labels: Proj-UIRefresh

Sign in to add a comment