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

Unified Diff: chrome/browser/resources/new_new_tab.js

Issue 392005: NTP: Hide "Make this my home page" after setting the home page.... (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: '' Created 11 years, 1 month 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
« no previous file with comments | « chrome/browser/resources/new_new_tab.html ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/resources/new_new_tab.js
===================================================================
--- chrome/browser/resources/new_new_tab.js (revision 31433)
+++ chrome/browser/resources/new_new_tab.js (working copy)
@@ -1236,7 +1236,7 @@
// This link allows user to make new tab page as homepage from the new tab
// page itself (without going to Options dialog box).
-document.addEventListener('DOMContentLoaded', showSetAsHomepageLink);
+document.addEventListener('DOMContentLoaded', showSetAsHomePageLink);
/**
* The sync code is not yet built by default on all platforms so we have to
@@ -1250,23 +1250,28 @@
}
function setAsHomePageLinkClicked(e) {
- chrome.send('SetHomepageLinkClicked');
+ chrome.send('setHomePage');
e.preventDefault();
}
-function showSetAsHomepageLink() {
- var setAsHomepageElement = $('set-as-homepage');
- var style = setAsHomepageElement.style;
+function showSetAsHomePageLink() {
+ var setAsHomePageElement = $('set-as-home-page');
+ var style = setAsHomePageElement.style;
if (document.documentElement.getAttribute('showsetashomepage') != 'true') {
// Hide the section (if new tab page is already homepage).
return;
}
style.display = 'block';
- var buttonElement = setAsHomepageElement.firstElementChild;
+ var buttonElement = setAsHomePageElement.firstElementChild;
buttonElement.addEventListener('click', setAsHomePageLinkClicked);
}
+function onHomePageSet(data) {
+ $('set-as-home-page').style.display = 'none';
+ showNotification(data[0], data[1]);
+}
+
function hideAllMenus() {
optionMenu.hide();
}
« no previous file with comments | « chrome/browser/resources/new_new_tab.html ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698