Avoid unnecessary width recalculation after bug fixed

Avoid unnecessary width recalculation after bug fixed

The issue http://crbug.com/237700 which was causing an infinite loop is a duplicate of http://crbug.com/238405 which has already been fixed, making unnecessary the workaround.

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=270249

[ltilve, 2014-05-05 21:21:51]

There was a TODO requesting to eliminate some code when the bug
http://crbug.com/237700 had been fixed.

As the changes for the issue marked as a duplicate of that bug have been already
committed and the bug has been marked as resolved, I'm removing the requested
code and the corresponding TODO.

[stevenjb, 2014-05-05 21:24:10]

This looks OK to me. +mukai@ who may have looked at this code more recently, but
as long as it's well tested, lgtm

[Jun Mukai, 2014-05-05 21:30:16]

The change itself looks good but I don't have much knowledge around this part.
Since dimich fixed the bug, better to defer to him.

[ltilve, 2014-05-13 18:52:51]

On 2014/05/05 21:30:16, Jun Mukai wrote:
> The change itself looks good but I don't have much knowledge around this part.
> Since dimich fixed the bug, better to defer to him.

Thank you both for your comments.

I haven't been able to have dimich taking a look to it. Maybe I can keep trying
to ping him, or should I ask for review to someone else? Or as far as the unit
tests and the trybots seem to be passing fine, I guess that perhaps we could CQ
it already.

wdyt?

[Dmitry Titov, 2014-05-13 19:42:11]

I'm terribly sorry, it got off my radar. Looking...


On Tue, May 13, 2014 at 11:52 AM, <ltilve@igalia.com> wrote:

> On 2014/05/05 21:30:16, Jun Mukai wrote:
>
>> The change itself looks good but I don't have much knowledge around this
>> part.
>> Since dimich fixed the bug, better to defer to him.
>>
>
> Thank you both for your comments.
>
> I haven't been able to have dimich taking a look to it. Maybe I can keep
> trying
> to ping him, or should I ask for review to someone else? Or as far as the
> unit
> tests and the trybots seem to be passing fine, I guess that perhaps we
> could CQ
> it already.
>
> wdyt?
>
> https://codereview.chromium.org/269013006/
>

To unsubscribe from this group and stop receiving emails from it, send an email
to chromium-reviews+unsubscribe@chromium.org.

[Dmitry Titov, 2014-05-13 19:43:50]

LGTM

[ltilve, 2014-05-13 21:43:06]

The CQ bit was checked by ltilve@igalia.com

[commit-bot: I haz the power, 2014-05-13 21:45:24]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/ltilve@igalia.com/269013006/1

[commit-bot: I haz the power, 2014-05-13 23:45:49]

Change committed as 270249