ASSERTION FAILED: extractedStyle in WebCore::ApplyStyleCommand::removeInlineStyleFromElement

ASSERTION FAILED: extractedStyle in WebCore::ApplyStyleCommand::removeInlineStyleFromElement

The last extractedStyle parameter of removeInlineStyleFromElement() is not mandatory and it's set
to default 0. This way we have to check its existence before the usage.

Backported from my WebKit patch: http://trac.webkit.org/changeset/154439

R=hayato@chromium.org
BUG=350414
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=169948

[hayato, 2014-03-10 04:59:08]

LGTM

https://codereview.chromium.org/190993002/diff/1/LayoutTests/editing/execComm...
File LayoutTests/editing/execCommand/extracted-style-assert.html (right):

https://codereview.chromium.org/190993002/diff/1/LayoutTests/editing/execComm...
LayoutTests/editing/execCommand/extracted-style-assert.html:16:
document.body.innerHTML = "PASS. WebKit didn't crash.";
Nit: Could you change 'WebKit' to 'Blink'? :)

[reni, 2014-03-10 08:14:56]

Renamed 'WebKit' to 'Blink' in the test and in the expected file.

[hayato, 2014-03-10 08:19:31]

On 2014/03/10 08:14:56, renata.hodovan wrote:
> Renamed 'WebKit' to 'Blink' in the test and in the expected file.

Thank you. LGTM.

[reni, 2014-03-10 08:25:17]

The CQ bit was checked by rhodovan.u-szeged@partner.samsung.com

[commit-bot: I haz the power, 2014-03-10 08:25:29]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/rhodovan.u-szeged@partner.samsung.com/...

[commit-bot: I haz the power, 2014-03-10 08:34:48]

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

[commit-bot: I haz the power, 2014-03-10 08:34:48]

Try jobs failed on following builders: mac_blink_rel

[reni, 2014-03-10 08:49:05]

Typo fix (removing an extra ';').

[reni, 2014-03-10 13:02:48]

On 2014/03/10 08:49:05, renata.hodovan wrote:
> Typo fix (removing an extra ';').

I'm sorry, I messed up the git update :/

[reni, 2014-03-17 10:02:14]

On 2014/03/10 13:02:48, renata.hodovan wrote:
> On 2014/03/10 08:49:05, renata.hodovan wrote:
> > Typo fix (removing an extra ';').
> 
> I'm sorry, I messed up the git update :/

Can I land it this way?

[hayato, 2014-03-25 04:45:59]

On 2014/03/17 10:02:14, renata.hodovan wrote:
> On 2014/03/10 13:02:48, renata.hodovan wrote:
> > On 2014/03/10 08:49:05, renata.hodovan wrote:
> > > Typo fix (removing an extra ';').
> > 
> > I'm sorry, I messed up the git update :/
> 
> Can I land it this way?

LGTM

[reni, 2014-03-25 14:06:46]

The CQ bit was checked by rhodovan.u-szeged@partner.samsung.com

[commit-bot: I haz the power, 2014-03-25 14:06:56]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/rhodovan.u-szeged@partner.samsung.com/...

[commit-bot: I haz the power, 2014-03-25 15:10:23]

Change committed as 169948