chrome/browser/resources/identity_api_ui/scope_approval_dialog.html - Issue 14768009: Identity API: Add component app for auth flow UI

Unified Diff: chrome/browser/resources/identity_api_ui/scope_approval_dialog.html

Issue 14768009: Identity API: Add component app for auth flow UI (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src

Patch Set: no permission message for identityPrivate Created 7 years, 7 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

« chrome/browser/resources/identity_api_ui/identity_api_ui.js ('K') | « chrome/browser/resources/identity_api_ui/manifest.json ('k') | chrome/browser/resources/identity_api_ui/scope_approval_dialog.js » ('j') | chrome/browser/resources/identity_api_ui/scope_approval_dialog.js » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')

Index: chrome/browser/resources/identity_api_ui/scope_approval_dialog.html

diff --git a/chrome/browser/resources/identity_api_ui/scope_approval_dialog.html b/chrome/browser/resources/identity_api_ui/scope_approval_dialog.html

new file mode 100644

index 0000000000000000000000000000000000000000..548ef23e7d91d22375f798b380199b6feae4b3ee

--- /dev/null

+++ b/chrome/browser/resources/identity_api_ui/scope_approval_dialog.html

@@ -0,0 +1,13 @@

+<html>

+ <head>

+ <title>Identity API Scope Approval Dialog</title>

James Hawkins 2013/05/13 22:24:07 Needs to be localized.

Michael Courage 2013/05/14 17:10:56 Would it be ok to just remove the title? It doesn'

James Hawkins 2013/05/15 15:24:45 Yes, remove it if it's not visible. Are you sure

Michael Courage 2013/05/15 18:39:35 You're right, it's visible after all. I added a ti

+ <link rel="stylesheet" type="text/css" href="identity_api_ui.css"/>

+ </head>

+ <body>

+ <div class="titlebar">

+ <span class="titlebar-close-button"></span>

+ </div>

+ <webview class="content" id="providerview" autosize="true"></webview>

+ <script src="scope_approval_dialog.js"></script>

James Hawkins 2013/05/13 22:24:07 Why is this not in the head element?

Michael Courage 2013/05/14 17:10:56 Just to avoid having to wait for DOMContentLoaded

+ </body>

+</html>