Issue 12426011: Creating a directory for out of process iframe test tools.

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

Issue 12426011: Creating a directory for out of process iframe test tools. (Closed)

Created:
7 years, 9 months ago by awong

Modified:
7 years, 9 months ago

Reviewers:
tonyg, nasko

CC:
chromium-reviews, chrome-speed-team+watch_google.com

Base URL:
svn://svn.chromium.org/chrome/trunk/src

Visibility:
Public.

More Reviews

Description

Creating a directory for out of process iframe test tools. TBR=tonyg BUG=188235 Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=187902

Patch Set 1 #

Created: 7 years, 9 months ago

Download [raw] [tar.bz2]

	Unified diffs	Side-by-side diffs	Stats (+3 lines, -223 lines)			Patch
A	tools/oopif/OWNERS	View	1 chunk	+4 lines, -0 lines	0 comments	Download
A +	tools/oopif/iframe_server.py	View	0 chunks	+-1 lines, --1 lines	0 comments	Download
D	tools/perf/perf_tools/iframe_server.py	View	1 chunk	+0 lines, -224 lines	0 comments	Download

Messages

Total messages: 4 (0 generated)

Expand Messages | Collapse Messages

tonyg

I'm not sure oopif is a recognizable enough acronym for a directory name. What else ...

7 years, 9 months ago (2013-03-13 18:34:49 UTC) #2

awong

Committed patchset #1 manually as r187902 (presubmit successful).

7 years, 9 months ago (2013-03-13 18:35:52 UTC) #3

awong

7 years, 9 months ago (2013-03-13 18:37:43 UTC) #4

Message was sent while issue was closed.

On 2013/03/13 18:35:52, awong wrote:
> Committed patchset #1 manually as r187902 (presubmit successful).

Since it was blocking more and more people, I just landed the move.  I imagine
we will have more tools in for out of process iframes soon.  We're finally
getting to the point of wanting to commit some code.  If you feel strongly about
it, I can move it back into tools/perf.

Expand Messages | Collapse Messages