Project: chromium Issues People Development process History Sign in
New issue
Advanced search Search tips
Starred by 1 user
Status: Verified
Closed: Feb 2017
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 3
Type: Bug

Sign in to add a comment
piex_loader.js is noisy in chromium browser_tests
Project Member Reported by, Feb 6 2017 Back to list
In chromium, the Promise created in the PiexLoader constructor (piex_loader.js) is rejected with "PiexLoader is not enabled for chromium build."

The Promise is not caught, so it always outputs this as a console error. It shows up in browser_tests logs as:

[25655:25655:0206/123627.428061:INFO:CONSOLE(0)] "Uncaught (in promise) PiexLoader is not enabled for chromium build.", source: chrome-extension://pmfjbimdmchhbnneeidfognadeopoehp/_generated_background_page.html (0)

Is this warning important or can it be removed? Is it a bug when tests encounter this warning (should calls to PiexLoader be predicated on whether it's available in the build?)?
Status: Started
Let's remove this.
Labels: OS-Chrome
Project Member Comment 3 by, Feb 9 2017
The following revision refers to this bug:

commit 6423d6b3eea01831f79139b2d9408dc565a45077
Author: hirono <>
Date: Thu Feb 09 05:04:28 2017

Remove rejected promise of PiexLoader which is not caught.

Previously if PiexLoader was not enabled for the build,
PiexLoader.naclPromise_ was rejected and the rejected promise
automatically appeared in console.log.

The CL fulfills a boolean value to naclPromise_ instead of
fullfills/rejects it to reducy noisy uncaught promise logs.

Bug= 689264 
Test=Manually run Files app with disabling Piex and checked the console.

Cr-Commit-Position: refs/heads/master@{#449204}


Status: Fixed
Labels: VerifyIn-59
Labels: code-change
Status: Verified
Sign in to add a comment