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

Unified Diff: chrome/browser/resources/options2/chromeos/change_picture_options.js

Issue 9382032: [uber page] Migrate the change picture sub-page to a dialog (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: rebase Created 8 years, 10 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: chrome/browser/resources/options2/chromeos/change_picture_options.js
diff --git a/chrome/browser/resources/options2/chromeos/change_picture_options.js b/chrome/browser/resources/options2/chromeos/change_picture_options.js
index 822df96574f5f75e04537dc069a552821e4dc875..0da7c0d12eaee49232aed27c38375fa963a90ec7 100644
--- a/chrome/browser/resources/options2/chromeos/change_picture_options.js
+++ b/chrome/browser/resources/options2/chromeos/change_picture_options.js
@@ -68,6 +68,8 @@ cr.define('options', function() {
// Old user image data (if present).
this.oldImage_ = null;
+ $('change-picture-overlay-confirm').onclick = this.closePage_;
+
chrome.send('onChangePicturePageInitialized');
},
@@ -96,7 +98,7 @@ cr.define('options', function() {
* @private
*/
closePage_: function() {
- OptionsPage.navigateToPage('personal');
+ OptionsPage.closeOverlay();
},
/**
« no previous file with comments | « chrome/browser/resources/options2/chromeos/change_picture_options.html ('k') | chrome/browser/resources/options2/options.html » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698