Remove BuildInfo#isGreaterThanN()

Remove BuildInfo#isGreaterThanN()

The NMR1 SDK has been published upstream, so BuildInfo#isGreaterThanN is
no longer needed. This patch also removes reflection for
UserManager#isDemoUser() and ShortcutManager#reportShortcutUsed(),
and updates AndroidManifest.xml to remove the conditional include
of launchershortcut.xml and network_security_config.xml.

BUG=707302
Review-Url: https://codereview.chromium.org/2785373004
Cr-Commit-Position: refs/heads/master@{#461787}
Committed: https://chromium.googlesource.com/chromium/src/+/df46c2242e4635dc77fce945bfc0ea0089efee31

[Theresa, 2017-03-31 20:16:12]

The CQ bit was checked by twellington@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-03-31 20:16:52]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[Theresa, 2017-03-31 20:28:07]

twellington@chromium.org changed reviewers:
+ agrieve@chromium.org, tedchoc@chromium.org

[Theresa, 2017-03-31 20:28:08]

tedchoc@ ptal at everything

agrieve@ ptal at BUILD.gn

[Theresa, 2017-03-31 20:28:24]

The CQ bit was checked by twellington@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-03-31 20:28:46]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[agrieve, 2017-03-31 20:43:07]

On 2017/03/31 20:28:08, Theresa wrote:
> tedchoc@ ptal at everything
> 
> agrieve@ ptal at BUILD.gn

lgtm

[commit-bot: I haz the power, 2017-03-31 21:37:42]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-03-31 21:37:43]

Dry run: Try jobs failed on following builders:
  android_arm64_dbg_recipe on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_arm6...)

[Theresa, 2017-03-31 21:53:13]

It looks like monochrome is still building with N so this isn't ready to land
unfortunately.

[Theresa, 2017-03-31 21:55:21]

Description was changed from

==========
Remove BuildInfo#isGreaterThanN()

The NMR1 SDK has been published upstream, so BuildInfo#isGreaterThanN is
no longer needed. This patch also removes reflection for
UserManager#isDemoUser() and ShortcutManager#reportShortcutUsed(),
and updates AndroidManifest.xml to remove the conditional include
of launchershortcut.xml

BUG=707302
==========

to

==========
Remove BuildInfo#isGreaterThanN()

The NMR1 SDK has been published upstream, so BuildInfo#isGreaterThanN is
no longer needed. This patch also removes reflection for
UserManager#isDemoUser() and ShortcutManager#reportShortcutUsed(),
and updates AndroidManifest.xml to remove the conditional include
of launchershortcut.xml and network_security_config.xml.

BUG=707302
==========

[Ted C, 2017-03-31 22:59:48]

lgtm

[mikecase (-- gone --), 2017-04-04 02:05:19]

mikecase@chromium.org changed reviewers:
+ mikecase@chromium.org

[mikecase (-- gone --), 2017-04-04 02:05:19]

trybots got passed compile step. This should be able to land now I think.

[Theresa, 2017-04-04 15:49:40]

The CQ bit was checked by twellington@chromium.org to run a CQ dry run

[Theresa, 2017-04-04 15:49:46]

On 2017/04/04 02:05:19, mikecase wrote:
> trybots got passed compile step. This should be able to land now I think.

Thank you!

[commit-bot: I haz the power, 2017-04-04 15:50:07]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-04-04 16:48:16]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-04-04 16:48:17]

Dry run: This issue passed the CQ dry run.

[Theresa, 2017-04-04 17:41:52]

The CQ bit was checked by twellington@chromium.org

[Theresa, 2017-04-04 17:41:53]

The patchset sent to the CQ was uploaded after l-g-t-m from agrieve@chromium.org
Link to the patchset: https://codereview.chromium.org/2785373004/#ps40001
(title: "Updates for network security config")

[commit-bot: I haz the power, 2017-04-04 17:42:42]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-04-04 18:46:25]

CQ is committing da patch.

Bot data: {"patchset_id": 40001, "attempt_start_ts": 1491327712915230,
"parent_rev": "f44eeb03c89211aa7018c8cc536f9cab3bbe3def", "commit_rev":
"df46c2242e4635dc77fce945bfc0ea0089efee31"}

[commit-bot: I haz the power, 2017-04-04 18:47:12]

Description was changed from

==========
Remove BuildInfo#isGreaterThanN()

The NMR1 SDK has been published upstream, so BuildInfo#isGreaterThanN is
no longer needed. This patch also removes reflection for
UserManager#isDemoUser() and ShortcutManager#reportShortcutUsed(),
and updates AndroidManifest.xml to remove the conditional include
of launchershortcut.xml and network_security_config.xml.

BUG=707302
==========

to

==========
Remove BuildInfo#isGreaterThanN()

The NMR1 SDK has been published upstream, so BuildInfo#isGreaterThanN is
no longer needed. This patch also removes reflection for
UserManager#isDemoUser() and ShortcutManager#reportShortcutUsed(),
and updates AndroidManifest.xml to remove the conditional include
of launchershortcut.xml and network_security_config.xml.

BUG=707302

Review-Url: https://codereview.chromium.org/2785373004
Cr-Commit-Position: refs/heads/master@{#461787}
Committed:
https://chromium.googlesource.com/chromium/src/+/df46c2242e4635dc77fce945bfc0...
==========

[commit-bot: I haz the power, 2017-04-04 18:47:13]

Committed patchset #3 (id:40001) as
https://chromium.googlesource.com/chromium/src/+/df46c2242e4635dc77fce945bfc0...