Cleanup - Get rid of unnecessary local variables.

Cleanup - Get rid of unnecessary local variables.

Trivial cleanup patch to get rid of not needed local variables.

TESTS=No tests as there is no behavior change.

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

[vanihegde, 2014-01-06 09:01:37]

Minor change, please take a look. Thanks.

https://codereview.chromium.org/124973002/diff/1/Source/core/editing/ApplySty...
File Source/core/editing/ApplyStyleCommand.cpp (left):

https://codereview.chromium.org/124973002/diff/1/Source/core/editing/ApplySty...
Source/core/editing/ApplyStyleCommand.cpp:399: currentFontSize =
computedFontSize(node);
computedFontSize() is already done at line 394.

[ojan, 2014-01-06 18:55:41]

https://codereview.chromium.org/124973002/diff/1/Source/core/editing/ApplySty...
File Source/core/editing/ApplyStyleCommand.cpp (left):

https://codereview.chromium.org/124973002/diff/1/Source/core/editing/ApplySty...
Source/core/editing/ApplyStyleCommand.cpp:399: currentFontSize =
computedFontSize(node);
On 2014/01/06 09:01:37, vanihegde wrote:
> computedFontSize() is already done at line 394.

Doesn't the line above this invalidate the computedFontSize from before?

https://codereview.chromium.org/124973002/diff/1/Source/core/editing/ApplySty...
File Source/core/editing/ApplyStyleCommand.cpp (right):

https://codereview.chromium.org/124973002/diff/1/Source/core/editing/ApplySty...
Source/core/editing/ApplyStyleCommand.cpp:695: Node* parentNode =
startNode->parentNode();
I'm not sure this code is any cleaner and I'd be surprised if this performed any
differently.

[vanihegde, 2014-01-07 08:30:46]

https://codereview.chromium.org/124973002/diff/1/Source/core/editing/ApplySty...
File Source/core/editing/ApplyStyleCommand.cpp (left):

https://codereview.chromium.org/124973002/diff/1/Source/core/editing/ApplySty...
Source/core/editing/ApplyStyleCommand.cpp:399: currentFontSize =
computedFontSize(node);
On 2014/01/06 18:55:41, ojan wrote:
> On 2014/01/06 09:01:37, vanihegde wrote:
> > computedFontSize() is already done at line 394.
> 
> Doesn't the line above this invalidate the computedFontSize from before?

Yes, it does. Sorry I missed it!

https://codereview.chromium.org/124973002/diff/1/Source/core/editing/ApplySty...
File Source/core/editing/ApplyStyleCommand.cpp (right):

https://codereview.chromium.org/124973002/diff/1/Source/core/editing/ApplySty...
Source/core/editing/ApplyStyleCommand.cpp:695: Node* parentNode =
startNode->parentNode();
On 2014/01/06 18:55:41, ojan wrote:
> I'm not sure this code is any cleaner and I'd be surprised if this performed
any
> differently.

Reverting.

[ojan, 2014-01-07 18:38:54]

lgtm

[commit-bot: I haz the power, 2014-01-07 18:39:10]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/vani.hegde@samsung.com/124973002/60001

[commit-bot: I haz the power, 2014-01-07 21:31:39]

Change committed as 164615