Changeset 53121 in webkit


Ignore:
Timestamp:
Jan 12, 2010 1:41:36 AM (14 years ago)
Author:
abarth@webkit.org
Message:

2010-01-12 Adam Barth <abarth@webkit.org>

Reviewed by Eric Seidel.

webkit-patch shouldn't waste my time for with a component prompt
https://bugs.webkit.org/show_bug.cgi?id=33521

The component field isn't very useful for bugs created with
webkit-patch because they're likely to be resolved quickly. Instead of
always prompting for a component, we should just default to the "New
Bugs" component. If the bug stays around for more than five minutes,
we can assign it a proper component.

  • Scripts/webkitpy/bugzilla.py:
Location:
trunk/WebKitTools
Files:
2 edited

Legend:

Unmodified
Added
Removed
  • trunk/WebKitTools/ChangeLog

    r53120 r53121  
     12010-01-12  Adam Barth  <abarth@webkit.org>
     2
     3        Reviewed by Eric Seidel.
     4
     5        webkit-patch shouldn't waste my time for with a component prompt
     6        https://bugs.webkit.org/show_bug.cgi?id=33521
     7
     8        The component field isn't very useful for bugs created with
     9        webkit-patch because they're likely to be resolved quickly.  Instead of
     10        always prompting for a component, we should just default to the "New
     11        Bugs" component.  If the bug stays around for more than five minutes,
     12        we can assign it a proper component.
     13
     14        * Scripts/webkitpy/bugzilla.py:
     15
    1162010-01-12  Adam Barth  <abarth@webkit.org>
    217
  • trunk/WebKitTools/Scripts/webkitpy/bugzilla.py

    r52870 r53121  
    400400        component_items = self.browser.find_control('component').items
    401401        component_names = map(lambda item: item.name, component_items)
    402         if not component or component not in component_names:
     402        if not component:
     403            component = "New Bugs"
     404        if component not in component_names:
    403405            component = self.prompt_for_component(component_names)
    404406        self.browser['component'] = [component]
Note: See TracChangeset for help on using the changeset viewer.