Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Unified Diff: chrome/ext/background.js

Issue 5455624317960192: Issue 2079 - Use chrome.tabs.update instead chrome.tabs.highlight for compatibility with Opera (Closed)
Patch Set: Created March 2, 2015, 7:05 p.m.
Use n/p to move between diff chunks; N/P to move between comments.
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: chrome/ext/background.js
===================================================================
--- a/chrome/ext/background.js
+++ b/chrome/ext/background.js
@@ -519,13 +519,7 @@
var tab = tabs[0];
chrome.windows.update(tab.windowId, {focused: true});
- chrome.tabs.highlight(
- {
- windowId: tab.windowId,
- tabs: [tab.index]
- },
- function() {}
- );
+ chrome.tabs.update(tab.id, {active: true});
if (callback)
callback(new Page(tab));
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld