| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Rename "version.h" to "chrome/common/chrome_version.h"
Update #include and GYP/GN references
Generate on all platforms
Remove redundant generated chrome/common/chrome_version.cc.version
Fix DEPS include_rules
BUG=542807
Committed: https://crrev.com/59647ecac9524d954010d3d10b539ee4a340bf94
Cr-Commit-Position: refs/heads/master@{#355196}
Committed: https://crrev.com/31f086c2013853749cac45522eb950b9d0ab0302
Cr-Commit-Position: refs/heads/master@{#355377}
Review URL: https://codereview.chromium.org/1405633002
Cr-Commit-Position: refs/heads/master@{#355434}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
https://codereview.chromium.org/1405633002/ )
Reason for revert:
Causing failures on the GN build (not gyp) for Mac:
http://build.chromium.org/p/chromium.mac/builders/Mac%20GN%20%28dbg%29/builds/23191
Original issue's description:
> Cleanup chrome/common/chrome_version.h handling
>
> Rename "version.h" to "chrome/common/chrome_version.h"
> Update #include and GYP/GN references
> Generate on all platforms
> Remove redundant generated chrome/common/chrome_version.cc.version
> Fix DEPS include_rules
>
> BUG=542807
>
> Committed: https://crrev.com/59647ecac9524d954010d3d10b539ee4a340bf94
> Cr-Commit-Position: refs/heads/master@{#355196}
>
> Committed: https://crrev.com/31f086c2013853749cac45522eb950b9d0ab0302
> Cr-Commit-Position: refs/heads/master@{#355377}
TBR=brettw@chromium.org,mdempsky@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=542807
Review URL: https://codereview.chromium.org/1418933002
Cr-Commit-Position: refs/heads/master@{#355393}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Rename "version.h" to "chrome/common/chrome_version.h"
Update #include and GYP/GN references
Generate on all platforms
Remove redundant generated chrome/common/chrome_version.cc.version
Fix DEPS include_rules
BUG=542807
Committed: https://crrev.com/59647ecac9524d954010d3d10b539ee4a340bf94
Cr-Commit-Position: refs/heads/master@{#355196}
Review URL: https://codereview.chromium.org/1405633002
Cr-Commit-Position: refs/heads/master@{#355377}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Windows 10 WOW64 has different thunk than Windows 8.
Test ServiceResolverTest.PatchesServices from sbox_unittests failed.
Library blacklist from src\chrome_elf\blacklist\blacklist.cc does not
work.
BUG=
R=cpu@chromium.org
R=jschuh@chromium.org
TEST=sbox_unittests on Win 10
Review URL: https://codereview.chromium.org/1407103003
Cr-Commit-Position: refs/heads/master@{#355307}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
https://codereview.chromium.org/1405633002/ )
Reason for revert:
Appears to be breaking at least Mac GN build
Example failure: http://build.chromium.org/p/chromium.mac/builders/Mac%20GN%20%28dbg%29/builds/23114
Original issue's description:
> Cleanup chrome/common/chrome_version.h handling
>
> Rename "version.h" to "chrome/common/chrome_version.h"
> Update #include and GYP/GN references
> Generate on all platforms
> Remove redundant generated chrome/common/chrome_version.cc.version
> Fix DEPS include_rules
>
> BUG=542807
>
> Committed: https://crrev.com/59647ecac9524d954010d3d10b539ee4a340bf94
> Cr-Commit-Position: refs/heads/master@{#355196}
TBR=brettw@chromium.org,mdempsky@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=542807
Review URL: https://codereview.chromium.org/1411173003
Cr-Commit-Position: refs/heads/master@{#355231}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Rename "version.h" to "chrome/common/chrome_version.h"
Update #include and GYP/GN references
Generate on all platforms
Remove redundant generated chrome/common/chrome_version.cc.version
Fix DEPS include_rules
BUG=542807
Review URL: https://codereview.chromium.org/1405633002
Cr-Commit-Position: refs/heads/master@{#355196}
|
|
|
|
|
|
|
|
|
|
|
|
| |
Passing std::string by reference can prevent extra copying of object.
BUG=367418
TEST=
R=caitkp@chromium.org,robertshield@chromium.org
Review URL: https://codereview.chromium.org/1352793003
Cr-Commit-Position: refs/heads/master@{#349887}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
These are needed to link in asan/lsan/etc. mode. GYP injects these automatically but GN doesn't have automatic dependency injection.
All tests and components (which cover the vast majority of such targets in actual use) are templates which add these dependencies already, so most of the time it never comes up.
Only a few of these changes are actually necessary to link all tests run on the asan bot (which is also what engineers will typically build, since almost nobody will build all in asan mode). But to reduce confusion from linker errors in asan mode, I annotated all executables and shared libraries I could find with the sanitizer dependency.
I moved a config out of a target in yasm, and sorted a fews deps on other targets.
TBR=xhwang@chromium.org (widevine)
TBR=achuith@chromium.org (tools/perf, tools/telemetry)
Review URL: https://codereview.chromium.org/1318343003
Cr-Commit-Position: refs/heads/master@{#346490}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Changes the version processing templates to not implicitly include the chrome version .rc file. This was something I've been meaning to do for a while which was forced by this patch, because it wants to call version.py with no sources or template file. I also had to remove some checks that either a template file or sources were defined (I wasn't aware this was valid to do when I wrote these checks).
The above-mentioned change necessitated referencing the version template file from a number of existing calls.
The version script now explicitly sets the output file with -o. Previously this was the implicit second argument, but this didn't work if there was no template file. -o is the same and it's better to be more explicit anyway.
Added copy rules for xdg-mime related scripts which are required by the installer.
Added a meta "installer" target which links to the Linux installers. This will also be a good place to hook up the Windows installer when we write it.
BUG=525839
TBR=thestig@chromium.org (linux installer), ddorwin@chromium.org (widevine)
Review URL: https://codereview.chromium.org/1311543003
Cr-Commit-Position: refs/heads/master@{#346279}
|
|
|
|
|
|
|
|
| |
BUG=
Review URL: https://codereview.chromium.org/1274273006
Cr-Commit-Position: refs/heads/master@{#342832}
|
|
|
|
|
|
|
|
|
|
|
| |
Apparently the trybots don't build all targets :-/
BUG=505316
TBR=brettw,robertshield
Review URL: https://codereview.chromium.org/1284523002.
Cr-Commit-Position: refs/heads/master@{#342537}
|
|
|
|
|
|
|
|
| |
BUG=
Review URL: https://codereview.chromium.org/1248863004
Cr-Commit-Position: refs/heads/master@{#341966}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Added lots of bug references to TODOs.
Added rules for some remoting test files.
Added manifests for chrome.dll, chrome_child.dll, and chrome_elf.dll
TBR=bradnelson for chrome/test/data/nacl
Review URL: https://codereview.chromium.org/1250103002
Cr-Commit-Position: refs/heads/master@{#340009}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Usually, fixed by removing the unused variables. Some variables had to
be guarded by #ifs. Some variables could be used instead.
Also removed unused test file sweep02_16b_mono_16KHz.raw.
BUG=505319
TBR=rpaquay@chromium.org
Review URL: https://codereview.chromium.org/1220133003
Cr-Commit-Position: refs/heads/master@{#337767}
|
|
|
|
|
|
|
|
| |
BUG=
Review URL: https://codereview.chromium.org/1226673003
Cr-Commit-Position: refs/heads/master@{#337488}
|
|
|
|
|
|
| |
Review URL: https://codereview.chromium.org/1158003002
Cr-Commit-Position: refs/heads/master@{#331602}
|
|
|
|
|
|
|
|
|
|
| |
so we only take a loader-lock the first time.
BUG=485656, 477137
Review URL: https://codereview.chromium.org/1144433004
Cr-Commit-Position: refs/heads/master@{#329911}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
the first time. (patchset #4 id:60001 of https://codereview.chromium.org/1132473003/)
Reason for revert:
Possibly causing mysterious fails on the Win DBG bots:
https://build.chromium.org/p/chromium.webkit/builders/Win7%20%28dbg%29/builds/16344
http://build.chromium.org/p/chromium.win/builders/Win7%20Tests%20%28dbg%29%281%29/builds/37896
Original issue's description:
> Cache IsNonBrowserProcess values, so we only take a loader-lock the first time.
>
> BUG=485656,477137
>
> Committed: https://crrev.com/78d41b5d4e59dc7d52d7e7d6530fdd0a78b36d33
> Cr-Commit-Position: refs/heads/master@{#329246}
TBR=siggi@chromium.org,grt@chromium.org,csharp@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=485656,477137
Review URL: https://codereview.chromium.org/1134253002
Cr-Commit-Position: refs/heads/master@{#329467}
|
|
|
|
|
|
|
|
| |
BUG=485656,477137
Review URL: https://codereview.chromium.org/1132473003
Cr-Commit-Position: refs/heads/master@{#329246}
|
|
|
|
|
|
|
|
| |
BUG=
Review URL: https://codereview.chromium.org/1131943002
Cr-Commit-Position: refs/heads/master@{#328814}
|
|
|
|
|
|
| |
Review URL: https://codereview.chromium.org/1113153003
Cr-Commit-Position: refs/heads/master@{#328162}
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
The first pass of moving the other Chrome-specific build flags out of the root build config file and into the new one. This requires both ffmpeg and Blink changes to finish landing.
Originally landed as https://codereview.chromium.org/1120803003/
TBR=scottmg@chromium.org
TBR=ddorwin@chromium.org (widevine)
CQ_EXTRA_TRYBOTS=tryserver.chromium.linux:android_chromium_gn_compile_dbg,android_chromium_gn_compile_rel;tryserver.chromium.win:win8_chromium_gn_rel,win8_chromium_gn_dbg;tryserver.chromium.mac:mac_chromium_gn_rel,mac_chromium_gn_dbg
Review URL: https://codereview.chromium.org/1123433002
Cr-Commit-Position: refs/heads/master@{#327998}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
id:40001 of https://codereview.chromium.org/1120803003/)
Reason for revert:
May have caused some link errors on windows: https://build.chromium.org/p/chromium.win/builders/Win%20x64%20GN/builds/814
Original issue's description:
> Add a GN flag for the Windows multi-dll build.
>
> The first pass of moving the other Chrome-specific build flags out of the root build config file and into the new one. This requires both ffmpeg and Blink changes to finish landing.
>
> R=scottmg@chromium.org
> TBR=ddorwin@chromium.org (widevine)
>
> Committed: https://crrev.com/edff53a35780ba2f515c7e8d4f7131c91151e1a3
> Cr-Commit-Position: refs/heads/master@{#327854}
TBR=scottmg@chromium.org,brettw@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
Review URL: https://codereview.chromium.org/1117063004
Cr-Commit-Position: refs/heads/master@{#327861}
|
|
|
|
|
|
|
|
|
|
|
| |
The first pass of moving the other Chrome-specific build flags out of the root build config file and into the new one. This requires both ffmpeg and Blink changes to finish landing.
R=scottmg@chromium.org
TBR=ddorwin@chromium.org (widevine)
Review URL: https://codereview.chromium.org/1120803003
Cr-Commit-Position: refs/heads/master@{#327854}
|
|
|
|
|
|
|
|
|
|
|
| |
Working on trimming down plugin warnings on the Windows clang build.
BUG=467287
TBR=caitkp@chromium.org
Review URL: https://codereview.chromium.org/1112853003
Cr-Commit-Position: refs/heads/master@{#327585}
|
|
|
|
|
|
|
|
|
|
| |
Also add two older values to dllhash.
BUG=
Review URL: https://codereview.chromium.org/1110123002
Cr-Commit-Position: refs/heads/master@{#327295}
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Please take look at
https://codereview.chromium.org/1090213002/patch/40001/50001
for details
BUG=
Review URL: https://codereview.chromium.org/1059333004
Cr-Commit-Position: refs/heads/master@{#327270}
|
|
|
|
|
|
|
|
| |
BUG=476648,479710
Review URL: https://codereview.chromium.org/1100413002
Cr-Commit-Position: refs/heads/master@{#326784}
|
|
|
|
|
|
|
|
|
|
|
|
| |
The Google style guide states that only one of {virtual,override,final} should be used for each declaration, since override implies virtual and final implies both virtual and override.
This patch was manually generated using a regex and a text editor.
BUG=417463
Review URL: https://codereview.chromium.org/1106533003
Cr-Commit-Position: refs/heads/master@{#326454}
|
|
|
|
|
|
|
|
| |
This has been causing a large number of browser crashes.
Review URL: https://codereview.chromium.org/1101563002
Cr-Commit-Position: refs/heads/master@{#326098}
|
|
|
|
|
|
|
|
| |
BUG=463495,463500
Review URL: https://codereview.chromium.org/1024183004
Cr-Commit-Position: refs/heads/master@{#322878}
|
|
|
|
|
|
|
|
|
| |
R=cait@chromium.org, jwd@chromium.org
BUG=
Review URL: https://codereview.chromium.org/1022523003
Cr-Commit-Position: refs/heads/master@{#322005}
|
|
|
|
|
|
| |
Review URL: https://codereview.chromium.org/951463003
Cr-Commit-Position: refs/heads/master@{#320086}
|
|
|
|
|
|
|
|
| |
BUG=345874
Review URL: https://codereview.chromium.org/974903002
Cr-Commit-Position: refs/heads/master@{#318897}
|
|
|
|
|
|
|
|
|
|
|
|
| |
Application of https://codereview.chromium.org/962003002/.
TBR=keybuk@chromium.org
R=brettw@chromium.org
BUG=456014
Review URL: https://codereview.chromium.org/960413003
Cr-Commit-Position: refs/heads/master@{#318574}
|
|
|
|
|
|
|
|
|
|
|
| |
Testing shows that this change is correct, but there was no evidence supporting the other change. Will manually run the x64 dbg test try bot for this change.
TBR=thakis@chromium.org,jvoung@chromium.org
BUG=458690,460563
Review URL: https://codereview.chromium.org/942003008
Cr-Commit-Position: refs/heads/master@{#318160}
|
|
|
|
|
|
|
|
|
|
|
|
| |
Changes process_version template. In GYP this is used in two ways and the GN version only supported one of these. This change also deletes an unnecessary fork of the template file, and fixes the dependencies for the extra files (previously they were just passed with "-f" and not added to the inputs of the target.
Adds a cdm_adapter template which basically matches the GYP version. Uses this for the media clearkey adapter and the widevine one in the official build.
Fixes for the crypto targets when compiling with the official build's checked-in sysroots.
Review URL: https://codereview.chromium.org/949233003
Cr-Commit-Position: refs/heads/master@{#318083}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
(patchset #3 id:40001 of https://codereview.chromium.org/921353002/)
Reason for revert:
Speculative revert after discussion on IRC, this seems pretty unlikely but may have caused the failures at:
http://build.chromium.org/p/chromium.win/builders/Win7%20Tests%20%28dbg%29%281%29/builds/35506
Original issue's description:
> sandbox: Fix some nt_internals.h function prototypes for Win64
>
> NtQuerySection should use ULONG instead of SIZE_T as it is dealing with PE section sizes.
>
> NtQueryVirtualMemory should use SIZE_T instead of ULONG as it is dealing with memory sizes.
>
> DynamoRIO, another project that consumes these native APIs, uses the prototypes introduced by this change.
>
> R=robertshield@chromium.org,rvargas@chromium.org
> BUG=458690
>
> Committed: https://crrev.com/4ede4e174a51d08262edfb2e2c05845504bd9b1a
> Cr-Commit-Position: refs/heads/master@{#317177}
TBR=jschuh@chromium.org,robertshield@chromium.org,thakis@chromium.org,wfh@chromium.org,rvargas@chromium.org,rnk@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=458690
Review URL: https://codereview.chromium.org/943043002
Cr-Commit-Position: refs/heads/master@{#317409}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
The latest GN binaries rename cpu_arch to current_cpu and/or target_cpu
as appropriate, build_cpu_arch to host_cpu, and os to current_os and target_os as appropriate.
R=brettw@chromium.org
TBR=ddorwin@chromium.org
EXTRA_TRYBOTS=tryserver.chromium.linux:android_chromium_gn_compile_dbg,android_chromium_gn_compile_rel;tryserver.chromium.win:win8_chromium_gn_rel,win8_chromium_gn_dbg;tryserver.chromium.mac:mac_chromium_gn_rel,mac_chromium_gn_dbg
BUG=344767
Review URL: https://codereview.chromium.org/913373002
Cr-Commit-Position: refs/heads/master@{#317223}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
NtQuerySection should use ULONG instead of SIZE_T as it is dealing with PE section sizes.
NtQueryVirtualMemory should use SIZE_T instead of ULONG as it is dealing with memory sizes.
DynamoRIO, another project that consumes these native APIs, uses the prototypes introduced by this change.
R=robertshield@chromium.org,rvargas@chromium.org
BUG=458690
Review URL: https://codereview.chromium.org/921353002
Cr-Commit-Position: refs/heads/master@{#317177}
|
|
|
|
|
|
|
|
| |
BUG=454018
Review URL: https://codereview.chromium.org/891813003
Cr-Commit-Position: refs/heads/master@{#314213}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Avoids ninja expecting an import .lib to be generated so that it doesn't
continually rebuild because the import lib is "missing".
To be landed after gyp roll at https://codereview.chromium.org/890043003/ lands.
R=robertshield@chromium.org
BUG=342974,451499
Review URL: https://codereview.chromium.org/886123002
Cr-Commit-Position: refs/heads/master@{#314008}
|
|
|
|
|
|
| |
Review URL: https://codereview.chromium.org/832583007
Cr-Commit-Position: refs/heads/master@{#312814}
|
|
|
|
|
|
|
|
|
|
| |
These dlls have been causing stability issues.
BUG=
Review URL: https://codereview.chromium.org/852843002
Cr-Commit-Position: refs/heads/master@{#312392}
|
|
|
|
|
|
|
|
| |
BUG=
Review URL: https://codereview.chromium.org/837033002
Cr-Commit-Position: refs/heads/master@{#311053}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This driver has persistent issues, we previously blacklisted it for
http://crbug.com/378465 but came up with a workaround and thus it
was allowed again. Now we're getting renderer hangs because it
blocks a sync ipc from completing on the audio thread.
Given the multiple issues and high install base, I think we should
permanently blacklist this dll from loading.
BUG=422522
TEST=none
Review URL: https://codereview.chromium.org/838683002
Cr-Commit-Position: refs/heads/master@{#310556}
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
One of the top crashes for Chrome 38 and 39 is a crash due
to some issue in YCWebCameraSource.ax, which is a component
of CyberLink Youcam software. Let's blacklist it for now,
to stop Chrome from crashing repeatedly.
BUG=424159
Review URL: https://codereview.chromium.org/831223003
Cr-Commit-Position: refs/heads/master@{#310284}
|
|
|
|
|
|
|
|
| |
BUG=429173
Review URL: https://codereview.chromium.org/698533002
Cr-Commit-Position: refs/heads/master@{#302274}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
BUG=427616
/analyze pointed out that the loop range-check used the comma operator
when && was probably intended. The bug was harmless, but confusing, so
I fixed it.
/analyze also pointed out that the code was checking data read by a
function and then checking whether the read succeeded. Reversing the
order of the checks has no effect except to get rid of the undefined
behavior.
Review URL: https://codereview.chromium.org/678193003
Cr-Commit-Position: refs/heads/master@{#301691}
|
|
|
|
|
|
|
|
| |
BUG=
Review URL: https://codereview.chromium.org/654753008
Cr-Commit-Position: refs/heads/master@{#300823}
|