Removed duplicated virtual computePreferredLogicalWidths()

Removed duplicated virtual computePreferredLogicalWidths()

RenderDeprecatedFlexibleBox, RenderMenuList and RenderSlider does not
need this duplicated code from RenderBlock.
Now code will go through the updateFirstLetter() check but first letter
is disabled for RenderDeprecatedFlexibleBox, RenderMenuList.

This patch removes the overridden function.

BUG=263225
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=183174

[h.joshi, 2014-09-29 16:57:44]

h.joshi@samsung.com changed reviewers:
+ eae@chromium.org, pdr@chromium.org

[h.joshi, 2014-10-02 03:38:28]

Pls review

[pdr., 2014-10-02 04:21:46]

The CQ bit was checked by pdr@chromium.org

[pdr., 2014-10-02 04:21:47]

Nice cleanup.

LGTM

https://codereview.chromium.org/602353002/diff/1/Source/core/rendering/Render...
File Source/core/rendering/RenderMenuList.cpp (left):

https://codereview.chromium.org/602353002/diff/1/Source/core/rendering/Render...
Source/core/rendering/RenderMenuList.cpp:366: LayoutUnit toAdd =
borderAndPaddingWidth();
This is actually different but, as far as I can tell, the old code was just
wrong.

[commit-bot: I haz the power, 2014-10-02 04:22:10]

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

[commit-bot: I haz the power, 2014-10-02 04:26:01]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-10-02 04:26:03]

Try jobs failed on following builders:
  linux_chromium_gn_rel on tryserver.blink
(http://build.chromium.org/p/tryserver.blink/builders/linux_chromium_gn_rel/bu...)
  win_blink_rel on tryserver.blink
(http://build.chromium.org/p/tryserver.blink/builders/win_blink_rel/builds/29812)
  mac_gpu on tryserver.chromium.gpu
(http://build.chromium.org/p/tryserver.chromium.gpu/builders/mac_gpu/builds/63354)

[pdr., 2014-10-02 04:37:04]

On 2014/10/02 at 04:26:03, commit-bot wrote:
> Try jobs failed on following builders:
>   linux_chromium_gn_rel on tryserver.blink
(http://build.chromium.org/p/tryserver.blink/builders/linux_chromium_gn_rel/bu...)
>   win_blink_rel on tryserver.blink
(http://build.chromium.org/p/tryserver.blink/builders/win_blink_rel/builds/29812)
>   mac_gpu on tryserver.chromium.gpu
(http://build.chromium.org/p/tryserver.chromium.gpu/builders/mac_gpu/builds/63354)

These failures are unrelated to this change (bot issues reported in #chromium)

[h.joshi, 2014-10-03 01:46:45]

@pdr: Thank you for patch review.

[h.joshi, 2014-10-03 01:46:59]

The CQ bit was checked by h.joshi@samsung.com

[commit-bot: I haz the power, 2014-10-03 01:48:00]

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

[commit-bot: I haz the power, 2014-10-03 02:47:03]

Committed patchset #1 (id:1) as 183174