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

Issue 707552 link

Starred by 9 users

Issue metadata

Status: Archived
Owner:
Closed: Apr 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 1
Type: Bug-Regression



Sign in to add a comment

Mysterious "Office Editing for Docs, Sheets and Slides" extension icon, but nothing listed in Extensions

Project Member Reported by w...@chromium.org, Apr 2 2017

Issue description

Chrome Version: 59.0.3054.0 dev
OS: ChromeOS Minnie

What steps will reproduce the problem?
(1) Sign-in.
(2) Open the toolbar action overflow area (the three-dot menu).
(3) Find the icon for Office Editing for Docs, Sheets and Slides, and hover over it.
(4) Open chrome://extensions.

What is the expected result?

Expect that it is possible to un-install the Office Editing for Docs, Sheets and Slides extension, or that it is not there.

Expect that hovering over it provides a description. Instead, the tooltip looks like an i18n placeholder tag, starting __MSG__.

Expect that it is listed under chrome://extensions. It is not listed, however; it's not clear how the icon is being generated.

Visiting the Offline Editing.. extension in the Chrome Web Store, the Web Store site indicates that it is not installed, and offers to install it.

This affects both of the accounts I have signed-in on this device.
 

Comment 1 by w...@chromium.org, Apr 2 2017

Screenshot 2017-04-01 at 16.07.27.png
5.4 KB View Download
That's very strange.  First step is to check out what the settings are for that extension in preferences.

If you have a version of chrome after 59.0.3047.0, you should be able to go to chrome://prefs-internals and ctrl+f for the extension id.  Otherwise, we'll have to go a more roundabout way.

If you can find the prefs entry for it, we can see a bit about what it is, and, hopefully, where it came from.  If you can scrub it for any PII and paste the json blob here, that'd be helpful.  (Feel free to ping/email for more details on this.)

Comment 3 by w...@chromium.org, Apr 6 2017

Labels: -Type-Bug -Pri-3 Pri-1 Type-Bug-Regression
Owner: kuscher@chromium.org
Status: Assigned (was: Untriaged)
As discussed off-bug, AIs here are:
1. Remove the action icon patch from the component extension version of Office Editing.
2. Fix the localisation of the icon tool-tip / extension name.
3. Follow-up on visibility of changes to the component extension to the CrOS release managers.

kuscher: Can you re-assign as appropriate? Is Ha the right owned for this?
Owner: hlo@chromium.org
Yes, assigning to hlo@ for now to to the revert
Cc: abodenha@chromium.org zelidrag@chromium.org sonnysasaka@chromium.org rookrishna@chromium.org dhadd...@chromium.org sdantul...@chromium.org
 Issue 705111  has been merged into this issue.
Labels: ReleaseBlock-Stable

Comment 7 by gkihumba@google.com, Apr 18 2017

Any updates?

Comment 8 by hlo@chromium.org, Apr 19 2017

We have fix ready. Currently blocked by quickoffice waterfall issue: https://bugs.chromium.org/p/chromium/issues/detail?id=713033. Waiting for help from chrome trooper
Project Member

Comment 9 by bugdroid1@chromium.org, Apr 21 2017

The following revision refers to this bug:
  https://quickoffice-internal.googlesource.com/html-office/+/bac3d1b83119f7a7ed82825f46a5a3919ae24a1c

commit bac3d1b83119f7a7ed82825f46a5a3919ae24a1c
Author: Atul Moglewar <atul.moglewar@synerzip.com>
Date: Fri Apr 21 03:15:22 2017

Comment 10 by hlo@chromium.org, Apr 24 2017

Raised a cl to get the new release into ChromeOS: https://chromereviews.googleplex.com/579307013/

Comment 11 by hlo@chromium.org, Apr 24 2017

Status: Fixed (was: Assigned)
CL has now been submitted into ChromeOS trunk. Leaving it to someone else to cherry pick if needed into a different ChromeOS branch. 
Labels: Merge-TBD
[Auto-generated comment by a script] We noticed that this issue is targeted for M-59; it appears the fix may have landed after branch point, meaning a merge might be required. Please confirm if a merge is required here - if so add Merge-Request-59 label, otherwise remove Merge-TBD label. Thanks.
Labels: Merge-Request-59
Owner: r...@chromium.org
Status: Assigned (was: Fixed)
rkc@ can someone on your team do the merge to 59?

Comment 14 by r...@chromium.org, Apr 25 2017

Cc: gkihumba@chromium.org
This was tricky to merge. Apparently for a DEPS change, we can't drover into the branch, it needs to be a new CL.

Grace helped me with the merge: https://chromereviews.googleplex.com/586727013/
Thank you Grace!

Project Member

Comment 15 by sheriffbot@chromium.org, Apr 25 2017

Labels: -Merge-Request-59 Hotlist-Merge-Approved Merge-Approved-59
Your change meets the bar and is auto-approved for M59. Please go ahead and merge the CL to branch 3071 manually. Please contact milestone owner if you have questions.
Owners: amineer@(Android), cmasso@(iOS), gkihumba@(ChromeOS), Abdul Syed@(Desktop)

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

Comment 16 by r...@chromium.org, Apr 25 2017

Status: Fixed (was: Assigned)

Comment 17 by r...@chromium.org, Apr 25 2017

Labels: Merge-Merged
Project Member

Comment 18 by sheriffbot@chromium.org, May 1 2017

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 19 by sheriffbot@chromium.org, May 5 2017

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, Jun 19 2017

Labels: -Merge-TBD

Comment 21 by dchan@chromium.org, Jan 22 2018

Status: Archived (was: Fixed)

Sign in to add a comment