Remove fallback tick software draw.

Remove fallback tick software draw.

The fallback software draw was needed to allow the scheduler to advance
in cases where the WebView is detached or offscreen. Because it is now
possible for the pending tree to activate without a draw, the fallback
tick is no longer needed in order for VisualStateListener callbacks
to fire.

BUG=431166
Committed: https://crrev.com/9addff46d3bc49f8d04e43eeb3f55e01a4982a00
Cr-Commit-Position: refs/heads/master@{#358156}

[Tobias Sargeant, 2015-11-05 00:09:53]

Patchset #1 (id:1) has been deleted

[Tobias Sargeant, 2015-11-05 17:31:17]

tobiasjs@chromium.org changed reviewers:
+ boliu@chromium.org

[boliu, 2015-11-05 17:48:03]

https://codereview.chromium.org/1428263003/diff/40001/android_webview/browser...
File android_webview/browser/browser_view_renderer.cc (right):

https://codereview.chromium.org/1428263003/diff/40001/android_webview/browser...
android_webview/browser/browser_view_renderer.cc:228: bool
BrowserViewRenderer::CompositeHw() {
can merge this back into OnHardwareDraw now

https://codereview.chromium.org/1428263003/diff/40001/android_webview/javates...
File
android_webview/javatests/src/org/chromium/android_webview/test/VisualStateTest.java
(right):

https://codereview.chromium.org/1428263003/diff/40001/android_webview/javates...
android_webview/javatests/src/org/chromium/android_webview/test/VisualStateTest.java:409:
final AtomicReference<AwTestContainerView> testContainerView =
new way to do thi is ThreadUtils.runOnUiThreadBlockingNoException:
See https://codereview.chromium.org/1407413011/

https://codereview.chromium.org/1428263003/diff/40001/android_webview/javates...
android_webview/javatests/src/org/chromium/android_webview/test/VisualStateTest.java:463:
jsObserver, "jsObserver", null);
call this somethign else to avoid confusing with the JavascriptEventObserver
things above.

[Tobias Sargeant, 2015-11-05 21:44:42]

Patchset #4 (id:80001) has been deleted

[Tobias Sargeant, 2015-11-05 21:44:48]

Patchset #3 (id:60001) has been deleted

[Tobias Sargeant, 2015-11-05 22:30:16]

https://codereview.chromium.org/1428263003/diff/40001/android_webview/browser...
File android_webview/browser/browser_view_renderer.cc (right):

https://codereview.chromium.org/1428263003/diff/40001/android_webview/browser...
android_webview/browser/browser_view_renderer.cc:228: bool
BrowserViewRenderer::CompositeHw() {
On 2015/11/05 17:48:03, boliu wrote:
> can merge this back into OnHardwareDraw now

Done.

https://codereview.chromium.org/1428263003/diff/40001/android_webview/javates...
File
android_webview/javatests/src/org/chromium/android_webview/test/VisualStateTest.java
(right):

https://codereview.chromium.org/1428263003/diff/40001/android_webview/javates...
android_webview/javatests/src/org/chromium/android_webview/test/VisualStateTest.java:409:
final AtomicReference<AwTestContainerView> testContainerView =
On 2015/11/05 17:48:03, boliu wrote:
> new way to do thi is ThreadUtils.runOnUiThreadBlockingNoException:
> See https://codereview.chromium.org/1407413011/

Done.

https://codereview.chromium.org/1428263003/diff/40001/android_webview/javates...
android_webview/javatests/src/org/chromium/android_webview/test/VisualStateTest.java:463:
jsObserver, "jsObserver", null);
On 2015/11/05 17:48:03, boliu wrote:
> call this somethign else to avoid confusing with the JavascriptEventObserver
> things above.

Done.

[boliu, 2015-11-05 22:31:36]

lgtm

[Tobias Sargeant, 2015-11-05 22:32:07]

The CQ bit was checked by tobiasjs@chromium.org

[commit-bot: I haz the power, 2015-11-05 22:35:12]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1428263003/100001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1428263003/100001

[commit-bot: I haz the power, 2015-11-05 22:51:44]

Committed patchset #3 (id:100001)

[commit-bot: I haz the power, 2015-11-05 22:52:44]

Patchset 3 (id:??) landed as
https://crrev.com/9addff46d3bc49f8d04e43eeb3f55e01a4982a00
Cr-Commit-Position: refs/heads/master@{#358156}

[Tobias Sargeant, 2015-11-24 10:14:43]

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

The reason for reverting is: Causes scheduler to stop activating after clearView
is called. PrepareTiles is required to advance the scheduler, so draws are
currently needed.

crbug.com/558743.