Revert "Crash intentionally when requestTiles with invalid key."
This reverts commit 81be6ff273388c5fba75942d2c0ed58e7becb610.
We don't get any reports of crash related to this. So revert this.
BUG: 17401299
BUG: 17369933
Change-Id: I051d82f0aebda4bd148109d658597e0756c9befb
diff --git a/android_webview/browser/browser_view_renderer.cc b/android_webview/browser/browser_view_renderer.cc
index 1bf4f3d..a736218 100644
--- a/android_webview/browser/browser_view_renderer.cc
+++ b/android_webview/browser/browser_view_renderer.cc
@@ -197,12 +197,7 @@
GlobalTileManager* manager = GlobalTileManager::GetInstance();
// The following line will call BrowserViewRenderer::SetTilesNum().
- bool success =
- manager->RequestTiles(new_policy.num_resources_limit, tile_manager_key_);
- CHECK(success) << "tile_manager_key_ is invalid."
- << "\n Current memory policy: bytes_limit " << memory_policy_.bytes_limit
- << " num_resources_limit " << memory_policy_.num_resources_limit
- << "\n hardware enabled " << hardware_enabled_;
+ manager->RequestTiles(new_policy.num_resources_limit, tile_manager_key_);
}
void BrowserViewRenderer::SetNumTiles(size_t num_tiles,
diff --git a/android_webview/browser/global_tile_manager.cc b/android_webview/browser/global_tile_manager.cc
index 5fc5d07..a2fe4b6 100644
--- a/android_webview/browser/global_tile_manager.cc
+++ b/android_webview/browser/global_tile_manager.cc
@@ -64,12 +64,9 @@
num_tiles_limit_ = num_tiles_limit;
}
-bool GlobalTileManager::RequestTiles(size_t new_num_of_tiles, Key key) {
+void GlobalTileManager::RequestTiles(size_t new_num_of_tiles, Key key) {
DCHECK(IsConsistent());
DCHECK(sequence_checker_.CalledOnValidSequencedThread());
- if (std::find(mru_list_.begin(), mru_list_.end(), *key) == mru_list_.end()) {
- return false;
- }
size_t old_num_of_tiles = (*key)->GetNumTiles();
size_t num_of_active_views = std::distance(mru_list_.begin(), key) + 1;
size_t tiles_per_view_limit;
@@ -84,7 +81,7 @@
if (new_total_allocated_tiles <= num_tiles_limit_) {
total_allocated_tiles_ = new_total_allocated_tiles;
(*key)->SetNumTiles(new_num_of_tiles, false);
- return true;
+ return;
}
// Does not have enough tiles. Now evict other clients' tiles.
@@ -96,11 +93,11 @@
new_total_allocated_tiles -= evicted_tiles;
total_allocated_tiles_ = new_total_allocated_tiles;
(*key)->SetNumTiles(new_num_of_tiles, false);
- return true;
+ return;
} else {
total_allocated_tiles_ = num_tiles_limit_;
(*key)->SetNumTiles(tiles_left + old_num_of_tiles + evicted_tiles, false);
- return true;
+ return;
}
}
diff --git a/android_webview/browser/global_tile_manager.h b/android_webview/browser/global_tile_manager.h
index 10d15e2..3473c20 100644
--- a/android_webview/browser/global_tile_manager.h
+++ b/android_webview/browser/global_tile_manager.h
@@ -36,8 +36,7 @@
// tiles are available for the client. If the number of tiles left is not
// enough to satisfy the request, the manager will evict tiles allocated to
// other clients.
- // Returns false if the key cannot be found in |mru_list_|.
- bool RequestTiles(size_t new_num_of_tiles, Key key);
+ void RequestTiles(size_t new_num_of_tiles, Key key);
Key PushBack(GlobalTileManagerClient* client);