Removing --renderer-assert-test command-line flag

Removing --renderer-assert-test command-line flag

BUG=447957
Committed: https://crrev.com/ab82b75dbda97e7caf83464c1c1665e72554c5b1
Cr-Commit-Position: refs/heads/master@{#311221}

[anujsharma, 2015-01-12 07:00:48]

anujk.sharma@samsung.com changed reviewers:
+ avi@chromium.org

[anujsharma, 2015-01-12 07:00:49]

PTAL

[Avi (use Gerrit), 2015-01-12 16:17:45]

On 2015/01/12 07:00:49, anujsharma wrote:
> PTAL

This looks like a debugging flag. We might need to keep these in. What was this
one used for, and is it still needed?

[anujsharma, 2015-01-13 03:09:22]

On 2015/01/12 16:17:45, Avi wrote:
> On 2015/01/12 07:00:49, anujsharma wrote:
> > PTAL
> 
> This looks like a debugging flag. We might need to keep these in. What was
this
> one used for, and is it still needed?

[anujsharma, 2015-01-13 03:19:02]

On 2015/01/13 03:09:22, anujsharma wrote:
> On 2015/01/12 16:17:45, Avi wrote:
> > On 2015/01/12 07:00:49, anujsharma wrote:
> > > PTAL
> > 
> > This looks like a debugging flag. We might need to keep these in. What was
> this
> > one used for, and is it still needed?

Thanks Avi for your time.
As per the spreadsheet shared by peter, it is slated for removal and Mr. darin
is the current owner.
I have cced Mr. darin in the bug to notify him. If you want, i will add him to
this review as well.

Thanks!!

[Avi (use Gerrit), 2015-01-13 03:40:09]

On 2015/01/13 03:19:02, anujsharma wrote:
> On 2015/01/13 03:09:22, anujsharma wrote:
> > On 2015/01/12 16:17:45, Avi wrote:
> > > On 2015/01/12 07:00:49, anujsharma wrote:
> > > > PTAL
> > > 
> > > This looks like a debugging flag. We might need to keep these in. What was
> > this
> > > one used for, and is it still needed?
> 
> Thanks Avi for your time.
> As per the spreadsheet shared by peter, it is slated for removal and Mr. darin
> is the current owner.
> I have cced Mr. darin in the bug to notify him. If you want, i will add him to
> this review as well.
> 
> Thanks!!

Thank you for pointing me at the spreadsheet entry. You are right, let's kill
it. LGTM

Also, the spreadsheet points to http://crbug.com/414272. Are you going to kill
that flag too?

[anujsharma, 2015-01-13 03:59:35]

On 2015/01/13 03:40:09, Avi wrote:
> On 2015/01/13 03:19:02, anujsharma wrote:
> > On 2015/01/13 03:09:22, anujsharma wrote:
> > > On 2015/01/12 16:17:45, Avi wrote:
> > > > On 2015/01/12 07:00:49, anujsharma wrote:
> > > > > PTAL
> > > > 
> > > > This looks like a debugging flag. We might need to keep these in. What
was
> > > this
> > > > one used for, and is it still needed?
> > 
> > Thanks Avi for your time.
> > As per the spreadsheet shared by peter, it is slated for removal and Mr.
darin
> > is the current owner.
> > I have cced Mr. darin in the bug to notify him. If you want, i will add him
to
> > this review as well.
> > 
> > Thanks!!
> 
> Thank you for pointing me at the spreadsheet entry. You are right, let's kill
> it. LGTM
> 
> Also, the spreadsheet points to http://crbug.com/414272. Are you going to kill
> that flag too?

Thanks Avi for lgtm. :)

Currently no one is working on that bug. I will check with bug owner and will
try to kill that flag too.

[anujsharma, 2015-01-13 03:59:43]

The CQ bit was checked by anujk.sharma@samsung.com

[commit-bot: I haz the power, 2015-01-13 04:00:51]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/845223002/20001

[commit-bot: I haz the power, 2015-01-13 05:43:22]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-01-13 05:45:02]

Patchset 2 (id:??) landed as
https://crrev.com/ab82b75dbda97e7caf83464c1c1665e72554c5b1
Cr-Commit-Position: refs/heads/master@{#311221}