Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(648)

Unified Diff: ios/web/active_state_manager_impl.mm

Issue 2251023003: Remove single active state restriction. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 4 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: ios/web/active_state_manager_impl.mm
diff --git a/ios/web/active_state_manager_impl.mm b/ios/web/active_state_manager_impl.mm
index ca0fadc24b01a9be54c7211805a5eb39c24b7982..2f6b88671bd45484ce54e737ac347d386e9d26f3 100644
--- a/ios/web/active_state_manager_impl.mm
+++ b/ios/web/active_state_manager_impl.mm
@@ -10,12 +10,6 @@
namespace web {
-namespace {
-// The number of ActiveStateManagers that are currently in active state.
-// At most one ActiveStateManager can be active at any given time.
-int g_active_state_manager_active_count = 0;
-} // namespace
-
ActiveStateManagerImpl::ActiveStateManagerImpl(BrowserState* browser_state)
: browser_state_(browser_state), active_(false) {
DCHECK_CURRENTLY_ON(WebThread::UI);
@@ -33,12 +27,6 @@ void ActiveStateManagerImpl::SetActive(bool active) {
if (active == active_) {
return;
}
- if (active) {
- ++g_active_state_manager_active_count;
- } else {
- --g_active_state_manager_active_count;
- }
- DCHECK_GE(1, g_active_state_manager_active_count);
active_ = active;
if (active) {
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698