Last Comment Bug 731972 - GCLI selection field should be clear when more options exist, and allow you to get them
: GCLI selection field should be clear when more options exist, and allow you t...
Status: RESOLVED FIXED
:
Product: Firefox
Classification: Client Software
Component: Developer Tools: Console (show other bugs)
: unspecified
: All All
: P2 normal (vote)
: Firefox 16
Assigned To: Joe Walker [:jwalker] (needinfo me or ping on irc)
:
: Brian Grinstead [:bgrins]
Mentors:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2012-03-01 05:26 PST by Joe Walker [:jwalker] (needinfo me or ping on irc)
Modified: 2012-08-08 15:16 PDT (History)
1 user (show)
See Also:
Crash Signature:
(edit)
QA Whiteboard:
Iteration: ---
Points: ---
Has Regression Range: ---
Has STR: ---


Attachments
v1 (14.41 KB, patch)
2012-07-05 12:57 PDT, Joe Walker [:jwalker] (needinfo me or ping on irc)
dcamp: review+
Details | Diff | Splinter Review

Description Joe Walker [:jwalker] (needinfo me or ping on irc) 2012-03-01 05:26:11 PST

    
Comment 1 Joe Walker [:jwalker] (needinfo me or ping on irc) 2012-04-29 11:25:12 PDT
GCLI Triage.
Comment 2 Joe Walker [:jwalker] (needinfo me or ping on irc) 2012-07-05 12:57:02 PDT
Created attachment 639437 [details] [diff] [review]
v1

Visual testing required before landing, which I'll do.
Comment 3 Dave Camp (:dcamp) 2012-07-05 13:09:20 PDT
Comment on attachment 639437 [details] [diff] [review]
v1

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

Carrying over r+
Comment 4 Joe Walker [:jwalker] (needinfo me or ping on irc) 2012-07-05 13:13:49 PDT
https://tbpl.mozilla.org/?tree=Try&rev=66c667117058
Comment 5 Joe Walker [:jwalker] (needinfo me or ping on irc) 2012-07-05 13:50:03 PDT
https://tbpl.mozilla.org/?tree=Fx-Team&rev=f8a4f9332cc8
Comment 6 Tim Taubert [:ttaubert] 2012-07-06 04:01:07 PDT
https://hg.mozilla.org/mozilla-central/rev/a7011201f228
Comment 7 Dave Camp (:dcamp) 2012-08-08 15:16:27 PDT
Comment on attachment 639437 [details] [diff] [review]
v1

Marking the patch with the r+ I said I was giving it awhile back.

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