"Failed to locate a binder for interface: dom_distiller::mojom::DistillabilityService" logspam all over the place |
|||||
Issue descriptionI'm seeing this in many tests: [446:446:0913/001215:ERROR:interface_registry.cc(99)] Failed to locate a binder for interface: dom_distiller::mojom::DistillabilityService [446:446:0913/001215:ERROR:interface_registry.cc(99)] Failed to locate a binder for interface: dom_distiller::mojom::DistillabilityService (eg here https://build.chromium.org/p/chromium.fyi/builders/ClangToTLinux%20tester/builds/2066/steps/browser_tests%20on%20Ubuntu-12.04/logs/PluginPowerSaverBrowserTest.PosterTests) ben, if this is a problem, should https://chromium.googlesource.com/chromium/src/+blame/master/services/shell/public/cpp/lib/interface_registry.cc use a CHECK instead of a LOG(ERROR)? Users won't be able to do anything with a LOG(ERROR), and it makes test output confusing. nyquist, the error suggests that distiller is doing something wrong, do you know who should look at that?
,
Sep 13 2016
The renderer is trying to connect to a service that was not created on the browser. The creation on the browser is tied to a web contents, so if there is no web contents, the connection will fail.
,
Sep 13 2016
Could this be the result of the recent announcement on chromium-dev@ and chromium-mojo@ ( https://groups.google.com/a/chromium.org/forum/#!topic/chromium-mojo/waQDNg2m-M0 ) that refers to: https://chromium.googlesource.com/chromium/src/+/master/content/public/app/mojo/content_browser_manifest.json and https://chromium.googlesource.com/chromium/src/+/master/content/public/app/mojo/content_renderer_manifest.json ?
,
Sep 13 2016
Probably not, it looks like this has been going on for longer than that patch has been around.
,
Sep 13 2016
This warning is throughout the WebGL conformance test logs on multiple plaforms, for example: https://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_ng/builds/295581/steps/webgl_conformance_tests%20on%20Intel%20GPU%20on%20Mac%20%28with%20patch%29%20on%20Mac-10.10/logs/stdio Please suppress it. At least change it to a VLOG.
,
Sep 17 2016
This logspam is really annoying. Is the reader mode intended only for Android? If so could the calling code at least be #ifdef'd for Android only?
,
Sep 20 2016
I'm going to wrap the connection code in #ifdef for android. I was mistakenly under the impression that the service was created for both desktop and mobile.
,
Sep 21 2016
,
Sep 23 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/a6b9e17dc1dcac1da44f723445dd4201bc1e0a5f commit a6b9e17dc1dcac1da44f723445dd4201bc1e0a5f Author: mdjones <mdjones@chromium.org> Date: Fri Sep 23 16:24:30 2016 Control distillability with command-line flag BUG= 646384 Review-Url: https://codereview.chromium.org/2361643004 Cr-Commit-Position: refs/heads/master@{#420617} [modify] https://crrev.com/a6b9e17dc1dcac1da44f723445dd4201bc1e0a5f/chrome/browser/chrome_content_browser_client.cc [modify] https://crrev.com/a6b9e17dc1dcac1da44f723445dd4201bc1e0a5f/chrome/browser/dom_distiller/distillable_page_utils_browsertest.cc [modify] https://crrev.com/a6b9e17dc1dcac1da44f723445dd4201bc1e0a5f/chrome/renderer/chrome_content_renderer_client.cc [modify] https://crrev.com/a6b9e17dc1dcac1da44f723445dd4201bc1e0a5f/components/dom_distiller/core/dom_distiller_switches.cc [modify] https://crrev.com/a6b9e17dc1dcac1da44f723445dd4201bc1e0a5f/components/dom_distiller/core/dom_distiller_switches.h
,
Sep 23 2016
|
|||||
►
Sign in to add a comment |
|||||
Comment 1 by nyquist@chromium.org
, Sep 13 2016