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

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: Response to CR + whitespace fix 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..7833480fef0398381f29bc4a50fd9eee69632227 100644
--- a/src/js/polyfills/runtime.polyfill.js
+++ b/src/js/polyfills/runtime.polyfill.js
@@ -102,7 +102,6 @@ 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;
/**
@@ -260,4 +259,4 @@ chrome.runtime.getPackageDirectoryEntry = function() {
// TODO(alger): Implement or stub onMessageExternal.
// TODO(alger): Implement or stub onRestartRequired.
-}).call(this);
+}).call(this);

Powered by Google App Engine
This is Rietveld 408576698