Permalink
Browse files

Cleanup and rename CodeInspector -> JITInspector

  • Loading branch information...
1 parent 0bbc32f commit 6c782d1b2ed723f51e9a3689011646fdaaa2532e Brian Hackett committed Mar 15, 2012
View
@@ -10,14 +10,15 @@ WITHOUT WARRANTY OF ANY KIND, either express or implied. See the License
for the specific language governing rights and limitations under the
License.
-The Original Code is Code Inspector code.
+The Original Code is JIT Inspector code.
The Initial Developer of the Original Code is Mozilla Foundation.
Portions created by the Initial Developer are Copyright (C) 2011
the Initial Developer. All Rights Reserved.
Contributor(s):
- Cedric Vivier <cedricv@neonux.com> (original author)
+ Brian Hackett <bhackett@mozilla.com>
+ Cedric Vivier <cedricv@neonux.com>
Alternatively, the contents of this file may be used under the terms of
either the GNU General Public License Version 2 or later (the "GPL"), or
View
@@ -2,7 +2,7 @@
# $ make
PWD=`pwd`
-XPI="${PWD}/build/CodeInspector.xpi"
+XPI="${PWD}/build/JITInspector.xpi"
FILES=\
LICENSE.txt \
README.md \
View
@@ -1,9 +1,4 @@
-# Code Inspector #
-
-
-## Description ##
-
-TODO:
+# JIT Inspector #
## Build Instructions ##
@@ -14,22 +9,12 @@ make
or:
-zip -r ../CodeInspector.xpi * -x ".git/*"
+zip -r ../JITInspector.xpi * -x ".git/*"
## Installation ##
-Note that the Code Inspector currently requires a custom Firefox Nightly Debug with
-additional patches at https://bugzilla.mozilla.org/show_bug.cgi?id=687134
-Drag and drop CodeInspector.xpi in Firefox, follow instructions to install add-on.
-
-
-## Testsuite ##
-
-To run the tests, you need to build Mozilla from source.
-Set environment variable OBJDIR to point to your tree's OBJDIR, then run :
-
-make test
+Drag and drop JITInspector.xpi in Firefox, follow instructions to install add-on.
## Legal ##
@@ -39,9 +24,3 @@ See LICENSE.txt for details.
The logo icon.png and derivatives are attributed to the W3C and licensed under
CC Attribution 3.0 Unported <http://creativecommons.org/licenses/by/3.0/>.
-
-
-## More ##
-
-For documentation, feedback, contributions :
-http://wiki.mozilla.org/DevTools/Features/CodeInspector
View
@@ -1,4 +1,4 @@
-content CodeInspector content/
-overlay chrome://browser/content/browser.xul chrome://CodeInspector/content/browser_overlay.xul
-locale CodeInspector en-US locale/en-US/
-skin CodeInspector classic/1.0 skin/
+content JITInspector content/
+overlay chrome://browser/content/browser.xul chrome://JITInspector/content/browser_overlay.xul
+locale JITInspector en-US locale/en-US/
+skin JITInspector classic/1.0 skin/
@@ -20,6 +20,7 @@
*
* Contributor(s):
* Brian Hackett <bhackett@mozilla.com> (original author)
+ * Cedric Vivier <cedricv@neonux.com>
*
* Alternatively, the contents of this file may be used under the terms of
* either the GNU General Public License Version 2 or later (the "GPL"), or
@@ -37,7 +38,7 @@
"use strict";
-const EXPORTED_SYMBOLS = ["CodeInspectorChrome"];
+const EXPORTED_SYMBOLS = ["JITInspectorChrome"];
const Cc = Components.classes;
const Ci = Components.interfaces;
@@ -46,9 +47,9 @@ const Cu = Components.utils;
Cu.import("resource://gre/modules/Services.jsm");
Cu.import("resource://gre/modules/NetUtil.jsm");
Cu.import("resource://gre/modules/FileUtils.jsm");
-Cu.import("chrome://CodeInspector/content/AdaptiveSplitView.jsm");
-Cu.import("chrome://CodeInspector/content/Coverage.jsm");
-Cu.import("chrome://CodeInspector/content/StyleEditorUtil.jsm");
+Cu.import("chrome://JITInspector/content/AdaptiveSplitView.jsm");
+Cu.import("chrome://JITInspector/content/Coverage.jsm");
+Cu.import("chrome://JITInspector/content/StyleEditorUtil.jsm");
function jsdump(str)
{
@@ -135,7 +136,7 @@ const LOAD_ERROR = "load-error";
const HTML_NS = "http://www.w3.org/1999/xhtml";
/**
- * CodeInspectorChrome constructor.
+ * JITInspectorChrome constructor.
*
* The 'chrome' of the Coverage Tool is all the UI that populates and updates
* the actual coverage reports.
@@ -146,9 +147,9 @@ const HTML_NS = "http://www.w3.org/1999/xhtml";
* Optional content DOMWindow to attach to this chrome.
* Default: the currently active browser tab content window.
*/
-function CodeInspectorChrome(aRoot, aContentWindow)
+function JITInspectorChrome(aRoot, aContentWindow)
{
- assert(aRoot, "Argument 'aRoot' is required to initialize CodeInspectorChrome.");
+ assert(aRoot, "Argument 'aRoot' is required to initialize JITInspectorChrome.");
this._root = aRoot;
this._document = this._root.ownerDocument;
@@ -181,7 +182,7 @@ function CodeInspectorChrome(aRoot, aContentWindow)
}
}
-CodeInspectorChrome.prototype = {
+JITInspectorChrome.prototype = {
/**
* Retrieve the content window attached to this chrome.
*
@@ -253,19 +254,19 @@ CodeInspectorChrome.prototype = {
get isContentAttached() this._isContentAttached,
/**
- * Add a listener for CodeInspectorChrome events.
+ * Add a listener for JITInspectorChrome events.
*
- * The listener implements ICodeInspectorChromeListener := {
+ * The listener implements IJITInspectorChromeListener := {
* onContentAttach: Called when a content window has been attached.
- * Arguments: (CodeInspectorChrome aChrome)
+ * Arguments: (JITInspectorChrome aChrome)
* @see contentWindow
*
* onContentDetach: Called when the content window has been detached.
- * Arguments: (CodeInspectorChrome aChrome)
+ * Arguments: (JITInspectorChrome aChrome)
* @see contentWindow
*
* onScriptAdded: Called when a script has been added to the UI.
- * Arguments (CodeInspectorChrome aChrome,
+ * Arguments (JITInspectorChrome aChrome,
* string scriptUri)
* }
*
@@ -618,7 +619,7 @@ function WalkScriptText(scriptIndex, contents)
var opcodes = {};
- // store opcodes on the CodeInspectorChrome for later use.
+ // store opcodes on the JITInspectorChrome for later use.
this.scripts[scriptIndex].opcodes = opcodes;
var maxActivity = 1;
@@ -35,17 +35,17 @@
-
- ***** END LICENSE BLOCK ***** -->
<!DOCTYPE window [
-<!ENTITY % CodeInspectorDTD SYSTEM "chrome://CodeInspector/locale/CodeInspector.dtd">
- %CodeInspectorDTD;
+<!ENTITY % JITInspectorDTD SYSTEM "chrome://JITInspector/locale/JITInspector.dtd">
+ %JITInspectorDTD;
]>
<?xml-stylesheet href="chrome://global/skin/global.css" type="text/css"?>
-<?xml-stylesheet href="chrome://CodeInspector/skin/AdaptiveSplitView.css" type="text/css"?>
-<?xml-stylesheet href="chrome://CodeInspector/skin/CodeInspectorChrome.css" type="text/css"?>
+<?xml-stylesheet href="chrome://JITInspector/skin/AdaptiveSplitView.css" type="text/css"?>
+<?xml-stylesheet href="chrome://JITInspector/skin/JITInspectorChrome.css" type="text/css"?>
<xul:window xmlns:xul="http://www.mozilla.org/keymaster/gatekeeper/there.is.only.xul"
xmlns="http://www.w3.org/1999/xhtml"
- id="CodeInspector-window"
+ id="JITInspector-window"
title="&window.title;"
- windowtype="Tools:CodeInspector"
+ windowtype="Tools:JITInspector"
width="640" height="440"
persist="screenX screenY width height sizemode">
<xul:script type="application/javascript" src="chrome://global/content/globalOverlay.js"/>
@@ -150,9 +150,9 @@ No profiles to display.
</div> <!-- .splitview-root -->
<xul:script type="application/javascript"><![CDATA[
-Components.utils.import("chrome://CodeInspector/content/CodeInspectorChrome.jsm");
+Components.utils.import("chrome://JITInspector/content/JITInspectorChrome.jsm");
let chromeRoot = document.getElementById("code-inspector-view");
-let chrome = new CodeInspectorChrome(chromeRoot);
+let chrome = new JITInspectorChrome(chromeRoot);
window.codeInspectorChrome = chrome;
]]></xul:script>
</xul:window>
@@ -53,7 +53,7 @@ const Cu = Components.utils;
Cu.import("resource://gre/modules/Services.jsm");
-const PROPERTIES_URL = "chrome://CodeInspector/locale/CodeInspector.properties";
+const PROPERTIES_URL = "chrome://JITInspector/locale/JITInspector.properties";
const console = Services.console;
const gStringBundle = Services.strings.createBundle(PROPERTIES_URL);
@@ -35,19 +35,19 @@
-
- ***** END LICENSE BLOCK ***** -->
<!DOCTYPE overlay [
-<!ENTITY % codeInspectorDTD SYSTEM "chrome://CodeInspector/locale/CodeInspector.dtd">
+<!ENTITY % codeInspectorDTD SYSTEM "chrome://JITInspector/locale/JITInspector.dtd">
%codeInspectorDTD;
]>
<?xml-stylesheet href="chrome://global/skin/global.css" type="text/css"?>
<overlay id="style-editor-browser-overlay"
xmlns="http://www.mozilla.org/keymaster/gatekeeper/there.is.only.xul">
<script type="application/javascript"><![CDATA[
- var CodeInspector = {
+ var JITInspector = {
prefEnabledName: "devtools.codeinspector.enabled",
open: function CT_open()
{
- const WINDOW_URL = "chrome://CodeInspector/content/CodeInspectorChrome.xul";
- const WINDOW_TYPE = "Tools:CodeInspector";
+ const WINDOW_URL = "chrome://JITInspector/content/JITInspectorChrome.xul";
+ const WINDOW_TYPE = "Tools:JITInspector";
const WINDOW_FLAGS = "chrome,centerscreen,innerHeight=735,innerWidth=705,dependent";
// focus currently open Coverage Tool window for this document, if any
@@ -75,20 +75,20 @@
label="&codeinspector.label;"
accesskey="&codeinspector.accesskey;"
key="key_codeinspector"
- command="Tools:CodeInspector"/>
+ command="Tools:JITInspector"/>
</menupopup>
<menupopup id="appmenu_webDeveloper_popup">
<menuitem id="appmenu_codeInspector"
insertbefore="appmenu_devToolsEndSeparator"
label="&codeinspector.label;"
key="key_codeinspector"
- command="Tools:CodeInspector"/>
+ command="Tools:JITInspector"/>
</menupopup>
<commandset id="mainCommandSet">
- <command id="Tools:CodeInspector"
+ <command id="Tools:JITInspector"
insertafter="Tools:Scratchpad"
- oncommand="CodeInspector.open();"/>
+ oncommand="JITInspector.open();"/>
</commandset>
<keyset id="mainKeyset">
@@ -97,6 +97,6 @@
keycode="&codeinspector.keycode;"
modifiers="shift"
keytext="&codeinspector.keytext;"
- command="Tools:CodeInspector"/>
+ command="Tools:JITInspector"/>
</keyset>
</overlay>
View
@@ -1,17 +1,17 @@
<?xml version="1.0"?>
<RDF xmlns="http://www.w3.org/1999/02/22-rdf-syntax-ns#" xmlns:em="http://www.mozilla.org/2004/em-rdf#">
<Description about="urn:mozilla:install-manifest">
- <em:id>devtools-CodeInspector@mozilla.com</em:id>
- <em:name>Code Inspector</em:name>
+ <em:id>bhackett@mozilla.com</em:id>
+ <em:name>JIT Inspector</em:name>
<em:version>0.1.0.0</em:version>
- <em:description>Tool to track code executed and other metrics (coverage, hotness, cyclomatic complexity and more)</em:description>
- <em:creator>Cedric Vivier (Mozilla DevTools)</em:creator>
- <em:homepageURL>https://wiki.mozilla.org/DevTools/Features/CodeInspector</em:homepageURL>
+ <em:description>Tool to track metrics for amount and optimization quality of JIT code executed</em:description>
+ <em:creator>Brian Hackett (Mozilla)</em:creator>
+ <em:homepageURL></em:homepageURL>
<em:type>2</em:type><!-- type=extension -->
<em:targetApplication>
<Description><!-- Firefox -->
<em:id>{ec8030f7-c20a-464f-9b0e-13a3a9e97384}</em:id>
- <em:minVersion>8.0.*</em:minVersion>
+ <em:minVersion>12.*</em:minVersion>
<em:maxVersion>100.0.*</em:maxVersion>
</Description>
</em:targetApplication>
File renamed without changes.
File renamed without changes.

0 comments on commit 6c782d1

Please sign in to comment.