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

Unified Diff: chrome/browser/resources/chromeos/login/offline_ad_login.html

Issue 2433363004: Chromad: added AD Join ui, authpolicy_client (Closed)
Patch Set: Not close password_fd in AuthPolicyClient Created 4 years, 2 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/chromeos/login/offline_ad_login.html
diff --git a/chrome/browser/resources/chromeos/login/offline_ad_login.html b/chrome/browser/resources/chromeos/login/offline_ad_login.html
new file mode 100644
index 0000000000000000000000000000000000000000..009e818568b4df93d005cec015c626a5f7fc9c38
--- /dev/null
+++ b/chrome/browser/resources/chromeos/login/offline_ad_login.html
@@ -0,0 +1,61 @@
+<!-- Copyright 2016 The Chromium Authors. All rights reserved.
+ Use of this source code is governed by a BSD-style license that can be
+ found in the LICENSE file. -->
+
+<link rel="import" href="chrome://resources/polymer/v1_0/iron-flex-layout/classes/iron-flex-layout.html">
+<link rel="import" href="chrome://resources/polymer/v1_0/polymer/polymer.html">
+
+<!--
+ Offline UI for the AD Domain joining and User authentication.
+
+ Example:
+ <offline-ad-login show-machine-input> </offline-ad-login>
+
+ Attributes:
+ 'showMachineInput' - If the input of the machine name should be shown.
xiyuan 2016/10/26 22:03:16 nit: If -> Whether
Roman Sorokin (ftl) 2016/10/27 13:10:45 Done.
+ Used for domain joining.
xiyuan 2016/10/26 22:03:16 nit: Indent 4 more spaces or align with the beginn
Roman Sorokin (ftl) 2016/10/27 13:10:45 Done.
+ 'realm' - The AD realm the device is managed by.
+ 'userRealm' - autocomplete realm for the user input.
xiyuan 2016/10/26 22:03:16 nit: autocomplete -> Autocomplete
Roman Sorokin (ftl) 2016/10/27 13:10:45 Done.
+
+ Events:
+ 'authCompleted' - fired when user enters login and password. Fires with an
xiyuan 2016/10/26 22:03:16 nit: fired -> Fired, or keep the lower case and fi
Roman Sorokin (ftl) 2016/10/27 13:10:45 Done.
+ argument |credentials| which contains.
+ |credentials| = { 'machineName': <machine name input>,
+ 'username': <username> (UPN),
+ 'password': <typed password> }
+ Methods:
+ 'focus' - focuses current input (user input or password input);
+ 'setUser' - accepts arguments |user| and |machineName|
+-->
+<dom-module name="offline-ad-login">
+ <link rel="stylesheet" href="offline_gaia.css">
+ <template>
+ <gaia-card id="gaiaCard" class="fit">
xiyuan 2016/10/26 22:03:16 nit: wrong indent
Roman Sorokin (ftl) 2016/10/27 13:10:45 Done.
+ <div class="header flex vertical layout end-justified start">
+ <h1 id="welcomeMsg" class="welcome-message">[[adWelcomeMessage]]
+ </h1>
+ </div>
+ <div class="footer flex vertical layout justified">
+ <gaia-input-form on-submit="onSubmit_"
+ disabled="[[disabled]]"
+ i18n-values="button-text:offlineLoginNextBtn">
+ <gaia-input id="machineNameInput" required hidden="[[!showMachineInput]]"
xiyuan 2016/10/26 22:03:16 What does it mean when the field is required and a
xiyuan 2016/10/26 22:03:16 nit: wrap to fit in 80 chars per line
Roman Sorokin (ftl) 2016/10/27 13:10:45 Right! It's required if it's shown. Added check in
Roman Sorokin (ftl) 2016/10/27 13:10:45 Done.
+ i18n-values="label:oauthEnrollADMachineNameInput">
+ </gaia-input>
+ <gaia-input id="userInput" type="email" required
+ domain="[[userRealm]]"
+ i18n-values="error:offlineLoginInvalidEmail;
+ label:ADLoginUser">
+ </gaia-input>
+ <gaia-input id="passwordInput" type="password" required
+ i18n-values="error:offlineLoginInvalidPassword;
+ label:ADLoginPassword">
+ </gaia-input>
+ </gaia-input-form>
+ <img src="chrome://theme/IDR_LOGO_GOOGLE_COLOR_90"
+ class="self-center" alt="">
+ </div>
+ </gaia-card>
+ </template>
+</dom-module>
+

Powered by Google App Engine
This is Rietveld 408576698