diff options
| author | mkosiba <mkosiba@chromium.org> | 2014-09-02 10:42:33 -0700 |
|---|---|---|
| committer | Commit bot <commit-bot@chromium.org> | 2014-09-02 17:54:14 +0000 |
| commit | 83ed44690ca93d4d1d1e92b8a69300a03dba1098 (patch) | |
| tree | 534d56877c3a8f508d7a583641ab51053102d4cd /android_webview/test | |
| parent | 5df416cab1a378864933a89cd9f7f81b91788633 (diff) | |
| download | chromium_src-83ed44690ca93d4d1d1e92b8a69300a03dba1098.zip chromium_src-83ed44690ca93d4d1d1e92b8a69300a03dba1098.tar.gz chromium_src-83ed44690ca93d4d1d1e92b8a69300a03dba1098.tar.bz2 | |
[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
Review URL: https://codereview.chromium.org/531583002
Cr-Commit-Position: refs/heads/master@{#292953}
Diffstat (limited to 'android_webview/test')
| -rw-r--r-- | android_webview/test/shell/src/org/chromium/android_webview/test/AwTestContainerView.java | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/android_webview/test/shell/src/org/chromium/android_webview/test/AwTestContainerView.java b/android_webview/test/shell/src/org/chromium/android_webview/test/AwTestContainerView.java index e626c44..8c51639 100644 --- a/android_webview/test/shell/src/org/chromium/android_webview/test/AwTestContainerView.java +++ b/android_webview/test/shell/src/org/chromium/android_webview/test/AwTestContainerView.java @@ -305,7 +305,8 @@ public class AwTestContainerView extends FrameLayout { @Override public void onMeasure(int widthMeasureSpec, int heightMeasureSpec) { mAwContents.onMeasure(widthMeasureSpec, heightMeasureSpec); - super.onMeasure(widthMeasureSpec, heightMeasureSpec); + // This method is intended to be overridden without calling super. Doing so clobbers state + // set by mAwContents.onMeasure. } @Override |