Modify UsbDeviceFilter::MatchesAny to return true for empty filters

Modify UsbDeviceFilter::MatchesAny to return true for empty filters

Since some code that calls UsbDeviceFilter::MatchesAny already checks if
filters is empty, it would be nice to move this check inside the function.
Also it will make writing test code a little easier.

BUG=639928
Committed: https://crrev.com/d893fe00f1bf2562d754475b85adb23b09f2559f
Cr-Commit-Position: refs/heads/master@{#413785}

[juncai, 2016-08-22 19:48:19]

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

[commit-bot: I haz the power, 2016-08-22 19:49:18]

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

[juncai, 2016-08-22 20:34:25]

juncai@chromium.org changed reviewers:
+ reillyg@chromium.org

[juncai, 2016-08-22 20:34:26]

Please take a look.

[Reilly Grant (use Gerrit), 2016-08-22 20:49:01]

lgtm

[commit-bot: I haz the power, 2016-08-22 21:22:04]

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

[commit-bot: I haz the power, 2016-08-22 21:22:05]

Dry run: Try jobs failed on following builders:
  linux_chromium_chromeos_ozone_rel_ng on master.tryserver.chromium.linux
(JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[juncai, 2016-08-22 23:33:43]

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

[commit-bot: I haz the power, 2016-08-22 23:34:09]

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

[juncai, 2016-08-23 00:41:05]

To fix unit test failure of
PrinterDetectorAppSearchEnabledTest.PrinterProvider_DifferentUsbProductId, need
to modify UsbPrinterManifestData::SupportsDevice() to deal with empty
|filters_|.

Please take a look again.

[commit-bot: I haz the power, 2016-08-23 01:24:36]

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

[commit-bot: I haz the power, 2016-08-23 01:24:37]

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

[Reilly Grant (use Gerrit), 2016-08-23 17:25:57]

lgtm

[juncai, 2016-08-23 17:31:44]

The CQ bit was checked by juncai@chromium.org

[commit-bot: I haz the power, 2016-08-23 17:32:14]

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, 2016-08-23 18:08:54]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-08-23 18:12:18]

Description was changed from

==========
Modify UsbDeviceFilter::MatchesAny to return true for empty filters

Since some code that calls UsbDeviceFilter::MatchesAny already checks if
filters is empty, it would be nice to move this check inside the function.
Also it will make writing test code a little easier.

BUG=639928
==========

to

==========
Modify UsbDeviceFilter::MatchesAny to return true for empty filters

Since some code that calls UsbDeviceFilter::MatchesAny already checks if
filters is empty, it would be nice to move this check inside the function.
Also it will make writing test code a little easier.

BUG=639928

Committed: https://crrev.com/d893fe00f1bf2562d754475b85adb23b09f2559f
Cr-Commit-Position: refs/heads/master@{#413785}
==========

[commit-bot: I haz the power, 2016-08-23 18:12:19]

Patchset 2 (id:??) landed as
https://crrev.com/d893fe00f1bf2562d754475b85adb23b09f2559f
Cr-Commit-Position: refs/heads/master@{#413785}