Skip to content

Commit

Permalink
[Mono.Android] abstract-ify inconsistent android.webkit API.
Browse files Browse the repository at this point in the history
As part of discussion at dotnet#1078 (comment) ,
we decided to make changes to those inconsistent methods that used to be
virtual and then became abstract, to become abstract even in old API level.

It is done at metadata fixup level.
  • Loading branch information
atsushieno committed Feb 14, 2018
1 parent cb68bc3 commit a6faa34
Showing 1 changed file with 12 additions and 0 deletions.
12 changes: 12 additions & 0 deletions src/Mono.Android/metadata
Original file line number Diff line number Diff line change
@@ -1,4 +1,16 @@
<metadata>

<!-- FIXME: these members used to be `virtual` and then became `abstract`.
They result in API breakage (reported as inter-api-level mismatch).
This is ancient Google's fault. To deal with the situation nicely,
we are marking them just as `abstract` even at the old API Level, so
that app developers don't get trapped due to missing members at run time.
-->
<attr path="/api/package[@name='android.webkit']/class[@name='CookieManager' or @name='WebBackForwardList' or @name='WebHistoryItem' or @name='WebIconDatabase' or @name='WebViewDatabase']/method[@static='false']" name="abstract">true</attr>
<attr path="/api/package[@name='android.webkit']/class[@name='WebSettings']/method[@static='false' and @name!='setTextSize']" name="abstract">true</attr>
<attr path="/api/package[@name='android.webkit']/class[@name='CookieManager' or @name='WebBackForwardList' or @name='WebHistoryItem' or @name='WebIconDatabase' or @name='WebViewDatabase' or @name='WebSettings']" name="abstract">true</attr>
<attr path="/api/package[@name='android.webkit']/class[@name='CookieManager' or @name='WebBackForwardList' or @name='WebHistoryItem' or @name='WebIconDatabase' or @name='WebViewDatabase' or @name='WebSettings']" name="static">false</attr>

<!-- Manifest.permission has its last name part all in lowercase and
regarded as obfuscated, so avoid that by explicitly marking not. -->
<attr path="/api/package[@name='android']/class[@name='Manifest.permission']" name="obfuscated">false</attr>
Expand Down

0 comments on commit a6faa34

Please sign in to comment.