New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 664130 link

Starred by 1 user

Issue metadata

Status: Verified
Owner:
Last visit > 30 days ago
Closed: Nov 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 1
Type: Bug



Sign in to add a comment

Public session demo app doesn't run for the second time.

Project Member Reported by oka@chromium.org, Nov 10 2016

Issue description

https://feedback.corp.google.com/#/Report/15656300129

Chrome OS: 8872.45.0 (Official Build) dev-channel kevin test
Chrome: 55.0.2883.42
Device: Kevin

What steps will reproduce the problem?
1. Login to public session mode using us retailer account. Setup instruction:
   https://drive.google.com/file/d/0B0f93Zklq9B1T3N3Z0psNzYyeHc/view
2. Observe demo app runs automatically.
3. right click demo app icon -> close.
4. Click demo app in the menu and observe

What is the expected result?
Demo app launches.

What happens instead?
Demo app doesn't launch. Nothing happens.
 

Comment 1 by oka@chromium.org, Nov 10 2016

Components: Platform>Apps>Demo

Comment 2 by oka@chromium.org, Nov 10 2016

Components: Platform>NaCl

Comment 3 by oka@chromium.org, Nov 10 2016

Cc: derat@chromium.org katierh@chromium.org llozano@chromium.org
Let me paste log snippets.

getrlimit(RLIMIT_NOFILE) failed
[1532:1532:1110/043334:ERROR:customization_document.cc(630)] Customization manifest is missing on server: <URL: 10>
getrlimit(RLIMIT_NOFILE) failed
getrlimit(RLIMIT_NOFILE) failed
[1:1:1110/213348:ERROR:KeyboardEventManager.cpp(344)] Not implemented reached in static bool blink<IPv6: 6>KeyboardEventManager<IPv6: 13>urrentCapsLockState()
[1532:1532:1110/213359:ERROR:object_proxy.cc(583)] Failed to call method: org.chromium.UpdateEngineInterface.SetChannel: object_path= /org/chromium/UpdateEngine: org.freedesktop.DBus.Error.Failed: update_engine/org.chromium.UpdateEngine.Error.Failed:Cannot set target channel explicitly when channel policy/settings is not delegated
[1532:1532:1110/213359:ERROR:update_engine_client.cc(343)] Failed to request setting channel
[13720:13721:1110/213402:ERROR:interface_registry.cc(99)] Capability spec prevented service service:content_browser from binding interface: mojom<IPv6: 6>ResourceUsageReporter
[1532:1532:1110/213402:ERROR:auto_enrollment_controller.cc(149)] Auto-enrollment client is already running.
[1532:1532:1110/213402:ERROR:easy_unlock_get_keys_operation.cc(76)] Easy unlock failed to get key data, code=32
getrlimit(RLIMIT_NOFILE) failed
[1532:1532:1110/213402:ERROR:easy_unlock_get_keys_operation.cc(76)] Easy unlock failed to get key data, code=32
[1532:1532:1110/213402:ERROR:object_proxy.cc(583)] Failed to call method: org.freedesktop.DBus.Properties.Set: object_path= /org/bluez/hci0: org.bluez.Error.Failed: Not Powered
(*) [1532:1532:1110/213402:ERROR:component_loader.cc(154)] Failed to parse extension manifest.
[1532:1532:1110/213403:ERROR:easy_unlock_get_keys_operation.cc(76)] Easy unlock failed to get key data, code=32
[1532:1532:1110/213403:ERROR:multi_user_window_manager_stub.cc(56)] Not implemented reached in virtual void <URL: 5>
getrlimit(RLIMIT_NOFILE) failed

