Chromium Code Reviews| Index: chrome/browser/resources/ntp4/new_tab.js |
| diff --git a/chrome/browser/resources/ntp4/new_tab.js b/chrome/browser/resources/ntp4/new_tab.js |
| index 53f3a53022cdb132620cf63f1dc5837b6b2e8fac..4a7b1de0027c59c89e9e7a3963deeef1263d1314 100644 |
| --- a/chrome/browser/resources/ntp4/new_tab.js |
| +++ b/chrome/browser/resources/ntp4/new_tab.js |
| @@ -359,12 +359,21 @@ cr.define('ntp4', function() { |
| * @param {string} loginHeader The first line of text. |
| * @param {string} loginSubHeader The second line of text. |
| */ |
| - function updateLogin(loginHeader, loginSubHeader) { |
| + function updateLogin(loginHeader, loginSubHeader, icon_url) { |
|
Evan Stade
2011/12/02 00:03:26
camel case and documentation
sail
2011/12/02 01:19:57
Done.
|
| if (loginHeader || loginSubHeader) { |
| $('login-container').hidden = false; |
| $('login-status-header').innerHTML = loginHeader; |
| $('login-status-sub-header').innerHTML = loginSubHeader; |
| $('card-slider-frame').classList.add('showing-login-area'); |
| + |
| + if (icon_url) { |
| + console.log("setting icon"); |
|
Evan Stade
2011/12/02 00:03:26
debug output?
sail
2011/12/02 01:19:57
Done.
|
| + $('login-status-icon').hidden = false; |
| + $('login-status-icon').style.backgroundImage = url(icon_url); |
| + } else { |
| + console.log("not setting icon"); |
| + $('login-status-icon').hidden = true; |
| + } |
| } else { |
| $('login-container').hidden = true; |
| $('card-slider-frame').classList.remove('showing-login-area'); |
| @@ -373,6 +382,8 @@ cr.define('ntp4', function() { |
| window.setTimeout(loginBubble.show.bind(loginBubble), 0); |
| chrome.send('loginMessageSeen'); |
| shouldShowLoginBubble = false; |
| + } else if (loginBubble) { |
| + loginBubble.reposition(); |
| } |
| } |