diff options
| author | finnur@chromium.org <finnur@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2013-05-02 20:48:48 +0000 |
|---|---|---|
| committer | finnur@chromium.org <finnur@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2013-05-02 20:48:48 +0000 |
| commit | 41208b9eb3885b1bd3a4e69542f541f1f7df8a23 (patch) | |
| tree | 08d93fe965bde57b8fac42faf31f172740937326 /chrome/browser | |
| parent | de72480d18b2da3f5586ad77cf4eb1d2977b6661 (diff) | |
| download | chromium_src-41208b9eb3885b1bd3a4e69542f541f1f7df8a23.zip chromium_src-41208b9eb3885b1bd3a4e69542f541f1f7df8a23.tar.gz chromium_src-41208b9eb3885b1bd3a4e69542f541f1f7df8a23.tar.bz2 | |
Fix: 'You have no extensions' message overlapping installed extensions.
If the ID specified as a query parameter doesn't match any extension in the list, there is a JS error when calculating the offset of the highlight, which leads to the message not being hidden. This fixes the JS error so the execution flow can continue normally.
BUG=224191
Review URL: https://chromiumcodereview.appspot.com/14852008
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@197971 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/browser')
| -rw-r--r-- | chrome/browser/resources/extensions/extension_list.js | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/chrome/browser/resources/extensions/extension_list.js b/chrome/browser/resources/extensions/extension_list.js index 843e0af..348f4c3 100644 --- a/chrome/browser/resources/extensions/extension_list.js +++ b/chrome/browser/resources/extensions/extension_list.js @@ -55,7 +55,7 @@ cr.define('options', function() { this.data_.extensions.forEach(this.createNode_, this); var idToHighlight = this.getIdQueryParam_(); - if (idToHighlight) { + if (idToHighlight && $(idToHighlight)) { // Scroll offset should be calculated slightly higher than the actual // offset of the element being scrolled to, so that it ends up not all // the way at the top. That way it is clear that there are more elements |