chrome/browser/resources/uber/uber_frame.html - Issue 9265020: [uber] make the navigation controls an iframe

Side by Side Diff

Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.

Keyboard Shortcuts

	File
u :	up to issue
j / k :	jump to file after / before current file
J / K :	jump to next file with a comment after / before current file
	Side-by-side diff
i :	toggle intra-line diffs
e :	expand all comments
c :	collapse all comments
s :	toggle showing all comments
n / p :	next / previous diff chunk or comment
N / P :	next / previous comment
<Up> / <Down> :	next / previous line

	Issue
u :	up to list of issues
j / k :	jump to patch after / before current patch
o / <Enter> :	open current patch in side-by-side view
i :	open current patch in unified diff view

	Issue List
j / k :	jump to issue after / before current issue
o / <Enter> :	open current issue

Side by Side Diff: chrome/browser/resources/uber/uber_frame.html

Issue 9265020: [uber] make the navigation controls an iframe (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src

Patch Set: Created 8 years, 11 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

« chrome/browser/resources/uber/uber.js ('K') | « chrome/browser/resources/uber/uber_frame.css ('k') | chrome/browser/resources/uber/uber_frame.js » ('j') | chrome/browser/resources/uber/uber_frame.js » ('J')
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')

OLD	NEW
(Empty)
	1 <!DOCTYPE html>

	2 <html i18n-values="dir:textdirection;" id="uber">

	3 <head>

	4 <meta charset="utf-8">

	5

	6 <link rel="stylesheet" href="chrome://resources/css/chrome_shared2.css">

	7 <link rel="stylesheet" href="uber_frame.css">

	8

	9 <script src="chrome://resources/js/cr.js"></script>

	10 <script src="chrome://uber-frame/uber_frame.js"></script>
	Dan Beam 2012/01/20 00:59:02 Why are these in the <head>? Why are these in the <head>?
	11 </head>

	12

	13 <body>

	14

	15 <h1 i18n-content="shortProductName"></h1>

	16 <ol>

	17 <li i18n-content="settingsDisplayName"

	18 i18n-values="controls:settingsHost" class="selected"></li>

	19 <li i18n-content="extensionsDisplayName"

	20 i18n-values="controls:extensionsHost"></li>
	csilv 2012/01/20 03:30:11 The mocks show 'Extensions' above 'Settings'. Not The mocks show 'Extensions' above 'Settings'. Not required for this CL, but I thought I'd mention it now. Evan Stade 2012/01/20 03:36:07 ok, I'll ask Glen (since the mocks have a lot more Show quoted text On 2012/01/20 03:30:11, csilv wrote: > The mocks show 'Extensions' above 'Settings'. Not required for this CL, but I > thought I'd mention it now. ok, I'll ask Glen (since the mocks have a lot more than just extensions and settings)
	21 <if expr="pp_ifdef('chromeos')">

	22 <li i18n-content="aboutPageDisplayName" id="aboutPageNavItem"

	23 i18n-values="controls:aboutPageHost"></li>

	24 </if>

	25 </ol>

	26

	27 <script src="chrome://uber-frame/strings.js"></script>

	28 <script src="chrome://resources/js/i18n_template.js"></script>

	29 <script src="chrome://resources/js/i18n_process.js"></script>

	30

	31 </body>

	32 </html>

OLD	NEW