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

Side by Side Diff: chrome/renderer/resources/extensions/inject_app_titlebar.js

Issue 10896032: HTML titlebars for v2 apps. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 8 years, 3 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 unified diff | Download patch | Annotate | Revision Log
OLDNEW
(Empty)
1 exports.didCreateDocumentElement = function(inject_titlebar) {
2 if (inject_titlebar) {
3 var root = new WebKitShadowRoot(document.childNodes[0]);
4 root.appendChild(document.createElement('style')).innerText =
5 "x-titlebar { height: 24px; width: 100%; position: absolute; left: 0; to p: 0; }\n" +
6 "div { padding-top: 24px; }\n" +
7 ":-webkit-full-screen * { display: none; }\n" +
8 ":-webkit-full-screen-document * { display: none; }\n" +
9 "div:-webkit-full-screen, div:-webkit-full-screen-document { padding-top : 0; }\n" +
10 "button { -webkit-widget-region: region(control rectangle); }\n" +
11 "button.close { border: 0; background-color: transparent; width: 16px; h eight: 16px; position: absolute; right: 4px; top: 4px; }\n" +
12 // TODO(jeremya): these are ui/resources/default_100_percent/close_bar*
13 // and it would be good to find a way to directly reference them
Mihai Parparita -not on Chrome 2012/08/30 01:13:47 We have a chrome-extension-resource for resources
14 // instead of duplicating them here.
15 "button.close { background-image: url(data:image/png;base64,iVBORw0KGgoA AAANSUhEUgAAABAAAAAQCAYAAAAf8/9hAAAA9ElEQVQ4T7VTQQ6CMBCk0HAyIfAQbiZ+QHyDL/QLxqvx 4MWDB+MvFAWMAuKsacmmSjkQSDbQ2Z3Z3WkQzsBHDOQ7owgsMdUacTGmi3BeIFYcNycgciGlfFRVtcd3 qoojz/PmdV0XOD8RGy1iCoQgT5G8IyREjni7ICcg58ilwA7A8i4BwgMUxkKIV9M0PggTAoFlJpnwLhO5 iEuFapq2s20CyoWIGbpeaRICyrI889FtAtqwGxdQ65yYsV8NcwVN5obR/uTJW4mQsfp2fgToGjPqbBjW eoJVfNRsbSskSO7+7BsAiznZdgu6Qe97lH+htysv+AA10msRAt5JYQAAAABJRU5ErkJggg==); }\n" +
16 "button.close:hover { background-image: url(data:image/png;base64,iVBORw 0KGgoAAAANSUhEUgAAABAAAAAQCAYAAAAf8/9hAAABTElEQVQ4T2NkoBAwUqifAcWA////KwANFAPiV4 yMjA+QDcclBzcApCA6Otpz2bJluQkJCf3z58/fDTMEnxyyAWZADQuAtj4B4ncpKSnbZs+efQjkCqjBmU DmMyD+ADSwD6j2FEgOxQWJiYmuCxYscIYawpWamnr89+/fHECxbKjmB2VlZbs6OzsvwFyHEQZATXZz5s yxAGr4BMR8QCwJDYvn1dXVO1taWi4ihw9GLID8m5aWZgt0viXUEBaQAUDNh9E1o3gBFuIgA6Be8QKK3Q XiLyA5oNMvIDsdph7DC9AASwcquI9sAJDNk5GRcX769OlHsXoBKapAoQ2KiQcgPwMDkbGrq8sGyP8DCh NQwM6aNeswRiACDYBF4yOgwnuwAAM5NTMz03rGjBnWsIAFql2ANxqB/l2B7F/kgCUYjUBbyEvKsFAlla Y4NwIAmJDPEd+LFvYAAAAASUVORK5CYII=); }\n" +
17 "button.close:active { background-image: url(data:image/png;base64,iVBOR w0KGgoAAAANSUhEUgAAABAAAAAQCAYAAAAf8/9hAAAAZ0lEQVQ4T2NkoBAwUqifgeoG2AFd1AfERUB8C M11WOXQXXAGSROyITDNMGkTGAPdAHSFIENAAOQqGEBxHbYwQDcE2ScYXsMViNgMwRYuOGOBIgMo8gLFg Ui1aCQ7IZGcNaieF0h2AQCMABwRdsuhtQAAAABJRU5ErkJggg==); }\n"
18 var container = root.appendChild(document.createElement('div'));
19 var titlebar = container.appendChild(document.createElement('x-titlebar'));
20 var closeButton = titlebar.appendChild(document.createElement('button'));
21 closeButton.setAttribute('class', 'close');
22 closeButton.addEventListener('click', function() { window.close(); });
23 container.appendChild(document.createElement('content'));
24 }
25 document.onreadystatechange = function() {
Mihai Parparita -not on Chrome 2012/08/30 01:13:47 Can you not use this to determine when to inject t
jeremya 2012/08/30 01:45:18 Experimentally, it doesn't. I only get the 'intera
jeremya 2012/08/30 01:50:06 (and also: readystatechange doesn't bubble to wind
26 console.log(document.readyState);
27 }
abarth-chromium 2012/08/29 23:51:20 Presumably you'll want to remove this part before
jeremya 2012/08/30 02:21:02 Done.
28 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698