Pass setScriptableObjectProperty calls on to Pepper again

Pass setScriptableObjectProperty calls on to Pepper again

We previously removed the ability for Pepper plugins to intercept
properties set on the plugin element as we thought that this
private feature wasn't used anywhere. It turns out that this feature
is still used by the deprecated gTalk plugin. This patch adds back
the ability for the plugin to intercept the call.

Note that we are missing the ability (in V8) to detect whether
the plugin has actually intercepted the call the avoid passing
the SetProperty request onto the DOM element. But that turns
out to be inconsequential for the gTalk plugin.

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

[raymes, 2014-10-09 17:00:52]

raymes@chromium.org changed reviewers:
+ jochen@chromium.org, xiyuan@chromium.org

[raymes, 2014-10-09 17:00:53]

Hi Jochen,

Could you please take a look at this urgently as it's a beta blocker for
http://code.google.com/p/chromium/issues/detail?id=414842

Feel free to ping me if you want to discuss more.

Thanks!
Raymes

[raymes, 2014-10-09 17:34:03]

raymes@chromium.org changed reviewers:
+ abarth@chromium.org

[raymes, 2014-10-09 17:41:35]

raymes@chromium.org changed reviewers:
+ haraken@chromium.org
- xiyuan@chromium.org

[raymes, 2014-10-09 17:41:35]

+haraken

PTAL if you have a chance, I'm trying to get this fix in urgently as it's for a
beta blocker.

Thanks!

[jochen (gone - plz use gerrit), 2014-10-09 18:02:35]

lgtm

[raymes, 2014-10-09 18:12:44]

The CQ bit was checked by raymes@chromium.org

[commit-bot: I haz the power, 2014-10-09 18:13:19]

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

[raymes, 2014-10-09 18:13:40]

Thank you!!

On Thu Oct 09 2014 at 11:02:37 AM <jochen@chromium.org> wrote:

> lgtm
>
>
>
> https://codereview.chromium.org/640073005/
>

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

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

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

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

Try jobs failed on following builders:
  win_blink_rel on tryserver.blink
(http://build.chromium.org/p/tryserver.blink/builders/win_blink_rel/builds/31012)

[raymes, 2014-10-09 20:48:06]

The CQ bit was checked by raymes@chromium.org

[commit-bot: I haz the power, 2014-10-09 20:48:39]

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

[commit-bot: I haz the power, 2014-10-09 21:38:47]

Committed patchset #3 (id:40001) as 183492