Add skia to list of valid trackers

Add skia to list of valid trackers.

Created to fix bug https://code.google.com/p/skia/issues/detail?id=1453 : bug=xxx points to Chromium instead of Skia.

Change staged here:
https://skia-codereview-staging.appspot.com/10001/

[iannucci, 2013-07-31 18:26:05]

lgtm

https://codereview.appspot.com/12159043/diff/1/codereview/views.py
File codereview/views.py (right):

https://codereview.appspot.com/12159043/diff/1/codereview/views.py#newcode2178
codereview/views.py:2178: valid_trackers = ('chromium', 'chromium-os',
'chrome-os-partner', 'gyp', 'skia',
80 columns? Or is the diff view being silly?

[rmistry, 2013-08-01 11:12:28]

https://codereview.appspot.com/12159043/diff/1/codereview/views.py
File codereview/views.py (right):

https://codereview.appspot.com/12159043/diff/1/codereview/views.py#newcode2178
codereview/views.py:2178: valid_trackers = ('chromium', 'chromium-os',
'chrome-os-partner', 'gyp', 'skia',
On 2013/07/31 18:26:05, iannucci wrote:
> 80 columns? Or is the diff view being silly?

Fixed. I had my column width set to 100 and thus did not notice.

[rmistry, 2013-08-01 11:21:24]

On 2013/08/01 11:12:28, rmistry wrote:
> https://codereview.appspot.com/12159043/diff/1/codereview/views.py
> File codereview/views.py (right):
> 
> https://codereview.appspot.com/12159043/diff/1/codereview/views.py#newcode2178
> codereview/views.py:2178: valid_trackers = ('chromium', 'chromium-os',
> 'chrome-os-partner', 'gyp', 'skia',
> On 2013/07/31 18:26:05, iannucci wrote:
> > 80 columns? Or is the diff view being silly?
> 
> Fixed. I had my column width set to 100 and thus did not notice.

Submitted and pushed to production. Made it the default version since it was the
only CL submitting after the version that was live.

[iannucci, 2013-08-01 18:03:30]

On 2013/08/01 11:21:24, rmistry wrote:
> On 2013/08/01 11:12:28, rmistry wrote:
> > https://codereview.appspot.com/12159043/diff/1/codereview/views.py
> > File codereview/views.py (right):
> > 
> >
https://codereview.appspot.com/12159043/diff/1/codereview/views.py#newcode2178
> > codereview/views.py:2178: valid_trackers = ('chromium', 'chromium-os',
> > 'chrome-os-partner', 'gyp', 'skia',
> > On 2013/07/31 18:26:05, iannucci wrote:
> > > 80 columns? Or is the diff view being silly?
> > 
> > Fixed. I had my column width set to 100 and thus did not notice.
> 
> Submitted and pushed to production. Made it the default version since it was
the
> only CL submitting after the version that was live.

Cool :)

In The Future, we should (all, myself included) get in the habit of sending a
note to chrome-infrastructure-team@ when we roll rietveld on chromiumcodereview.
I know we have been inconsistent about this in the past, but we should converge
towards that. It would also be useful to provide any special rollback
instructions in case of datastore changes, etc :)

[rmistry, 2013-08-01 18:08:36]

On 2013/08/01 18:03:30, iannucci wrote:
> On 2013/08/01 11:21:24, rmistry wrote:
> > On 2013/08/01 11:12:28, rmistry wrote:
> > > https://codereview.appspot.com/12159043/diff/1/codereview/views.py
> > > File codereview/views.py (right):
> > > 
> > >
> https://codereview.appspot.com/12159043/diff/1/codereview/views.py#newcode2178
> > > codereview/views.py:2178: valid_trackers = ('chromium', 'chromium-os',
> > > 'chrome-os-partner', 'gyp', 'skia',
> > > On 2013/07/31 18:26:05, iannucci wrote:
> > > > 80 columns? Or is the diff view being silly?
> > > 
> > > Fixed. I had my column width set to 100 and thus did not notice.
> > 
> > Submitted and pushed to production. Made it the default version since it was
> the
> > only CL submitting after the version that was live.
> 
> Cool :)
> 
> In The Future, we should (all, myself included) get in the habit of sending a
> note to chrome-infrastructure-team@ when we roll rietveld on
chromiumcodereview.
> I know we have been inconsistent about this in the past, but we should
converge
> towards that. It would also be useful to provide any special rollback
> instructions in case of datastore changes, etc :)

SGTM. I joined chrome-infrastructure-team@ a few mins ago.