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

Unified Diff: trunk/src/chrome/browser/resources/options/import_data_overlay.js

Issue 22671003: Revert 216860 "Add option to import from bookmarks html file to ..." (Closed) Base URL: svn://svn.chromium.org/chrome/
Patch Set: Created 7 years, 4 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: trunk/src/chrome/browser/resources/options/import_data_overlay.js
===================================================================
--- trunk/src/chrome/browser/resources/options/import_data_overlay.js (revision 216877)
+++ trunk/src/chrome/browser/resources/options/import_data_overlay.js (working copy)
@@ -58,10 +58,6 @@
ImportDataOverlay.dismiss();
};
- $('import-choose-file').onclick = function() {
- chrome.send('chooseBookmarksFile');
- };
-
$('import-data-show-bookmarks-bar').onchange = function() {
// Note: The callback 'toggleShowBookmarksBar' is handled within the
// browser options handler -- rather than the import data handler --
@@ -121,10 +117,6 @@
*/
updateCheckboxes_: function() {
var index = $('import-browsers').selectedIndex;
- var bookmarksFileSelected = index == this.browserProfiles.length - 1;
- $('import-choose-file').hidden = !bookmarksFileSelected;
- $('import-data-commit').hidden = bookmarksFileSelected;
-
var browserProfile;
if (this.browserProfiles.length > index)
browserProfile = this.browserProfiles[index];
@@ -132,7 +124,6 @@
for (var i = 0; i < importOptions.length; i++) {
var checkbox = $('import-' + importOptions[i]);
var enable = browserProfile && browserProfile[importOptions[i]];
- checkbox.checked = enable;
this.setUpCheckboxState_(checkbox, enable);
var checkboxWithLabel = $('import-' + importOptions[i] + '-with-label');
checkboxWithLabel.style.display = enable ? '' : 'none';

Powered by Google App Engine
This is Rietveld 408576698