Ignore lint error RtlHardcoded for AwZoomControls

Ignore lint error RtlHardcoded for AwZoomControls

The following lint error is reported while building
android_webview_apk can be ignored:

AwZoomControls.java:76 Use "`Gravity.END`" instead of "`Gravity.RIGHT`"
to ensure correct behavior in right-to-left locales: RtlHardcoded [warning]
((FrameLayout.LayoutParams) params).gravity = Gravity.RIGHT;

BUG=None.

Committed: https://crrev.com/21b5e19d993ae5e4b0830789966f75138f58f758
Cr-Commit-Position: refs/heads/master@{#320864}

[wajahat, 2015-03-16 14:27:15]

wajahat.s@samsung.com changed reviewers:
+ benm@chromium.org

[wajahat, 2015-03-16 14:27:16]

Pl. review, Thanks!

[wajahat, 2015-03-16 14:29:15]

wajahat.s@samsung.com changed reviewers:
+ boliu@chromium.org
- benm@chromium.org

[boliu, 2015-03-16 20:40:18]

lgtm

https://codereview.chromium.org/1013713003/diff/1/android_webview/java/src/or...
File android_webview/java/src/org/chromium/android_webview/AwZoomControls.java
(right):

https://codereview.chromium.org/1013713003/diff/1/android_webview/java/src/or...
android_webview/java/src/org/chromium/android_webview/AwZoomControls.java:78:
((FrameLayout.LayoutParams) params).gravity = Gravity.RIGHT;
I think this originally came from here:

https://android.googlesource.com/platform/frameworks/base/+/ics-mr1/core/java...

It's been like this since day one, and at this point maintaining backward
compatible behavior probably makes more sense than changing this to Gravity.END
(as lint suggests) to address RTL issues.

[wajahat, 2015-03-17 04:16:16]

On 2015/03/16 20:40:18, boliu wrote:
> lgtm
> 
>
https://codereview.chromium.org/1013713003/diff/1/android_webview/java/src/or...
> File android_webview/java/src/org/chromium/android_webview/AwZoomControls.java
> (right):
> 
>
https://codereview.chromium.org/1013713003/diff/1/android_webview/java/src/or...
> android_webview/java/src/org/chromium/android_webview/AwZoomControls.java:78:
> ((FrameLayout.LayoutParams) params).gravity = Gravity.RIGHT;
> I think this originally came from here:
> 
>
https://android.googlesource.com/platform/frameworks/base/+/ics-mr1/core/java...
> 
> It's been like this since day one, and at this point maintaining backward
> compatible behavior probably makes more sense than changing this to
Gravity.END
> (as lint suggests) to address RTL issues.

Yes I too think so, Thanks for review!

[wajahat, 2015-03-17 04:16:21]

The CQ bit was checked by wajahat.s@samsung.com

[commit-bot: I haz the power, 2015-03-17 04:16:31]

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

[commit-bot: I haz the power, 2015-03-17 04:49:16]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2015-03-17 04:49:42]

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