Bug 1016330 - (Part 1) Make CSS Coverage work with B2G; r=paul

https://bugzilla.mozilla.org/show_bug.cgi?id=1016330#c3
> prettifyCSS needs to be used client side or moved at the toolkit level.
Done

https://bugzilla.mozilla.org/show_bug.cgi?id=1016330#c4
> Move the call to makeRemote to DeveloperToolbar.jsm
Not done. See https://bugzilla.mozilla.org/show_bug.cgi?id=1016330#c6

https://bugzilla.mozilla.org/show_bug.cgi?id=1016330#c5
> We need to test first there's a cssusage actor.
> A toolbox might be debugging gecko < 32.
Done

https://bugzilla.mozilla.org/show_bug.cgi?id=1016330#c2
> Kill reference to this._tabActor.browser. This might create issues
> with sub documents ("load" is sent to browser even for inner documents.
> It's not the case with tabActor.window).
Future work.
This commit is contained in:
Joe Walker
2014-06-08 22:33:36 +01:00
parent 363009873c
commit 9b07866b90
7 changed files with 93 additions and 73 deletions

View File

@@ -32,6 +32,9 @@ exports.items = [
description: l10n.lookup("csscoverageStartDesc"),
exec: function*(args, context) {
let usage = yield csscoverage.getUsage(context.environment.target);
if (usage == null) {
throw new Error(l10n.lookup("csscoverageNoRemoteError"));
}
yield usage.start(context.environment.chromeWindow,
context.environment.target);
}
@@ -43,6 +46,9 @@ exports.items = [
exec: function*(args, context) {
let target = context.environment.target;
let usage = yield csscoverage.getUsage(target);
if (usage == null) {
throw new Error(l10n.lookup("csscoverageNoRemoteError"));
}
yield usage.stop();
yield gDevTools.showToolbox(target, "styleeditor");
}
@@ -54,6 +60,9 @@ exports.items = [
exec: function*(args, context) {
let target = context.environment.target;
let usage = yield csscoverage.getUsage(target);
if (usage == null) {
throw new Error(l10n.lookup("csscoverageNoRemoteError"));
}
yield usage.oneshot();
yield gDevTools.showToolbox(target, "styleeditor");
}
@@ -65,6 +74,9 @@ exports.items = [
exec: function*(args, context) {
let target = context.environment.target;
let usage = yield csscoverage.getUsage(target);
if (usage == null) {
throw new Error(l10n.lookup("csscoverageNoRemoteError"));
}
let running = yield usage.toggle();
if (running) {
@@ -81,6 +93,10 @@ exports.items = [
description: l10n.lookup("csscoverageReportDesc"),
exec: function*(args, context) {
let usage = yield csscoverage.getUsage(context.environment.target);
if (usage == null) {
throw new Error(l10n.lookup("csscoverageNoRemoteError"));
}
return {
isTypedData: true,
type: "csscoveragePageReport",