New issue
Advanced search Search tips

Issue 914967 link

Starred by 1 user
Status: Assigned
Owner:
mrefaat@chromium.org
Cc:
ios-bugs@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: iOS
Pri: 3
Type: Task
q4



Sign in to add a comment

Find an alternative to the usage webState.tab in BVC

Project Member Reported by mrefaat@chromium.org, Dec 13 
currently current tab is used to get the active webstate.
Try not to depend on tab to get the active webState in BVC.
Project Member

Comment 1 by bugdroid1@chromium.org, Dec 19 

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

commit 6c13fac5295b865418f8aa3cd53e4ef622d49724
Author: mrefaat <mrefaat@chromium.org>
Date: Wed Dec 19 00:19:12 2018

Use self.currentWebState as an abstraction to get current web state.

Bug: 914967
Change-Id: Ic6fdfbde78ccaf3cf484ed785b43fbf2946d8968
Reviewed-on: https://chromium-review.googlesource.com/c/1377498
Commit-Queue: Mohammad Refaat <mrefaat@chromium.org>
Reviewed-by: edchin <edchin@chromium.org>
Cr-Commit-Position: refs/heads/master@{#617685}
[modify] https://crrev.com/6c13fac5295b865418f8aa3cd53e4ef622d49724/ios/chrome/browser/ui/browser_view_controller.mm
►

Sign in to add a comment