-
Findit authored
This reverts commit d7c1e86c. Reason for revert: Findit (https://goo.gl/kROfz5) identified CL at revision 635829 as the culprit for failures in the build cycles as shown on: https://findit-for-me.appspot.com/waterfall/culprit?key=ag9zfmZpbmRpdC1mb3ItbWVyRAsSDVdmU3VzcGVjdGVkQ0wiMWNocm9taXVtL2Q3YzFlODZjMzNjNDIzMjJmMWQ0M2QwOTU3NmUyZjNjNDFhOGUyNDEM Sample Failed Build: https://ci.chromium.org/buildbot/chromium.memory/Linux%20ChromiumOS%20MSan%20Tests/11520 Sample Failed Step: browser_tests Original change's description: > Delay querying for printers list until the policy file is parsed > > A list of enterpise printers is not ready immediately after creation of > CupsPrintersManager. It caused a problem for testing procedure since from the > level of autotest_private_api we do not know if the final list is ready or > there are still some ongoing calculations. A new method called > OnEnterprisePrintersInitialized() was added to the observer of > CupsPrintersManager to solve this issue. Also whole code related to > enterprise printers were refactored, large part of functionality was moved > from SyncedPrintersManager to new class EnterprisePrintersProvider. Classes > ExternalPrintersPrefBridge and DeviceExternalPrintersSettingsBridge have been > merged into one class called CalculatorsPoliciesBinder. > > BUG=chromium:816996 > TEST=Tested on nautilus with the use of CL:1451591 > > Change-Id: I50745f18beacab213216e5f367db0d023190e9e4 > Reviewed-on: https://chromium-review.googlesource.com/c/1220547 > Commit-Queue: Piotr Pawliczek <pawliczek@chromium.org> > Reviewed-by: Achuith Bhandarkar <achuith@chromium.org> > Reviewed-by: Maksim Ivanov <emaxx@chromium.org> > Reviewed-by: Sean Kau <skau@chromium.org> > Auto-Submit: Piotr Pawliczek <pawliczek@chromium.org> > Cr-Commit-Position: refs/heads/master@{#635829} No-Presubmit: true No-Tree-Checks: true No-Try: true BUG=chromium:816996 Change-Id: I03bae898e61c3107977268bf33d1582fe6eacb81 Reviewed-on: https://chromium-review.googlesource.com/c/1491134 Cr-Commit-Position: refs/heads/master@{#635900}
8547d0e0