New issue
Advanced search Search tips

Issue 911733 link

Starred by 1 user

Issue metadata

Status: Assigned
Owner:
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: iOS
Pri: 3
Type: Bug
q4
iOS



Sign in to add a comment

Tab cleanup misc.

Project Member Reported by mrefaat@google.com, Dec 4

Issue description

Random cleanup, remove unneeded code, and simplify some methods.
 
Project Member

Comment 1 by bugdroid1@chromium.org, Dec 4

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

commit 3b3c96ab2740878b4561deece189abc6d7666b2f
Author: mrefaat <mrefaat@chromium.org>
Date: Tue Dec 04 20:19:18 2018

Remove IsLoading DCHECK from didLoadPageWithSuccess in tab


Bug: 911733
Change-Id: I1c481d7796977f607336f9c2e6244326d6d0b93d
Reviewed-on: https://chromium-review.googlesource.com/c/1361655
Reviewed-by: Eugene But <eugenebut@chromium.org>
Commit-Queue: Mohammad Refaat <mrefaat@chromium.org>
Cr-Commit-Position: refs/heads/master@{#613672}
[modify] https://crrev.com/3b3c96ab2740878b4561deece189abc6d7666b2f/ios/chrome/browser/tabs/tab.mm

Sign in to add a comment