Prevent inherited style from propagating into a plugin placeholder.

Prevent inherited style from propagating into a plugin placeholder.

Without setting 'all: initial', it's possible for text colours and such to be
inherited by the placeholder, which is unexpected.

TEST=fast/plugins/plugin-placeholder-inherit.html
BUG=364716
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=184878

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

[jbroman, 2014-10-30 15:27:54]

jbroman@chromium.org changed reviewers:
+ esprehn@chromium.org

[jbroman, 2014-10-30 15:27:55]

This should be pretty trivial.

Ref test uses "color" (most likely thing to be visible in practice),
"text-decoration" (not inherited in the strictest sense but propagates
similarly) and "text-transform" (also inherited, and less likely than "color" to
be explicitly set in the placeholder CSS in the future -- thus more likely to
detect regressions going forward).

[jbroman, 2014-11-04 23:16:49]

jbroman@chromium.org changed reviewers:
+ mkwst@chromium.org

[jbroman, 2014-11-04 23:17:13]

+mkwst in case esprehn is too busy to do reviews

bump (5 days with no response)

[Mike West, 2014-11-05 07:40:02]

LGTM.

[esprehn, 2014-11-05 16:45:02]

lgtm

[jbroman, 2014-11-05 17:46:23]

The CQ bit was checked by jbroman@chromium.org

[commit-bot: I haz the power, 2014-11-05 17:47:15]

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

[commit-bot: I haz the power, 2014-11-05 17:49:14]

Committed patchset #3 (id:40001) as 184878

[dcheng, 2014-11-05 19:21:42]

A revert of this CL (patchset #3 id:40001) has been created in
https://codereview.chromium.org/706623003/ by dcheng@chromium.org.

The reason for reverting is: Causing layout test
fast/plugins/plugin-placeholder-inherit.html to fail..

[jbroman, 2014-11-11 21:15:05]

On 2014/11/05 19:21:42, dcheng wrote:
> A revert of this CL (patchset #3 id:40001) has been created in
> https://codereview.chromium.org/706623003/ by mailto:dcheng@chromium.org.
> 
> The reason for reverting is: Causing layout test
> fast/plugins/plugin-placeholder-inherit.html to fail..

One of these failures was a bad merge, and the other is a bug, which I've filed:
  https://code.google.com/p/chromium/issues/detail?id=432257

I'm going to leave this CL here until that issue has been addressed.

[jbroman, 2014-11-13 14:49:48]

On 2014/11/11 21:15:05, jbroman wrote:
> On 2014/11/05 19:21:42, dcheng wrote:
> > A revert of this CL (patchset #3 id:40001) has been created in
> > https://codereview.chromium.org/706623003/ by mailto:dcheng@chromium.org.
> > 
> > The reason for reverting is: Causing layout test
> > fast/plugins/plugin-placeholder-inherit.html to fail..
> 
> One of these failures was a bad merge, and the other is a bug, which I've
filed:
>   https://code.google.com/p/chromium/issues/detail?id=432257
> 
> I'm going to leave this CL here until that issue has been addressed.

This bug has been fixed; attempting to reland this CL.

[jbroman, 2014-11-13 14:49:53]

The CQ bit was checked by jbroman@chromium.org

[commit-bot: I haz the power, 2014-11-13 14:50:14]

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

[commit-bot: I haz the power, 2014-11-13 15:43:46]

Committed patchset #4 (id:60001) as 185298