New issue
Advanced search Search tips

Issue 804552 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Mar 2018
Cc:
Components:
EstimatedDays: ----
NextAction: 2018-03-28
OS: iOS
Pri: 2
Type: Task
Q1

Blocked on:
issue 804500
issue 804551



Sign in to add a comment

Ensure the count in the segmented control updates.

Project Member Reported by marq@chromium.org, Jan 22 2018

Issue description

Ensure the count in the segmented control updates.

 

Comment 1 by marq@chromium.org, Jan 23 2018

Components: UI>Browser
Labels: MS-Tab-Grid Pri-2
Owner: marq@chromium.org
Status: Available (was: Unconfirmed)

Comment 2 by marq@chromium.org, Jan 23 2018

Labels: S-Close-One-Tab

Comment 3 by marq@chromium.org, Jan 23 2018

Labels: small

Comment 4 by marq@chromium.org, Jan 23 2018

Blockedon: 804551

Comment 5 by marq@chromium.org, Jan 23 2018

Blockedon: 804500

Comment 6 by marq@chromium.org, Jan 23 2018

Labels: Q1

Comment 7 by marq@chromium.org, Jan 25 2018

Status: Assigned (was: Available)

Comment 8 by marq@chromium.org, Mar 15 2018

NextAction: 2018-03-28

Comment 9 by marq@chromium.org, Mar 15 2018

Status: Started (was: Assigned)
Project Member

Comment 10 by bugdroid1@chromium.org, Mar 16 2018

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

commit 50905e908c1d8b750e730861696d59872d4c99eb
Author: Mark Cogan <marq@google.com>
Date: Fri Mar 16 16:02:56 2018

[iOS] Update tab counts in tab grid page control.

This CL wires the tab grid page controls tab counts into the grid
view controllers. To do this, a public |itemCount| property is added to
the GridViewControllers, and the TabGridViewController updates the
counts in the page control whenever an event occurs that would change
the count (opening or closing a tab), and when the VC is initially set
up.

Bug:  804501 ,  804552 
Cq-Include-Trybots: master.tryserver.chromium.mac:ios-simulator-cronet;master.tryserver.chromium.mac:ios-simulator-full-configs
Change-Id: I9162b45e04b102ce4c1b885356ebf3bf0873396b
Reviewed-on: https://chromium-review.googlesource.com/964143
Reviewed-by: edchin <edchin@chromium.org>
Commit-Queue: Mark Cogan <marq@chromium.org>
Cr-Commit-Position: refs/heads/master@{#543716}
[modify] https://crrev.com/50905e908c1d8b750e730861696d59872d4c99eb/ios/chrome/browser/ui/tab_grid/grid_view_controller.h
[modify] https://crrev.com/50905e908c1d8b750e730861696d59872d4c99eb/ios/chrome/browser/ui/tab_grid/grid_view_controller.mm
[modify] https://crrev.com/50905e908c1d8b750e730861696d59872d4c99eb/ios/chrome/browser/ui/tab_grid/tab_grid_page_control.h
[modify] https://crrev.com/50905e908c1d8b750e730861696d59872d4c99eb/ios/chrome/browser/ui/tab_grid/tab_grid_page_control.mm
[modify] https://crrev.com/50905e908c1d8b750e730861696d59872d4c99eb/ios/chrome/browser/ui/tab_grid/tab_grid_view_controller.mm

Project Member

Comment 11 by bugdroid1@chromium.org, Mar 16 2018

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

commit d89dd9673b7ffaacdc1b346a1a424350d9eee3fa
Author: Mark Cogan <marq@google.com>
Date: Fri Mar 16 17:36:32 2018

[iOS] Update tab counts in grid via delegate

Bug:  804552 
Cq-Include-Trybots: master.tryserver.chromium.mac:ios-simulator-cronet;master.tryserver.chromium.mac:ios-simulator-full-configs
Change-Id: I183368403e1977c28dd0696532feff849fcbf0df
Reviewed-on: https://chromium-review.googlesource.com/966625
Commit-Queue: edchin <edchin@chromium.org>
Reviewed-by: edchin <edchin@chromium.org>
Cr-Commit-Position: refs/heads/master@{#543751}
[modify] https://crrev.com/d89dd9673b7ffaacdc1b346a1a424350d9eee3fa/ios/chrome/browser/ui/tab_grid/grid_view_controller.h
[modify] https://crrev.com/d89dd9673b7ffaacdc1b346a1a424350d9eee3fa/ios/chrome/browser/ui/tab_grid/grid_view_controller.mm
[modify] https://crrev.com/d89dd9673b7ffaacdc1b346a1a424350d9eee3fa/ios/chrome/browser/ui/tab_grid/tab_grid_view_controller.mm

Comment 12 by marq@chromium.org, Mar 19 2018

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

Sign in to add a comment