| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
|
|
|
|
| |
BUG=394532
TEST=unit_tests
Review URL: https://codereview.chromium.org/444543002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@287899 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
| |
BUG=399744
Review URL: https://codereview.chromium.org/440663004
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@287546 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
| |
BUG=370153
Review URL: https://codereview.chromium.org/434143002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@287116 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
BUG=397137,396244
TEST=chrome_elf_unittests
NOTRY=true
Review URL: https://codereview.chromium.org/418033004
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@285377 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
| |
minisp.dll (seems to be malware)
cespy.dll (CovenantEyes)
cplushook.dll (seems to be malware)
BUG=
Review URL: https://codereview.chromium.org/354063002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@280443 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Also added a unit test to environment_data_collection_win_unittest.cc
Edited chrome_elf_init_win.cc and blacklist.cc so that the registry is
only cleared immediately before being repopulated (and safe browsing is not left
with an empty registry to read from).
BUG=
Review URL: https://codereview.chromium.org/346763003
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@280380 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
| |
BUG=
Review URL: https://codereview.chromium.org/353873002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@279814 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
| |
BUG=
Review URL: https://codereview.chromium.org/357733002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@279741 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
| |
BUG=
Review URL: https://codereview.chromium.org/349693002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@279267 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
| |
BUG=
Review URL: https://codereview.chromium.org/343613003
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@278393 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
| |
BUG=
Review URL: https://codereview.chromium.org/332193003
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@277791 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Also added UMA recording of the blacklist disabled and retry succeeded events.
Deprecating the blacklist start up events 'BLACKLIST_THUNK_SETUP_FAILED' and
'BLACKLIST_INTERCEPTION_FAILED' as we are switching over to a more coarse
grained recording of the set up status (this information can be gathered from
crash reporting now anyways).
BUG=
Review URL: https://codereview.chromium.org/311893005
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@277614 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
(https://codereview.chromium.org/325323003/)
Reason for revert:
No crashes have been found after 37.0.2046.2 and Canary on Windows is now on 37.0.2051.2, hence it looks good. Not sure if was the blacklisting or the "calling CoInitializeEx twice" patch which did the trick here.
I would like to revert the blacklisting patch to see if we can cope with the CoInitializeEx patch only. It would be great if we could avoid adding another DLL to the blacklist.
Original issue's description:
> Blacklists virtualCamera.ax.
>
> Tries to avoid crash in media::CoreAudioUtil::CreateDeviceEnumerator by blacklisting a DLL which accounts for a large amount of crashes we see in CreateDeviceEnumerator.
>
> BUG=383715, 378465
>
> Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=276946
TBR=tommi@chromium.org,robertshield@chromium.org
NOTREECHECKS=true
NOTRY=true
BUG=383715, 378465
Review URL: https://codereview.chromium.org/335073002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@277320 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
| |
This comment roughly explains when the blacklist should be used, as well
as linking to the blacklist policy.
R=robertshield@chromium.org
BUG=
Review URL: https://codereview.chromium.org/334913002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@277108 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
Tries to avoid crash in media::CoreAudioUtil::CreateDeviceEnumerator by blacklisting a DLL which accounts for a large amount of crashes we see in CreateDeviceEnumerator.
BUG=383715, 378465
Review URL: https://codereview.chromium.org/325323003
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@276946 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
These DLLs were causing major instabilities on Stable Channel.
BUG=379218
NOTRY=true
Review URL: https://codereview.chromium.org/320653003
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@275859 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
The finch blacklist is collected from finch in chrome_elf_init_win.cc,
stored in a registry, and added to the blacklist in blacklist.cc.
BUG=
Review URL: https://codereview.chromium.org/300933002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@275086 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
| |
BUG=
Review URL: https://codereview.chromium.org/298893004
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@273377 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
BUG=NONE
TEST=NONE
Review URL: https://codereview.chromium.org/292123011
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@272337 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
| |
BUG=362783
Review URL: https://codereview.chromium.org/292933002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@271840 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
| |
We need a +4GB base so the loader will randomize above 4GB.
BUG=361670
R=robertshield@chromium.org,jam@chromium.org
NOTRY=TRUE
Review URL: https://codereview.chromium.org/233413008
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@263540 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
We've seen different failure related to the fact that we use the same flag without providing the same API and functionality.
BUG=
TBR=jamesr
Review URL: https://codereview.chromium.org/212643015
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@259855 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
| |
BUG=353978
Review URL: https://codereview.chromium.org/208273006
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@259157 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
| |
Review URL: https://codereview.chromium.org/206223006
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@259014 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
1. Add functionality to ServiceResolverThunk to copy a thunk without patching.
2. Move chrome_elf thunk-handling code to a common location.
3. Use a thunk instead of a f'n ptr for redirects.
BUG=334379
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=255151
Review URL: https://codereview.chromium.org/183833004
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@257749 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
| |
BUG=348383
Review URL: https://codereview.chromium.org/197863002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@256751 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
Use the exported functions iterator in PEImage instead of writing our
own.
BUG=
Review URL: https://codereview.chromium.org/192933003
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@256424 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
(Crashy on today's canary)
> Make chrome_elf use thunks instead of function pointers.
>
> 1. Add functionality to ServiceResolverThunk to copy a thunk without patching.
> 2. Move chrome_elf thunk-handling code to a common location.
> 3. Use a thunk instead of a f'n ptr for redirects.
>
> BUG=334379
>
> Review URL: https://codereview.chromium.org/183833004
TBR=caitkp@chromium.org
Review URL: https://codereview.chromium.org/189803007
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@255703 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
BUG=349926
TEST=NONE
Review URL: https://codereview.chromium.org/183973039
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@255570 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
| |
1. Add functionality to ServiceResolverThunk to copy a thunk without patching.
2. Move chrome_elf thunk-handling code to a common location.
3. Use a thunk instead of a f'n ptr for redirects.
BUG=334379
Review URL: https://codereview.chromium.org/183833004
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@255151 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
SafeGetImageInfo shouldn't raise exceptions.
BUG=348693
Review URL: https://codereview.chromium.org/185533008
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@254870 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
If the registry says the blacklist is enabled, then enable it, even if
it was enabled for a different version of Chrome.
BUG=329023
Review URL: https://codereview.chromium.org/178543011
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@254819 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
| |
BUG=345287
Review URL: https://codereview.chromium.org/174013007
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@254480 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
Use both IsSandboxed and command line flag.
BUG=329023
TEST=chrome_elf_unittests
Review URL: https://codereview.chromium.org/181863002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@254308 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
This will cover most of the CreateFile calls and save us from death by
circular gyp dependencies.
Review URL: https://codereview.chromium.org/183773003
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@254231 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
| |
This ensure that the dll code is referenced, instead of the target
making its own copy.
TBR=sky@chromium.org
BUG=329023
Review URL: https://codereview.chromium.org/181373003
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@253847 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
code (https://codereview.chromium.org/176873012/)
Reason for revert:
Broke unit_tests on Win 7 debug:
http://build.chromium.org/p/chromium.win/buildstatus?builder=Win7%20Tests%20%28dbg%29%281%29&number=26698
Original issue's description:
> Link chrome_elf.dll instead of statically linking the blacklist code
>
> This ensure that the dll code is referenced, instead of the target
> making its own copy.
>
>
> BUG=329023
>
> Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=253366
TBR=robertshield@chromium.org,sky@chromium.org,csharp@chromium.org
NOTREECHECKS=true
NOTRY=true
BUG=329023
Review URL: https://codereview.chromium.org/181343003
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@253399 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
| |
This ensure that the dll code is referenced, instead of the target
making its own copy.
BUG=329023
Review URL: https://codereview.chromium.org/176873012
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@253366 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
| |
TEST=Manual: Start chrome.exe, wait ~60s and check chrome://histograms, the Stability.FileAtPath metric should have run (at least) once, and the resulting value should be 6, assuming the check succeeded.
Review URL: https://codereview.chromium.org/169093007
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@253251 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
| |
Review URL: https://codereview.chromium.org/154653002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@252870 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
| |
BUG=343792
OK, it's been almost a week now, with regular pings to OWNERs, so I guess it's OK to TBR... It's only Gyp/script changes anyway... Feel free to uncheck the CQ box if you disagree...
TBR=beng@chromium.org, darin@chromium.org
Review URL: https://codereview.chromium.org/165893002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@252481 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
It is ok for SafeGetImageInfo to fails to get the image info, so catch
exceptions instead of dying.
BUG=343925
Review URL: https://codereview.chromium.org/171153003
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@252062 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
Begin blacklisting datamngr.dl, lmrn.dll and hk.dll from the browser
process.
BUG=329023
Review URL: https://codereview.chromium.org/171153002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@251886 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
| |
BUG=334379
Review URL: https://codereview.chromium.org/154463002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@251823 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This shouldn't be happening, but we got some crash reports suggesting it
does. Unable to repo locally so this stat will verify it does occur and
then can be used to verify our fixes actually fix it.
TBR=asvitkine@chromium.org
BUG=329023
Review URL: https://codereview.chromium.org/166953002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@251454 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Reverting again. Confirmed that this is indeed the CL that broke
the Win7 x64 sync tests.
> Revert 251095 "Revert 250828 "Add a UMA stat to track if the Bro..."
>
> Speculative re-land to see if this causes sync tests failures again.
> If it does, then we will know if this is the cause.
>
> > Revert 250828 "Add a UMA stat to track if the Browser blacklist ..."
> >
> > Speculative revert for failures here:
> >
> > http://build.chromium.org/p/chromium.win/builders/Win7%20Sync%20x64/builds/11201
> >
> > > Add a UMA stat to track if the Browser blacklist is Set on the Renderer
> > >
> > > This shouldn't be happening, but we got some crash reports suggesting it
> > > does. Unable to repo locally so this stat will verify it does occur and
> > > then can be used to verify our fixes actually fix it.
> > >
> > > BUG=329023
> > >
> > > Review URL: https://codereview.chromium.org/140763008
> >
> > TBR=csharp@chromium.org
> >
> > Review URL: https://codereview.chromium.org/163633005
>
> TBR=asvitkine@chromium.org
>
> Review URL: https://codereview.chromium.org/164833002
TBR=asvitkine@chromium.org
Review URL: https://codereview.chromium.org/164913003
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@251174 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Speculative re-land to see if this causes sync tests failures again.
If it does, then we will know if this is the cause.
> Revert 250828 "Add a UMA stat to track if the Browser blacklist ..."
>
> Speculative revert for failures here:
>
> http://build.chromium.org/p/chromium.win/builders/Win7%20Sync%20x64/builds/11201
>
> > Add a UMA stat to track if the Browser blacklist is Set on the Renderer
> >
> > This shouldn't be happening, but we got some crash reports suggesting it
> > does. Unable to repo locally so this stat will verify it does occur and
> > then can be used to verify our fixes actually fix it.
> >
> > BUG=329023
> >
> > Review URL: https://codereview.chromium.org/140763008
>
> TBR=csharp@chromium.org
>
> Review URL: https://codereview.chromium.org/163633005
TBR=asvitkine@chromium.org
Review URL: https://codereview.chromium.org/164833002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@251134 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Speculative revert for failures here:
http://build.chromium.org/p/chromium.win/builders/Win7%20Sync%20x64/builds/11201
> Add a UMA stat to track if the Browser blacklist is Set on the Renderer
>
> This shouldn't be happening, but we got some crash reports suggesting it
> does. Unable to repo locally so this stat will verify it does occur and
> then can be used to verify our fixes actually fix it.
>
> BUG=329023
>
> Review URL: https://codereview.chromium.org/140763008
TBR=csharp@chromium.org
Review URL: https://codereview.chromium.org/163633005
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@251095 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
| |
This shouldn't be happening, but we got some crash reports suggesting it
does. Unable to repo locally so this stat will verify it does occur and
then can be used to verify our fixes actually fix it.
BUG=329023
Review URL: https://codereview.chromium.org/140763008
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@250828 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
(https://codereview.chromium.org/160373002/)
Reason for revert:
As discussed, this shouldn't interfere with the metrics we care about.
Original issue's description:
> Revert blacklist part of crrev.com/250682.
>
> Don't want the new check in just yet, it will interfere with metrics.
>
> BUG=329023
> TBR=csharp
> NOTRY=true
> NOTREECHECKS=TRUE
>
> Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=250704
TBR=csharp@chromium.org
NOTREECHECKS=true
NOTRY=true
BUG=329023
Review URL: https://codereview.chromium.org/160643002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@250758 0039d316-1c4b-4281-b951-d872f2087c98
|