Closed Bug 1116199 Opened 7 years ago Closed 7 years ago

Console eval broken with servers before 37 (missing evaluateJSAsync)

Categories

(DevTools :: Console, defect)

defect
Not set
normal

Tracking

(firefox37+ fixed)

RESOLVED FIXED
Firefox 37
Tracking Status
firefox37 + fixed

People

(Reporter: jryans, Assigned: bgrins)

References

Details

(Keywords: regression)

Attachments

(1 file, 1 obsolete file)

In bug 1088861, the console changed to use a new "evaluateJSAsync" actor method.

However servers before 37 don't have this method, so console eval is now broken for them.

Traits or similar feature detection should be used client-side to detect if this server method is available before use.
Brian, would you be able to look at this since you worked on bug 1088861?
Flags: needinfo?(bgrinstead)
[Tracking Requested - why for this release]: DevTools console broken when connecting to any previous version, e.g. all B2G simulators, old Fennec, etc.
(In reply to J. Ryan Stinnett [:jryans] from comment #1)
> Brian, would you be able to look at this since you worked on bug 1088861?

Yes, I'll fix this
Assignee: nobody → bgrinstead
Status: NEW → ASSIGNED
Flags: needinfo?(bgrinstead)
What do you think?  I tested with a simulator and all seemed well.
Attachment #8544116 - Flags: review?(jryans)
Comment on attachment 8544116 [details] [diff] [review]
evaluateJSAsync-backwards-compat.patch

Review of attachment 8544116 [details] [diff] [review]:
-----------------------------------------------------------------

Looks good, thanks!

::: toolkit/devtools/webconsole/client.js
@@ +136,5 @@
>     * See evaluateJS for parameter and response information.
>     */
>    evaluateJSAsync: function(aString, aOnResponse, aOptions = {})
>    {
> +    // Pre-37 clients need to do sync evaluation.

Nit: Pre-37 servers don't support async evaluation
Attachment #8544116 - Flags: review?(jryans) → review+
Updated comment
Attachment #8544116 - Attachment is obsolete: true
Attachment #8544118 - Flags: review+
https://hg.mozilla.org/mozilla-central/rev/0dd788e4b214
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Whiteboard: [fixed-in-fx-team]
Target Milestone: --- → Firefox 37
Product: Firefox → DevTools
You need to log in before you can comment on or make changes to this bug.