chrome/browser/resources/chromeos/wallpaper_manager/harness.html - Issue 10698099: Implement the new wallpaper manager launcher window.

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/chromeos/wallpaper_manager/harness.html

Issue 10698099: Implement the new wallpaper manager launcher window. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src

Patch Set: Nits Created 8 years, 5 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/chromeos/extensions/wallpaper_manager_api.cc ('K') | « chrome/browser/chromeos/extensions/wallpaper_manager_api.cc ('k') | chrome/browser/resources/chromeos/wallpaper_manager/js/background.js » ('j') | chrome/browser/resources/chromeos/wallpaper_manager/js/background.js » ('J')
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')

OLD	NEW
(Empty)
	1 <!DOCTYPE HTML>

	2 <!--

	3 ------

	4 Copyright (c) 2012 The Chromium Authors. All rights reserved.

	5 Use of this source code is governed by a BSD-style license that can be

	6 found in the LICENSE file.

	7 -->

	8 <html i18n-values="dir:textdirection;">

	9 <head>

	10 <title>Wallpaper Manager Test Harness</title>

	11 <style>

	12 #dialog {

	13 position: absolute;

	14 width: 800px;

	15 height: 640px;

	16 border: none;

	17 -webkit-box-shadow: 0px 0px 20px rgba(0, 0, 0, 1);

	18 }

	19 #title-bar {

	20 background-color: gray;

	21 width: 800px;

	22 height: 30px;

	23 }

	24 </style>

	25 </head>

	26 <body>

	27 <div id=screen>

	28 <div id=container>

	29 <div id=title-bar>

	30 </div>

	31 <iframe id=dialog src="main.html"></iframe>

	32 </div>

	33 </div>

	34 <script src="js/mock_chrome.js"></script>
	flackr 2012/07/04 22:20:57 You could probably conditionally include mock_chro You could probably conditionally include mock_chrome.js from main.html file in the <if expr="0">. This way it would use mock_chrome when run outside of chrome and the script would not get included within chrome. bshe 2012/07/05 13:54:50 I was just hoping to make the mock looks as real a I was just hoping to make the mock looks as real as possible. So I add a grey title bar here to mock the the real title bar in the extension window. Also, I want to mock the set wallpaper function. Using harness, I can set the width and height of the wallpaper picker ui(iframe) and mock set wallpaper function by set the body background image of harness.html. But if I directly access main.html, I can't set a specific width/height for the picker ui(it's the same as the width and height of body) and it is no easy to mock the set wallpaper function. On 2012/07/04 22:20:57, flackr wrote: Show quoted text > You could probably conditionally include mock_chrome.js from main.html file in > the <if expr="0">. This way it would use mock_chrome when run outside of chrome > and the script would not get included within chrome.
	35 </body>

	36 </html>

OLD	NEW