Closed Bug 1155658 Opened 9 years ago Closed 9 years ago

"illegal argument" error should be "invalid argument"

Categories

(Remote Protocol :: Marionette, defect)

Product:
Remote Protocol
Component:
Marionette
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(firefox40 fixed)

Status:
RESOLVED FIXED
Milestone:
mozilla40
Tracking Flags:
Tracking Status
firefox40 --- fixed

People

(Reporter: ato, Assigned: ato)

References

(Blocks 1 open bug)

Details

(Keywords: pi-marionette-server)

Attachments

(1 file, 1 obsolete file)

MozReview Request: Bug 1155658: Rename "illegal argument" error to "invalid argument"
39 bytes, text/x-review-board-request
chmanchester
: review+
Details 
The string code for the IllegalArgumentError in the Marionette server is wrong; it should ready "invalid argument" to be in compliance with the specification.
Assignee: nobody → ato
Blocks: webdriver
Status: NEW → ASSIGNED
Keywords: ateam-marionette-server
Attached file MozReview Request: bz://1155658/ato (obsolete) — 
/r/7211 - Bug 1155658: Rename "illegal argument" error to "invalid argument"

Pull down this commit:

hg pull -r 4a9701c793d99d4b0447d93811419709e954d074 https://reviewboard-hg.mozilla.org/gecko/
Attachment #8594069 - Flags: review?(dburns)
It's fine to rename the string associated with it because it's not yet in use in any of the clients.
Comment on attachment 8594069 [details]
MozReview Request: bz://1155658/ato

/r/7211 - Bug 1155658: Rename "illegal argument" error to "invalid argument"

Pull down this commit:

hg pull -r 4a9701c793d99d4b0447d93811419709e954d074 https://reviewboard-hg.mozilla.org/gecko/
Attachment #8594069 - Flags: review?(cmanchester)
Comment on attachment 8594069 [details]
MozReview Request: bz://1155658/ato

https://reviewboard.mozilla.org/r/7209/#review6137

Ship It!
Attachment #8594069 - Flags: review?(cmanchester) → review+
Attachment #8594069 - Flags: review?(dburns)
https://hg.mozilla.org/mozilla-central/rev/8c56a1c8e458
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
status-firefox40: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla40

        Attachment #8594069 -
        Attachment is obsolete: true

        Attachment #8620076 -
        Flags: review+

    
  
Product: Testing → Remote Protocol

          You need to log in
          before you can comment on or make changes to this bug.
        






  

    
  







  

    

      
Attachment

      

      
      
      
      
    

    

      

        

          

            
General

            

              Creator: 



            

            
Created: 

            
Updated: 

            
Size: