My favorites | Sign in
Project Home Downloads Wiki Issues Code Search
New issue   Search
for
  Advanced search   Search tips   Subscriptions
Issue 51105: About:conflicts page
16 people starred this issue and may be notified of changes. Back to list
Status:  Fixed
Owner:  finnur@chromium.org
Closed:  Jan 2011
Cc:  hbridge@google.com, eroman@chromium.org, huanr@chromium.org, mattm@chromium.org, cl...@chromium.org, finnur@chromium.org
M-9

Restricted
  • Only users with EditIssue permission may comment.


Sign in to add a comment
 
Project Member Reported by cl...@chromium.org, Aug 3, 2010
Create page at about:dlls that does the following:

1. Enumerates the dlls in the current process
2. For LSPs, enumerates the entries in the winsock catalog
3. Checks if any of these dlls or LSPs fits a blacklist of known 3rd party crashers
4. If they fit the blacklist, serves known ways to resolve the issue (i.e. "Try disabling Speedbit Video Accelerator")
5. Collects data on the occurrence of different dlls and LSPs.

Goal: decrease 3rd party crashes by improving user access to relevant information.
Aug 4, 2010
#3 huanr@chromium.org
(No comment was entered for this change.)
Status: Available
Labels: -Area-Undefined -Mstone-X -Feature-Unknown -Dev-TestPlan-No -Dev-LegalReview-No -Dev-PrivacyReview-No -Dev-SecurityReview-No -Dev-UIReview-No Area-Internals Mstone-7
Aug 5, 2010
#4 thakis@chromium.org
(No comment was entered for this change.)
Labels: OS-Windows
Aug 18, 2010
#5 cl...@chromium.org
I mocked up a draft of the page, attached. Some notes:

1. Dlls with warnings would be those that fit a hard-coded blacklist
2. Help tips would be hard-coded for dlls on the blacklist
3. Software fields would be populated with dll metadata

Please provide feedback.
about dlls.png
50.0 KB   View   Download
Aug 26, 2010
#6 lafo...@chromium.org
Seeing no checkins I'm punting this to m8.
Labels: -Mstone-7 Mstone-8
Aug 26, 2010
#8 cl...@chromium.org
Attached is a list of dlls known to conflict with Chrome. It also lists the software title, bug number, and help content for each "Learn more" link.
dllcrashers.png
52.8 KB   View   Download
Sep 2, 2010
#9 cl...@chromium.org
(No comment was entered for this change.)
Blockedon: 53474
Sep 28, 2010
#10 cl...@chromium.org
(No comment was entered for this change.)
Blockedon: -53474 57239
Sep 30, 2010
#12 cl...@chromium.org
CC'ing Finnur who's interested in implementing this.
Cc: fin...@chromium.org
Oct 1, 2010
#13 finnur@chromium.org
(No comment was entered for this change.)
Status: Assigned
Owner: fin...@chromium.org
Oct 8, 2010
#14 finnur@chromium.org
This is blocked on a bug marked for M9. Makes sense to also target this for M9, which I think was the intent.
Labels: -Mstone-8 Mstone-9
Oct 21, 2010
#15 finnur@chromium.org
This feature has been renamed about:compatibility.
Summary: About:compatibility page
Oct 21, 2010
#16 lafo...@chromium.org
Moving P2s with an owner to Mstone 10, bring back to M9 if you think it's important and you don't have higher priority work.
Labels: -Mstone-9 Mstone-10
Oct 26, 2010
#17 finnur@chromium.org
(No comment was entered for this change.)
Labels: -Mstone-10 Mstone-9
Nov 3, 2010
#18 finnur@chromium.org
(No comment was entered for this change.)
Blockedon: 61723
Nov 5, 2010
#19 lafo...@chromium.org
Given our current velocity, we need to punt 500 bugs from m9.  Moving p2 bugs, that are not started and have an owner, to the next milestone.  If this issue absolutely needs to be fixed in the current milestone please move it back, however, at this time the focus should be on p1 bugs.
Labels: -mstone-9 Mstone-10
Nov 8, 2010
#20 bugdroid1@gmail.com
The following revision refers to this bug:
    http://src.chromium.org/viewvc/chrome?view=rev&revision=65366

------------------------------------------------------------------------
r65366 | finnur@chromium.org | Mon Nov 08 03:17:05 PST 2010

Changed paths:
 M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/app/theme/theme_resources.grd?r1=65366&r2=65365&pathrev=65366
 M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/browser.cc?r1=65366&r2=65365&pathrev=65366
 M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/browser.h?r1=65366&r2=65365&pathrev=65366
 M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/about_flags.cc?r1=65366&r2=65365&pathrev=65366
 M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/views/toolbar_view.cc?r1=65366&r2=65365&pathrev=65366
 M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/wrench_menu_model.cc?r1=65366&r2=65365&pathrev=65366
 M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/views/toolbar_view.h?r1=65366&r2=65365&pathrev=65366
 M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/common/chrome_switches.cc?r1=65366&r2=65365&pathrev=65366
 A http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/dom_ui/conflicts_ui.cc?r1=65366&r2=65365&pathrev=65366
 M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/common/chrome_switches.h?r1=65366&r2=65365&pathrev=65366
 M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/common/notification_type.h?r1=65366&r2=65365&pathrev=65366
 A http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/dom_ui/conflicts_ui.h?r1=65366&r2=65365&pathrev=65366
 M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/chrome_tests.gypi?r1=65366&r2=65365&pathrev=65366
 M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/common/url_constants.cc?r1=65366&r2=65365&pathrev=65366
 M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/browser_resources.grd?r1=65366&r2=65365&pathrev=65366
 M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/common/url_constants.h?r1=65366&r2=65365&pathrev=65366
 M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/app/google_chrome_strings.grd?r1=65366&r2=65365&pathrev=65366
 A http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/resources/about_conflicts.html?r1=65366&r2=65365&pathrev=65366
 A http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/enumerate_modules_model_unittest_win.cc?r1=65366&r2=65365&pathrev=65366
 A http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/enumerate_modules_model_win.cc?r1=65366&r2=65365&pathrev=65366
 M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/browser_about_handler.cc?r1=65366&r2=65365&pathrev=65366
 A http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/enumerate_modules_model_win.h?r1=65366&r2=65365&pathrev=65366
 M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/app/generated_resources.grd?r1=65366&r2=65365&pathrev=65366
 M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/app/chrome_dll_resource.h?r1=65366&r2=65365&pathrev=65366
 M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/dom_ui/dom_ui_factory.cc?r1=65366&r2=65365&pathrev=65366
 M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/app/chromium_strings.grd?r1=65366&r2=65365&pathrev=65366
 M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/chrome_browser.gypi?r1=65366&r2=65365&pathrev=65366

First cut of the about:conflicts page, listing all DLLs loaded in the Chrome process.

BUG=http://crbug.com/51105, http://crbug.com/57239
TEST=Unit tests included.

Review URL: http://codereview.chromium.org/4524002
------------------------------------------------------------------------
Nov 16, 2010
#21 finnur@chromium.org
Feature has been renamed.
Summary: About:conflicts page
Nov 19, 2010
#24 bugdroid1@gmail.com
The following revision refers to this bug:
    http://src.chromium.org/viewvc/chrome?view=rev&revision=66756

------------------------------------------------------------------------
r66756 | finnur@chromium.org | Fri Nov 19 02:35:36 PST 2010

Changed paths:
 M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/resources/about_conflicts.html?r1=66756&r2=66755&pathrev=66756
 M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/enumerate_modules_model_win.cc?r1=66756&r2=66755&pathrev=66756
 M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/enumerate_modules_model_win.h?r1=66756&r2=66755&pathrev=66756

Add registered shell extensions to enumerated module list on about:conflicts.

BUG=51105
TEST=None

Review URL: http://codereview.chromium.org/5092007
------------------------------------------------------------------------
Nov 20, 2010
#26 bugdroid1@gmail.com
The following revision refers to this bug:
    http://src.chromium.org/viewvc/chrome?view=rev&revision=66880

------------------------------------------------------------------------
r66880 | finnur@chromium.org | Sat Nov 20 02:00:19 PST 2010

Changed paths:
 M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/enumerate_modules_model_win.cc?r1=66880&r2=66879&pathrev=66880
 M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/enumerate_modules_model_win.h?r1=66880&r2=66879&pathrev=66880

Polish the Enumerate Modules implementation a bit.

1) De-dup the registered DLLs since a lot of the same DLLs will be registered multiple times, ie. mswsock.dll
2) Every module loaded from a %temp% dir should be viewed as suspicious.
3) Add some histogram counters (xml changes coming in a followup CL).

BUG=http://crbug.com/51105
TEST=None

Review URL: http://codereview.chromium.org/5183007
------------------------------------------------------------------------
Nov 22, 2010
#27 finnur@chromium.org
 Issue 57239  has been merged into this issue.
Nov 26, 2010
#28 bugdroid1@gmail.com
The following revision refers to this bug:
    http://src.chromium.org/viewvc/chrome?view=rev&revision=67446

------------------------------------------------------------------------
r67446 | finnur@chromium.org | Fri Nov 26 02:23:39 PST 2010

Changed paths:
 M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/enumerate_modules_model_win.cc?r1=67446&r2=67445&pathrev=67446

Remove the test DLLs from the blacklist.
Demo is done.

TBR=huanr
BUG=51105
TEST=None

Review URL: http://codereview.chromium.org/5307004
------------------------------------------------------------------------
Nov 29, 2010
#29 bugdroid1@gmail.com
The following revision refers to this bug:
    http://src.chromium.org/viewvc/chrome?view=rev&revision=67598

------------------------------------------------------------------------
r67598 | finnur@chromium.org | Mon Nov 29 14:19:18 PST 2010

Changed paths:
 M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/resources/about_conflicts.html?r1=67598&r2=67597&pathrev=67598
 M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/enumerate_modules_model_unittest_win.cc?r1=67598&r2=67597&pathrev=67598
 M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/enumerate_modules_model_win.cc?r1=67598&r2=67597&pathrev=67598
 M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/app/generated_resources.grd?r1=67598&r2=67597&pathrev=67598
 M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/enumerate_modules_model_win.h?r1=67598&r2=67597&pathrev=67598

EnumerateModules: Address UI review comments.

Make it a simple list with columns. Sort by status (conflicts at the
top), then by location and module name. Also, convert the type of DLL
to an enum so we can dedup the list (and for non-loaded DLLs show that
they are not loaded yet, but are of type: Shell Extension / WinSock.

Make sure lower bound version specified on the blacklist is inclusive
(first version that broke) and the higher bound version is exclusive
(first version that worked).

This allows us to pointpoint exactly when the failure started and
when the fix was introduced, instead of doing 0.9999 shenanigans.

Specify an upper bound for the idmmbc.dll conflict, since Henry's
outreach produced a fix in version 6.03 of the download manager.

BUG=51105
TEST=Unit test

Review URL: http://codereview.chromium.org/5278012
------------------------------------------------------------------------
Nov 29, 2010
#30 bugdroid1@gmail.com
The following revision refers to this bug:
    http://src.chromium.org/viewvc/chrome?view=rev&revision=67624

------------------------------------------------------------------------
r67624 | finnur@chromium.org | Mon Nov 29 16:28:26 PST 2010

Changed paths:
 M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/ui/views/toolbar_view.cc?r1=67624&r2=67623&pathrev=67624

Adjust the position of the Wrench menu badge.

It was two pixels to far to the left.

TBR=glen
BUG=51105
TEST=None

Review URL: http://codereview.chromium.org/5348005
------------------------------------------------------------------------
Nov 30, 2010
#31 bugdroid1@gmail.com
The following revision refers to this bug:
    http://src.chromium.org/viewvc/chrome?view=rev&revision=67702

------------------------------------------------------------------------
r67702 | finnur@chromium.org | Tue Nov 30 02:18:59 PST 2010

Changed paths:
 M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/dom_ui/conflicts_ui.cc?r1=67702&r2=67701&pathrev=67702

Add UMA counter for when about:conflicts is viewed.

BUG=51105
TEST=None

Review URL: http://codereview.chromium.org/5259008
------------------------------------------------------------------------
Dec 6, 2010
#32 bugdro...@chromium.org
The following revision refers to this bug:
    http://src.chromium.org/viewvc/chrome?view=rev&revision=68346

------------------------------------------------------------------------
r68346 | finnur@chromium.org | Mon Dec 06 04:54:49 PST 2010

Changed paths:
 M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/resources/about_conflicts.html?r1=68346&r2=68345&pathrev=68346

Polish the about:conflicts page.

Address review comments from Nico and add a separating line as
requested by Glen.

BUG=51105
TEST=None

Review URL: http://codereview.chromium.org/5640001
------------------------------------------------------------------------
Dec 8, 2010
#33 bugdro...@chromium.org
The following revision refers to this bug:
    http://src.chromium.org/viewvc/chrome?view=rev&revision=68656

------------------------------------------------------------------------
r68656 | finnur@chromium.org | Wed Dec 08 14:55:08 PST 2010

Changed paths:
 M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/enumerate_modules_model_win.cc?r1=68656&r2=68655&pathrev=68656
 M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/app/generated_resources.grd?r1=68656&r2=68655&pathrev=68656

Update the blacklist to reflect latests statistics.

Also change the URL as per Support.

BUG=51105
TEST=None

Review URL: http://codereview.chromium.org/5629005
------------------------------------------------------------------------
Dec 8, 2010
#34 bugdro...@chromium.org
The following revision refers to this bug:
    http://src.chromium.org/viewvc/chrome?view=rev&revision=68676

------------------------------------------------------------------------
r68676 | finnur@chromium.org | Wed Dec 08 16:22:29 PST 2010

Changed paths:
 M http://src.chromium.org/viewvc/chrome/branches/597/src/chrome/browser/enumerate_modules_model_win.cc?r1=68676&r2=68675&pathrev=68676
 M http://src.chromium.org/viewvc/chrome/branches/597/src/chrome/app/generated_resources.grd?r1=68676&r2=68675&pathrev=68676

Merge 68656 - Update the blacklist to reflect latests statistics.

Also change the URL as per Support.

BUG=51105
TEST=None

Review URL: http://codereview.chromium.org/5629005

TBR=finnur@chromium.org
Review URL: http://codereview.chromium.org/5522013
------------------------------------------------------------------------
Dec 9, 2010
#35 kerz@chromium.org
P2 bugs with an owner that are not marked as started are being automatically moved to mstone:11.
Labels: -Mstone-10 MovedFrom-10 Mstone-11
Dec 9, 2010
#36 cl...@chromium.org
(No comment was entered for this change.)
Status: Started
Labels: -Mstone-11 Mstone-9
Jan 6, 2011
#37 bugdro...@chromium.org
The following revision refers to this bug:
    http://src.chromium.org/viewvc/chrome?view=rev&revision=70600

------------------------------------------------------------------------
r70600 | finnur@chromium.org | Thu Jan 06 00:57:45 PST 2011

Changed paths:
 M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/enumerate_modules_model_win.cc?r1=70600&r2=70599&pathrev=70600
 M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/diagnostics/diagnostics_model.cc?r1=70600&r2=70599&pathrev=70600
 M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/diagnostics/diagnostics_model_unittest.cc?r1=70600&r2=70599&pathrev=70600
 M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/enumerate_modules_model_win.h?r1=70600&r2=70599&pathrev=70600
 M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/diagnostics/diagnostics_model.h?r1=70600&r2=70599&pathrev=70600
 M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/diagnostics/diagnostics_main.cc?r1=70600&r2=70599&pathrev=70600
 M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/diagnostics/diagnostics_test.h?r1=70600&r2=70599&pathrev=70600
 M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/resources/about_conflicts.html?r1=70600&r2=70599&pathrev=70600
 M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/enumerate_modules_model_unittest_win.cc?r1=70600&r2=70599&pathrev=70600
 M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/diagnostics/recon_diagnostics.cc?r1=70600&r2=70599&pathrev=70600
 M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/diagnostics/recon_diagnostics.h?r1=70600&r2=70599&pathrev=70600

Integrate about:conflicts with --diagnostics

This should enable the user to find conflicting dlls that are crashing Chrome on startup.

This changelist introduces a limited_mode for scanning, since the diagnostics tests run without the help of many Chrome services, so we can't use the File thread for asynchronous scanning or the notification system.

Also changed the scanning so that if no signature/description is given in the blacklist, then we mark the DLL as a confirmed match if the name and the location match (before we used to match it as a 'suspected' match). A very sizable chunk of the blacklist is malware, which has no signature, so this gives us the ability to trigger the wrench badge on finding malware (that part remains opt-in through about:flags).

Also added failure count to diagnostics output (sometimes the error scrolls off screen, so having a note at the bottom helps).

BUG=51105
TEST=All diagnostics tests run as part of the unit tests.

Review URL: http://codereview.chromium.org/6098004
------------------------------------------------------------------------
Jan 7, 2011
#38 bugdro...@chromium.org
The following revision refers to this bug:
    http://src.chromium.org/viewvc/chrome?view=rev&revision=70727

------------------------------------------------------------------------
r70727 | finnur@chromium.org | Fri Jan 07 02:16:30 PST 2011

Changed paths:
 M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/app/generated_resources.grd?r1=70727&r2=70726&pathrev=70727
 M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/dom_ui/conflicts_ui.cc?r1=70727&r2=70726&pathrev=70727
 M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/resources/about_conflicts.html?r1=70727&r2=70726&pathrev=70727

Polish about:conflicts a bit as requested by Clem/UI review.

Add column header for data.
Move the Learn more link to appear last.

BUG=51105
TEST=None


Review URL: http://codereview.chromium.org/5981012
------------------------------------------------------------------------
Jan 9, 2011
#39 bugdro...@chromium.org
The following revision refers to this bug:
    http://src.chromium.org/viewvc/chrome?view=rev&revision=70860

------------------------------------------------------------------------
r70860 | finnur@chromium.org | Sun Jan 09 04:57:58 PST 2011

Changed paths:
 M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/enumerate_modules_model_win.cc?r1=70860&r2=70859&pathrev=70860

Make sure the histogram counters for about:conflicts get uploaded.

BUG=51105
TEST=None

Review URL: http://codereview.chromium.org/6098009
------------------------------------------------------------------------
Jan 11, 2011
#40 finnur@chromium.org
I am calling this on done. Any further work can be filed as a separate bug.

Separately, I checked in a change that fixes the UMA metrics for when the Wrench menu is opened. UMA metrics should therefore be in place for this feature for M10.
Status: Fixed
Feb 17, 2011
#41 cl...@chromium.org
(No comment was entered for this change.)
Labels: ConOps
Mar 23, 2011
#42 j...@chromium.org
(No comment was entered for this change.)
Labels: -ConOps Hotlist-ConOps
Oct 12, 2012
#43 bugdro...@chromium.org
This issue has been closed for some time. No one will pay attention to new comments.
If you are seeing this bug or have new data, please click New Issue to start a new bug.
Labels: Restrict-AddIssueComment-Commit
Blockedon: -chromium:53474 -chromium:57239 -chromium:61723 chromium:53474 chromium:57239 chromium:61723
Blocking: -chromium:60119 chromium:60119
Mar 10, 2013
#44 bugdro...@chromium.org
(No comment was entered for this change.)
Labels: -Area-Internals -Mstone-9 M-9 Cr-Internals
Mar 13, 2013
#45 bugdro...@chromium.org
(No comment was entered for this change.)
Labels: -Restrict-AddIssueComment-Commit Restrict-AddIssueComment-EditIssue
Sign in to add a comment

Powered by Google Project Hosting