Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(425)

Unified Diff: src/js/polyfills/runtime.polyfill.js

Issue 1644913002: Caterpillar now generates app.info.js, a metadata script. Resolves #12. (Closed) Base URL: git@github.com:chromium/caterpillar.git@dependency-script-injection
Patch Set: Created 4 years, 11 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: src/js/polyfills/runtime.polyfill.js
diff --git a/src/js/polyfills/runtime.polyfill.js b/src/js/polyfills/runtime.polyfill.js
index 4d528795c5e31ace78ec1aca418cb7b04bbfac04..2c4fe22ba78c2163bb208a7b9fd73ed51bf48abc 100644
--- a/src/js/polyfills/runtime.polyfill.js
+++ b/src/js/polyfills/runtime.polyfill.js
@@ -102,8 +102,7 @@ chrome.runtime.lastError = null;
/**
* The ID of the app, or null if not applicable.
*/
-// TODO(alger): Consider getting this from the Chrome App manifest.
-chrome.runtime.id = null;
+chrome.runtime.id = chrome.caterpillar.appId;
/**
* Retrieves the JavaScript 'window' object for the background page running

Powered by Google App Engine
This is Rietveld 408576698