Provide and use a simpler version of ExtensionTabUtil::GetTabById.
14 out of 24 usages of ExtensionTabUtil::GetTabById didn't require
|browser|, |tab_strip| and |tab_index| and specified nullptr for
them. Provide a simpler version without these params and update
callers to use the simplified version.
Bug: None
Test: No visible changes expected, internal only cleanup.
Change-Id: I3cbdeee148c335b49c058b9498be097f43043c8b
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1717458
Reviewed-by: Devlin <[email protected]>
Commit-Queue: Istiaque Ahmed <[email protected]>
Cr-Commit-Position: refs/heads/master@{#681201}
diff --git a/chrome/browser/extensions/api/debugger/debugger_api.cc b/chrome/browser/extensions/api/debugger/debugger_api.cc
index e5cd5ab18..9b092996 100644
--- a/chrome/browser/extensions/api/debugger/debugger_api.cc
+++ b/chrome/browser/extensions/api/debugger/debugger_api.cc
@@ -421,10 +421,10 @@
bool DebuggerFunction::InitAgentHost() {
if (debuggee_.tab_id) {
- WebContents* web_contents = NULL;
+ WebContents* web_contents = nullptr;
bool result = ExtensionTabUtil::GetTabById(*debuggee_.tab_id, GetProfile(),
include_incognito_information(),
- NULL, NULL, &web_contents, NULL);
+ &web_contents);
if (result && web_contents) {
// TODO(rdevlin.cronin) This should definitely be GetLastCommittedURL().
GURL url = web_contents->GetVisibleURL();
