Index: chrome/browser/extensions/api/tabs/tabs_api.cc |
diff --git a/chrome/browser/extensions/api/tabs/tabs_api.cc b/chrome/browser/extensions/api/tabs/tabs_api.cc |
index 302de3ce81035206e90912b4361e4c2b51a94163..fded9c9929014d6e5a67fbaa945e339164d95a5c 100644 |
--- a/chrome/browser/extensions/api/tabs/tabs_api.cc |
+++ b/chrome/browser/extensions/api/tabs/tabs_api.cc |
@@ -1676,20 +1676,7 @@ WebContents* TabsCaptureVisibleTabFunction::GetWebContentsForID(int window_id) { |
return NULL; |
} |
- // Use the last committed URL rather than the active URL for permissions |
- // checking, since the visible page won't be updated until it has been |
- // committed. A canonical example of this is interstitials, which show the |
- // URL of the new/loading page (active) but would capture the content of the |
- // old page (last committed). |
- // |
- // TODO(creis): Use WebContents::GetLastCommittedURL instead. |
- // http://crbug.com/237908. |
- NavigationEntry* last_committed_entry = |
- contents->GetController().GetLastCommittedEntry(); |
- GURL last_committed_url = last_committed_entry ? |
- last_committed_entry->GetURL() : GURL(); |
if (!PermissionsData::CanCaptureVisiblePage(GetExtension(), |
- last_committed_url, |
SessionID::IdForTab(contents), |
&error_)) { |
return NULL; |