[android_webview] Don't call super.onMeasure in AwTestContainerView

[android_webview] Don't call super.onMeasure in AwTestContainerView

It shouldn't matter much but the onMeasure method should be overridden
without calling super.

BUG=None
TEST=None

Committed: https://crrev.com/83ed44690ca93d4d1d1e92b8a69300a03dba1098
Cr-Commit-Position: refs/heads/master@{#292953}

[mkosiba (inactive), 2014-09-01 15:09:13]

mkosiba@chromium.org changed reviewers:
+ boliu@chromium.org, sievers@chromium.org

[boliu, 2014-09-02 14:56:02]

rs lgtm

[mkosiba (inactive), 2014-09-02 17:04:31]

The CQ bit was checked by mkosiba@chromium.org

[commit-bot: I haz the power, 2014-09-02 17:06:43]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/mkosiba@chromium.org/531583002/1

[commit-bot: I haz the power, 2014-09-02 17:43:44]

Committed patchset #1 (id:1) as 8a343c07122c84791ce9eb8651bcca77e2f862df

[no sievers, 2014-09-02 20:30:34]

WebView.onMeasure() calls super.onMeasure(), why wouldn't this?

It breaks the measurement of the children if you don't.

[no sievers, 2014-09-02 20:31:51]

On 2014/09/02 20:30:34, sievers wrote:
> WebView.onMeasure() calls super.onMeasure(), why wouldn't this?
> 
> It breaks the measurement of the children if you don't.

which also causes the shell to use sw rendering again, because the SurfaceView
is not getting measured.

needs to at least call measureChildren().

[boliu, 2014-09-02 20:42:09]

On 2014/09/02 20:31:51, sievers wrote:
> On 2014/09/02 20:30:34, sievers wrote:
> > WebView.onMeasure() calls super.onMeasure(), why wouldn't this?
> > 
> > It breaks the measurement of the children if you don't.
> 
> which also causes the shell to use sw rendering again, because the SurfaceView
> is not getting measured.
> 
> needs to at least call measureChildren().

Perhaps mAwContents.onMeasure should come *after* super.onMeasure?

[mkosiba (inactive), 2014-09-03 08:50:06]

On 2014/09/02 20:30:34, sievers wrote:
> WebView.onMeasure() calls super.onMeasure(), why wouldn't this?
> 
> It breaks the measurement of the children if you don't.

My bad. I only looked at the glue layer code and forgot we have tons of super
calls in the actual WebView class itself.
https://codereview.chromium.org/536873002/ reverts this but also moves the calls
around so that the order of state modification is correct.

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

Patchset 1 (id:??) landed as
https://crrev.com/83ed44690ca93d4d1d1e92b8a69300a03dba1098
Cr-Commit-Position: refs/heads/master@{#292953}