getrlimit(RLIMIT_NOFILE) failed
(*) [1532:1715:1110/213409:ERROR:service_manager.cc(306)] Error: invalid Name: CD67277C109884C48C737BB834F4F7CD
(*) [1,4134371328:12:34:09.080969] Native Client module will be loaded at base address 0x0000000000000000
[1532:1532:1110/213409:ERROR:easy_unlock_get_keys_operation.cc(76)] Easy unlock failed to get key data, code=32
getrlimit(RLIMIT_NOFILE) failed
getrlimit(RLIMIT_NOFILE) failed
getrlimit(RLIMIT_NOFILE) failed
getrlimit(RLIMIT_NOFILE) failed
getrlimit(RLIMIT_NOFILE) failed
[1532:1532:1110/213415:ERROR:easy_unlock_get_keys_operation.cc(76)] Easy unlock failed to get key data, code=32
(*) [1532:1532:1110/213415:ERROR:device_local_account_management_policy_provider.cc(737)] oeeolgpknofhhcaifmiidhjoeobnonle is not of a supported type. Extension type: 5
(*) [1532:1532:1110/213415:ERROR:device_local_account_management_policy_provider.cc(820)] oeeolgpknofhhcaifmiidhjoeobnonle: app manifest entry is allowed only for hosted_app or platform_app extension type. Current extension type: 5
(*) [1532:1532:1110/213415:ERROR:external_cache.cc(109)] ExternalCache extension at /var/cache/device_local_account_extensions/7075626C69636163636F756E742E4075732D72657461696C6D6F64652E636F6D/oeeolgpknofhhcaifmiidhjoeobnonle-3.0-de9bf3fbb423a626a844f2eb49ca402fe4f723e9f983e0e3974e2dac190af08b.crx failed to install, deleting it.
getrlimit(RLIMIT_NOFILE) failed
getrlimit(RLIMIT_NOFILE) failed
getrlimit(RLIMIT_NOFILE) failed
getrlimit(RLIMIT_NOFILE) failed
getrlimit(RLIMIT_NOFILE) failed
[1532:1715:1110/213419:ERROR:service_manager.cc(306)] Error: invalid Name: 6EF39A64D3324939BFF45851A15DB353
getrlimit(RLIMIT_NOFILE) failed
[1,4134371328:12:34:19.751707] Native Client module will be loaded at base address 0x0000000000000000
getrlimit(RLIMIT_NOFILE) failed
[1532:1715:1110/213420:ERROR:service_manager.cc(306)] Error: invalid Name: 9B7D5A84FD910888ECFC3D3921B77BA9
[1,4134371328:12:34:20.057813] Native Client module will be loaded at base address 0x0000000000000000


The lines with (*) look suspicious to me. Namely the following:
(*) [1532:1532:1110/213402:ERROR:component_loader.cc(154)] Failed to parse extension manifest.

(*) [1532:1715:1110/213409:ERROR:service_manager.cc(306)] Error: invalid Name: CD67277C109884C48C737BB834F4F7CD
(*) [1,4134371328:12:34:09.080969] Native Client module will be loaded at base address 0x0000000000000000

(*) [1532:1532:1110/213415:ERROR:device_local_account_management_policy_provider.cc(737)] oeeolgpknofhhcaifmiidhjoeobnonle is not of a supported type. Extension type: 5
(*) [1532:1532:1110/213415:ERROR:device_local_account_management_policy_provider.cc(820)] oeeolgpknofhhcaifmiidhjoeobnonle: app manifest entry is allowed only for hosted_app or platform_app extension type. Current extension type: 5
(*) [1532:1532:1110/213415:ERROR:external_cache.cc(109)] ExternalCache extension at /var/cache/device_local_account_extensions/7075626C69636163636F756E742E4075732D72657461696C6D6F64652E636F6D/oeeolgpknofhhcaifmiidhjoeobnonle-3.0-de9bf3fbb423a626a844f2eb49ca402fe4f723e9f983e0e3974e2dac190af08b.crx failed to install, deleting it.

Let me have wider audience.

Comment 4 by derat@chromium.org, Nov 10 2016

Cc: tnagel@chromium.org isandrk@chromium.org
I don't know anything about this, but just to save others the trouble of looking it up, extension type 5 is TYPE_LEGACY_PACKAGED_APP.

The error message is coming from chrome/browser/chromeos/extensions/device_local_account_management_policy_provider.cc:

