Delta Between Two Patch Sets: chrome/ext/popup.js
Issue 5220277533278208:
Made the bubble use the ext object from the background page instead of including ext/background.js (Closed)
Left Patch Set: Don't use with statement
Created Jan. 15, 2014, 4:53 p.m.
Right Patch Set: Assigned imported variables to window
Created Jan. 16, 2014, 10:49 a.m.
| Left: | ||
| Right: |
Use n/p to move between diff chunks;
N/P to move between comments.
« no previous file with change/comment
|
« no previous file
|
metadata.common » ('j')
|
no next file with change/comment »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')
| LEFT | RIGHT |
|---|---|
| 1 (function() | 1 (function() |
| 2 { | 2 { |
| 3 var backgroundPage = chrome.extension.getBackgroundPage(); | 3 var backgroundPage = chrome.extension.getBackgroundPage(); |
| 4 ext = backgroundPage.ext; | 4 window.ext = backgroundPage.ext; |
| 5 TabMap = backgroundPage.TabMap; | 5 window.TabMap = backgroundPage.TabMap; |
|
Wladimir Palant
2014/01/16 07:46:43
Assigning to undeclared variables isn't the fine a
| |
| 6 })(); | 6 })(); |
| LEFT | RIGHT |
« no previous file
|
metadata.common » ('j')
|
Toggle Intra-line Diffs ('i')
|
Expand Comments ('e')
|
Collapse Comments ('c')
|
Toggle Comments ('s')
