New issue
Advanced search Search tips

Issue 898379 link

Starred by 2 users
Status: Fixed
Owner:
mrefaat@chromium.org
Closed: Oct 24
Cc:
ios-bugs@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: iOS
Pri: 2
Type: Task
q4



Sign in to add a comment

Remove lastVisitedTimestamp from tab

Project Member Reported by mrefaat@chromium.org, Oct 24 
Remove the property & method & tests.
Project Member

Comment 1 by bugdroid1@chromium.org, Oct 24 

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

commit ac9ac88fef0675464570d0d406e0dfe034d74f48
Author: mrefaat <mrefaat@chromium.org>
Date: Wed Oct 24 20:08:02 2018

[IOSTabRefactoring] Remove lastVisitedTimestamp

Remove with its users.

Bug:  898379 
Change-Id: Ie5a127c17c93b811e230c19b5998d81feee81270
Reviewed-on: https://chromium-review.googlesource.com/c/1298234
Reviewed-by: Eugene But <eugenebut@chromium.org>
Commit-Queue: Mohammad Refaat <mrefaat@chromium.org>
Cr-Commit-Position: refs/heads/master@{#602423}
[modify] https://crrev.com/ac9ac88fef0675464570d0d406e0dfe034d74f48/ios/chrome/browser/tabs/tab.h
[modify] https://crrev.com/ac9ac88fef0675464570d0d406e0dfe034d74f48/ios/chrome/browser/tabs/tab.mm
[modify] https://crrev.com/ac9ac88fef0675464570d0d406e0dfe034d74f48/ios/chrome/browser/tabs/tab_model_selected_tab_observer.mm

Comment 2 by mrefaat@chromium.org, Oct 24

Status: Fixed (was: Assigned)

Comment 3 by mrefaat@chromium.org, Oct 24

Labels: -Pri-3 small q4 Pri-2

Comment 4 by mrefaat@chromium.org, Oct 25

Labels: -ios-tab-refactoring-cleanup S-tab-remove-obsolete-methods ios-tab-refactoring O-ios-tab-cleanup
►

Sign in to add a comment