aw: Disable blink scheduler on the Android WebView.

aw: Disable blink scheduler on the Android WebView.

We need to hook up input handling through the blink scheduler before we can enable it
for WebView, so disable it for now on Android WebView until we fix this.

BUG=431598
Committed: https://crrev.com/a229c4cacb8932e8fe574791333325500ac6339e
Cr-Commit-Position: refs/heads/master@{#303611}

[rmcilroy, 2014-11-10 14:03:28]

rmcilroy@chromium.org changed reviewers:
+ mkosiba@chromium.org

[rmcilroy, 2014-11-10 14:03:28]

Martin: Could you please take a look, thanks.

[rmcilroy, 2014-11-10 14:04:14]

Patchset #1 (id:1) has been deleted

[mkosiba (inactive), 2014-11-10 21:54:49]

lgtm

[rmcilroy, 2014-11-10 21:56:49]

The CQ bit was checked by rmcilroy@chromium.org

[commit-bot: I haz the power, 2014-11-10 21:57:30]

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

[boliu, 2014-11-10 21:58:57]

Wait.. On the bug, Sami said it's only for benchmarking. If so, can we just sent
this flag for whatever benchmark we are running (we have benchmarks? that's
awesome) insteading of changing production code?

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

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

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

Try jobs failed on following builders:
  mac_gpu on tryserver.chromium.gpu
(http://build.chromium.org/p/tryserver.chromium.gpu/builders/mac_gpu/builds/79644)
  win_gpu on tryserver.chromium.gpu
(http://build.chromium.org/p/tryserver.chromium.gpu/builders/win_gpu/builds/84572)
  win8_chromium_rel on tryserver.chromium.win
(http://build.chromium.org/p/tryserver.chromium.win/builders/win8_chromium_rel...)
  win_chromium_compile_dbg on tryserver.chromium.win
(http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_comp...)
  win_chromium_x64_rel on tryserver.chromium.win
(http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_x64_...)

[rmcilroy, 2014-11-10 22:06:11]

On 2014/11/10 21:58:57, boliu wrote:
> Wait.. On the bug, Sami said it's only for benchmarking. If so, can we just
sent
> this flag for whatever benchmark we are running (we have benchmarks? that's
> awesome) insteading of changing production code?

I think what Sami meant was the flag is only meant to be for benchmarking,
however we can't enable the scheduler for webview yet until we fix the input
event handling on webview to go through the scheduler (see details at the start
of the bug). Adding this flag here is temporary until that's fixed to allow us
to enable the scheduler by default for Chrome while we work through the input
events on webview.

[boliu, 2014-11-10 22:13:13]

On 2014/11/10 22:06:11, rmcilroy wrote:
> On 2014/11/10 21:58:57, boliu wrote:
> > Wait.. On the bug, Sami said it's only for benchmarking. If so, can we just
> sent
> > this flag for whatever benchmark we are running (we have benchmarks? that's
> > awesome) insteading of changing production code?
> 
> I think what Sami meant was the flag is only meant to be for benchmarking,
> however we can't enable the scheduler for webview yet until we fix the input
> event handling on webview to go through the scheduler (see details at the
start
> of the bug). Adding this flag here is temporary until that's fixed to allow us
> to enable the scheduler by default for Chrome while we work through the input
> events on webview.

Should fix that bug asap then. Can we agree to fix it say by m41 branch?

Cries a bit that webview isn't treated the same as chrome

[rmcilroy, 2014-11-11 07:42:28]

> Should fix that bug asap then. Can we agree to fix it say by m41 branch?

Yes, we will make sure to fix it by m41.

[rmcilroy, 2014-11-11 07:43:05]

The CQ bit was checked by rmcilroy@chromium.org

[commit-bot: I haz the power, 2014-11-11 07:44:24]

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

[commit-bot: I haz the power, 2014-11-11 08:28:43]

Committed patchset #2 (id:40001)

[commit-bot: I haz the power, 2014-11-11 08:29:19]

Patchset 2 (id:??) landed as
https://crrev.com/a229c4cacb8932e8fe574791333325500ac6339e
Cr-Commit-Position: refs/heads/master@{#303611}