Fix PreRunValidation calls in two of UIThreadExtensionFunction's subclasses.

Fix PreRunValidation calls in two of UIThreadExtensionFunction's subclasses.

From what I can tell, skipping base (UITEF) class's PreRunValidation isn't
a good idea. Make those subclasses include calling the base function.

The two subclasses are:
AppCurrentWindowInternalExtensionFunction and
FileSystemProviderInternalFunction

BUG=None
Test=None, in theory we should correctly send errors to extension APIs when
browser is shutting down.

Review-Url: https://codereview.chromium.org/2617023003
Cr-Commit-Position: refs/heads/master@{#441839}
Committed: https://chromium.googlesource.com/chromium/src/+/83e30d0ef0da7e8559ffec15b1228735b9d816df

[lazyboy, 2017-01-05 22:21:01]

lazyboy@chromium.org changed reviewers:
+ rdevlin.cronin@chromium.org

[Devlin, 2017-01-05 23:34:01]

lgtm.

When is there going to be a C++ feature for ensuring super class's methods are
called? ;)

[lazyboy, 2017-01-05 23:52:07]

:)

[lazyboy, 2017-01-05 23:52:11]

The CQ bit was checked by lazyboy@chromium.org

[commit-bot: I haz the power, 2017-01-05 23:52:40]

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-01-06 02:38:02]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1483660331097430, "parent_rev":
"a7eac57463496e91447f07ffa2831c10d9264279", "commit_rev":
"83e30d0ef0da7e8559ffec15b1228735b9d816df"}

[commit-bot: I haz the power, 2017-01-06 02:38:32]

Description was changed from

==========
Fix PreRunValidation calls in two of UIThreadExtensionFunction's subclasses.

From what I can tell, skipping base (UITEF) class's PreRunValidation isn't
a good idea. Make those subclasses include calling the base function.

The two subclasses are:
AppCurrentWindowInternalExtensionFunction and
FileSystemProviderInternalFunction

BUG=None
Test=None, in theory we should correctly send errors to extension APIs when
browser is shutting down.
==========

to

==========
Fix PreRunValidation calls in two of UIThreadExtensionFunction's subclasses.

From what I can tell, skipping base (UITEF) class's PreRunValidation isn't
a good idea. Make those subclasses include calling the base function.

The two subclasses are:
AppCurrentWindowInternalExtensionFunction and
FileSystemProviderInternalFunction

BUG=None
Test=None, in theory we should correctly send errors to extension APIs when
browser is shutting down.

Review-Url: https://codereview.chromium.org/2617023003
Cr-Commit-Position: refs/heads/master@{#441839}
Committed:
https://chromium.googlesource.com/chromium/src/+/83e30d0ef0da7e8559ffec15b122...
==========

[commit-bot: I haz the power, 2017-01-06 02:38:33]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/83e30d0ef0da7e8559ffec15b122...