bool IsSafeForPublicSession(const extensions::Extension* extension) {
  bool safe = true;
  if (!extension->is_extension() &&
      !extension->is_hosted_app() &&
      !extension->is_platform_app() &&
      !extension->is_shared_module() &&
      !extension->is_theme()) {
    LOG(ERROR) << extension->id()
               << " is not of a supported type. Extension type: "
               << extension->GetType();
    safe = false;
  }
  ...

Cc-ing people who last touched that code.

Comment 5 by tnagel@chromium.org, Nov 10 2016

Cc: -tnagel@chromium.org
Removing myself since PS auto-whitelisting is now Ivan's domain.
Were you able to run this app in the past?

Comment 7 by oka@chromium.org, Nov 11 2016

It reproduced on Linux (ToT).

Comment 8 by oka@chromium.org, Nov 11 2016

Owner: oka@chromium.org
Status: Started (was: Available)
This app is of type LEGACY_PACKAGED_APP which was never supported on Public Sessions (to the best of my knowledge). Currently we support Regular Extensions, Hosted Apps, and Platform Apps. Can you update the app into a Platform App?
What does it mean that it reproduced on Linux (ToT)?

How old is this app? Is it a newly written app? When is the last time it worked under Public Session (I doubt this one)? Sorry I'm lacking a lot of information.

This error is the most worrisome to me (with a badly parsed manifest, everything can happen later):
[1532:1532:1110/213402:ERROR:component_loader.cc(154)] Failed to parse extension manifest.

Comment 11 by oka@chromium.org, Nov 17 2016

Components: -Platform>NaCl -Infra>Client>NaCl
Sorry. It turned out to be purely JS side issue. Removing irrelevent people.

Comment 12 by oka@chromium.org, Nov 17 2016

Cc: -llozano@chromium.org -derat@chromium.org -katierh@chromium.org -isandrk@chromium.org

Comment 13 by oka@chromium.org, Nov 17 2016

I sent out the fix for review.
https://chrome-internal-review.googlesource.com/#/c/305118/
Project Member

Comment 14 by bugdroid1@chromium.org, Nov 25 2016

The following revision refers to this bug:
  https://chrome-internal.googlesource.com/chromeos/chromeos-assets/+/a0939f26da00495a532b41d576f49837b3652d17

commit a0939f26da00495a532b41d576f49837b3652d17
Author: Keigo Oka <oka@google.com>
Date: Thu Nov 17 09:01:23 2016

Comment 15 by oka@chromium.org, Nov 25 2016

Labels: Merge-Request-56

Comment 16 by dimu@chromium.org, Nov 26 2016

Labels: -Merge-Request-56 Merge-Approved-56 Hotlist-Merge-Approved
Your change meets the bar and is auto-approved for M56 (branch: 2924)
Project Member

Comment 17 by bugdroid1@chromium.org, Nov 29 2016

Labels: merge-merged-release-R56-9000.B
The following revision refers to this bug:
  https://chrome-internal.googlesource.com/chromeos/chromeos-assets/+/b2936e1df8c298b6684e0ffe96fad523285c06d2

commit b2936e1df8c298b6684e0ffe96fad523285c06d2
Author: Keigo Oka <oka@google.com>
Date: Thu Nov 17 09:01:23 2016

Comment 18 by oka@chromium.org, Nov 29 2016

Status: Fixed (was: Started)
Project Member

Comment 19 by sheriffbot@chromium.org, Nov 30 2016

This issue has been approved for a merge. Please merge the fix to any appropriate branches as soon as possible!

If all merges have been completed, please remove any remaining Merge-Approved labels from this issue.

Thanks for your time! To disable nags, add the Disable-Nags label.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Project Member

Comment 20 by sheriffbot@chromium.org, Dec 4 2016

This issue has been approved for a merge. Please merge the fix to any appropriate branches as soon as possible!

If all merges have been completed, please remove any remaining Merge-Approved labels from this issue.

Thanks for your time! To disable nags, add the Disable-Nags label.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Status: Verified (was: Fixed)
Verified with:
M	ChromeOS	Chrome	ARC	Type	Channel
56	9000.21.0	56.0.2924.19	3550082	release	dev
Device:  Peppy MP

Sign in to add a comment