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

Unified Diff: chrome/browser/resources/settings/on_startup_page/startup_urls_page.html

Issue 1849543006: MD Settings: validate startup URLs (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@startup-browser-proxy
Patch Set: compile fix Created 4 years, 9 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/settings/on_startup_page/startup_urls_page.html
diff --git a/chrome/browser/resources/settings/on_startup_page/startup_urls_page.html b/chrome/browser/resources/settings/on_startup_page/startup_urls_page.html
index 07b7c1104507f49717aee83283d36a314aac6abc..6819ca7d9e05f4daecc57daaae3da841b2f4496e 100644
--- a/chrome/browser/resources/settings/on_startup_page/startup_urls_page.html
+++ b/chrome/browser/resources/settings/on_startup_page/startup_urls_page.html
@@ -30,7 +30,7 @@
<iron-icon on-tap="onRemoveUrlTap_" icon="clear"></iron-icon>
</div>
</template>
- <div class="list-item list-button" on-tap="onAddPageTap_"
+ <div class="list-item list-button" id="addPage" on-tap="onAddPageTap_"
i18n-content="onStartupAddNewPage">
</div>
<div class="list-item list-button" on-tap="onUseCurrentPagesTap_"
@@ -41,7 +41,7 @@
<settings-dialog id="addUrlDialog">
<div class="title" i18n-content="onStartupAddNewPage"></div>
<div class="body">
- <paper-input class="flex" always-float-label
+ <paper-input class="flex" always-float-label id="newUrl"
i18n-values="label:onStartupSiteUrl" value="{{newUrl_}}">
</paper-input>
</div>
@@ -49,8 +49,8 @@
<div class="action-buttons">
<paper-button class="cancel-button" on-tap="onCancelTap_"
id="cancel" i18n-content="cancel"></paper-button>
- <paper-button class="action-button" on-tap="onAddTap_"
- i18n-content="add" disabled="[[!isAddEnabled_(newUrl_)]]">
+ <paper-button id="add" class="action-button" on-tap="onAddTap_"
+ i18n-content="add" disabled="[[!isNewUrlValid_]]">
</paper-button>
</div>
</div>

Powered by Google App Engine
This is Rietveld 408576698