- 01 Aug, 2012 40 commits
-
-
chrome-admin@google.com authored
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@149491 0039d316-1c4b-4281-b951-d872f2087c98
-
joi@chromium.org authored
This means, add the intended DEPS rules for a Browser Component (depend on nothing within chrome/browser outside the component except chrome/browser/api) and add a temporary owner for pure refactoring changes so that such changes can be done more quickly. Larger changes (that affect functionality or interfaces in non-refactoring ways) will still be sent to a proper OWNER. With the new DEPS, a presubmit check will warn developers that add new temporarily-allowed (the files designated with a "!" rule in the DEPS file). See http://www.chromium.org/developers/design-documents/browser-components BUG=140037 Review URL: https://chromiumcodereview.appspot.com/10825134 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@149490 0039d316-1c4b-4281-b951-d872f2087c98
-
http://codereview.chromium.org/10825089/vandebo@chromium.org authored
This made the ChromiumOS bot blow up. make: *** No rule to make target `c/Release/obj.target/skia_opts/src/opts/SkBitmapProcState_matrixProcs_neon.o', needed by `c/Release/obj.tar http://build.chromium.org/p/chromium.chromiumos/builders/ChromiumOS%20%28daisy%29/builds/1703/steps/cbuildbot/logs/stdio roll skia to 4889 Review URL: https://chromiumcodereview.appspot.com/10827122 TBR=reed@google.com Review URL: https://chromiumcodereview.appspot.com/10843036 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@149489 0039d316-1c4b-4281-b951-d872f2087c98
-
newt@chromium.org authored
This adds an explicit "platform" option to repack_locales.py, so resources for one platform (e.g. Android) can be compiled on another (e.g. Linux). BUG=136951 Review URL: https://chromiumcodereview.appspot.com/10830085 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@149488 0039d316-1c4b-4281-b951-d872f2087c98
-
thestig@chromium.org authored
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@149487 0039d316-1c4b-4281-b951-d872f2087c98
-
chrome-admin@google.com authored
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@149486 0039d316-1c4b-4281-b951-d872f2087c98
-
thestig@chromium.org authored
BUG=none TEST=none TBR=vandebo Review URL: https://chromiumcodereview.appspot.com/10830125 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@149485 0039d316-1c4b-4281-b951-d872f2087c98
-
rsesek@chromium.org authored
This change does three things: * Removes RefCoununted from the class. There's no visible reason for it to be. * Changes the constructor to only take one param, as the other two params can be derived from Browser. * Re-orders the methods in a logical way, and makes the implementation match the interface order. BUG=114373 TEST=No behavior changes. Review URL: https://chromiumcodereview.appspot.com/10825115 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@149484 0039d316-1c4b-4281-b951-d872f2087c98
-
grt@chromium.org authored
NOTRY=true BUG=none TEST=none Review URL: https://chromiumcodereview.appspot.com/10831087 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@149483 0039d316-1c4b-4281-b951-d872f2087c98
-
pauljensen@chromium.org authored
BUG=135203 TEST= Review URL: https://chromiumcodereview.appspot.com/10824115 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@149482 0039d316-1c4b-4281-b951-d872f2087c98
-
thestig@chromium.org authored
BUG=none TEST=none Review URL: https://chromiumcodereview.appspot.com/10825126 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@149481 0039d316-1c4b-4281-b951-d872f2087c98
-
chrome-admin@google.com authored
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@149480 0039d316-1c4b-4281-b951-d872f2087c98
-
scottmg@chromium.org authored
r1442 Allow <(CONFIGURATION_NAME) to be used in path for ninja. r1443 ninja windows: don't spam reg.exe calls if we can't find DirectX installation r1444 make unitests less fragile in the face of debug output changes. r1445 Fixing issues with path. r1446 ninja windows: unfork back to 'depfile' name rather than 'deplist' R=thakis@chromium.org Review URL: https://chromiumcodereview.appspot.com/10824127 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@149479 0039d316-1c4b-4281-b951-d872f2087c98
-
pkasting@chromium.org authored
BUG=none TEST=none TBR=apavlov Review URL: https://chromiumcodereview.appspot.com/10823126 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@149478 0039d316-1c4b-4281-b951-d872f2087c98
-
ericu@google.com authored
All changes in perf_test.js are reorders--no actual code change. BUG=137764 TEST=self Review URL: https://chromiumcodereview.appspot.com/10828119 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@149477 0039d316-1c4b-4281-b951-d872f2087c98
-
thestig@chromium.org authored
BUG=none TEST=none Review URL: https://chromiumcodereview.appspot.com/10827111 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@149476 0039d316-1c4b-4281-b951-d872f2087c98
-
alexeypa@chromium.org authored
- Specify access bits on the duplicated handle correctly. - Avoid touching an uninitialized buffer in case of an error. BUG=139841 TEST=RestrictedTokenTest.DenyOwnerSidCustom, RestrictedTokenTest.AddRestrictingSidCurrentUserCustom Review URL: https://chromiumcodereview.appspot.com/10844003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@149475 0039d316-1c4b-4281-b951-d872f2087c98
-
sievers@chromium.org authored
BUG=136923 Review URL: https://chromiumcodereview.appspot.com/10821134 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@149474 0039d316-1c4b-4281-b951-d872f2087c98
-
alecflett@chromium.org authored
R=jsbell@chromium.org,dgrogan@chromium.org BUG= Review URL: https://chromiumcodereview.appspot.com/10826048 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@149473 0039d316-1c4b-4281-b951-d872f2087c98
-
groby@chromium.org authored
This is a re-upload of patch from issue 10790087, which got reverted for a memory leak in the tests. This CL fixes the test. R=mpcomplete@chromium.org,gbillock@chromium.org BUG=130432 TEST=CWSIntentsRegistryTest.LocalizeMatchingLocale:ExtensionL10nUtil.GetAllFallbackLocales:ExtensionL10nUtil.LoadMessageCatalogs* Review URL: https://chromiumcodereview.appspot.com/10828085 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@149472 0039d316-1c4b-4281-b951-d872f2087c98
-
thestig@chromium.org authored
BUG=none TEST=none Review URL: https://chromiumcodereview.appspot.com/10831099 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@149470 0039d316-1c4b-4281-b951-d872f2087c98
-
chrome-admin@google.com authored
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@149469 0039d316-1c4b-4281-b951-d872f2087c98
-
http://codereview.chromium.org/10825089/reed@google.com authored
roll skia to 4889 Review URL: https://chromiumcodereview.appspot.com/10827122 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@149468 0039d316-1c4b-4281-b951-d872f2087c98
-
thestig@chromium.org authored
BUG=none TESt=none Review URL: https://chromiumcodereview.appspot.com/10825127 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@149467 0039d316-1c4b-4281-b951-d872f2087c98
-
fsamuel@chromium.org authored
There are a couple of broken macros that will be used by the browser plugin. BUG=none TEST=compiles (+ local testing with browser plugin) Review URL: https://chromiumcodereview.appspot.com/10829084 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@149466 0039d316-1c4b-4281-b951-d872f2087c98
-
thakis@chromium.org authored
BUG=139138 TBR=tony Review URL: https://chromiumcodereview.appspot.com/10831108 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@149465 0039d316-1c4b-4281-b951-d872f2087c98
-
dschuff@chromium.org authored
Primarily a merge of the streaming-related code into PnaclTranslationThread R=jvoung@chromium.org,sehr@chromium.org BUG=http://code.google.com/p/nativeclient/issues/detail?id=2195 TEST=nacl_integration Review URL: https://chromiumcodereview.appspot.com/10827109 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@149464 0039d316-1c4b-4281-b951-d872f2087c98
-
xiaomings@google.com authored
BUG= TBR=jam@chromium.org Review URL: https://chromiumcodereview.appspot.com/10834071 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@149463 0039d316-1c4b-4281-b951-d872f2087c98
-
satorux@chromium.org authored
This change reduces ~400 lines from gdata_file_system.cc BUG=130669 TEST=none; just move code around. should not change the behaviors TBR=ben@chromium.org for chrome_browser.gypi Review URL: https://chromiumcodereview.appspot.com/10829118 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@149462 0039d316-1c4b-4281-b951-d872f2087c98
-
rch@chromium.org authored
BUG=139700 Review URL: https://chromiumcodereview.appspot.com/10832098 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@149461 0039d316-1c4b-4281-b951-d872f2087c98
-
michaelbai@chromium.org authored
Review URL: https://chromiumcodereview.appspot.com/10821061 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@149460 0039d316-1c4b-4281-b951-d872f2087c98
-
sky@chromium.org authored
detaching. Without this the hwnd would remain parented so that if the widget containing the NativeViewHost was destroyed then the hwnd would get destroyed by virtue of being a child. Previously we had this logic in NativeTabContentsViewWin (lost in WebView refactoring), but it seems better to promote to NativeViewHostWin. BUG=132090 TEST=none R=ben@chromium.org,jam@chromium.org Review URL: https://chromiumcodereview.appspot.com/10832011 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@149459 0039d316-1c4b-4281-b951-d872f2087c98
-
gbillock@chromium.org authored
R=aa@chromium.org BUG=128964 Review URL: https://chromiumcodereview.appspot.com/10821120 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@149458 0039d316-1c4b-4281-b951-d872f2087c98
-
vandebo@chromium.org authored
Asan didn't like this: http://build.chromium.org/p/chromium.memory/builders/Linux%20ASAN%20Tests%20%282%29/builds/414/steps/media_unittests/logs/stdio - Added code to handle the signal_byte contained within WebM encrypted Blocks. - Added a unittest to aes_decryptor to hanlde an encrypted WebM Block with an unencrypted frame. BUG=139876 TEST=Run media_unittests --gtest_filter=AesDecryptor* and all tests must pass. Review URL: https://chromiumcodereview.appspot.com/10823110 TBR=fgalligan@chromium.org Review URL: https://chromiumcodereview.appspot.com/10831115 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@149457 0039d316-1c4b-4281-b951-d872f2087c98
-
thestig@chromium.org authored
BUG=none TEST=none Review URL: https://chromiumcodereview.appspot.com/10830101 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@149456 0039d316-1c4b-4281-b951-d872f2087c98
-
joi@chromium.org authored
into a DEPS file. The idea behind this change is that the tool now lets you do this: a) Update a DEPS file for some directory with the intended rules for the directory, e.g. rules disallowing certain directories (say "chrome/browser") and allowing others (e.g. "chrome/browser/apis"). b) Run this tool on that particular directory to generate a list of temp-allow rules. c) Paste the temp-allow rules into the DEPS file for the directory. d) Work on eliminating dependencies until the temp-allow rules list in the DEPS file reaches zero. To accomplish this change, and fix the bug that existed that the checkdeps tool didn't work correctly if you specified a start directory, I refactored the code to always use the approach I had introduced in CheckAddedCppIncludes, which when you request rules for a given directory generates all parent directories' rules first, up to the base directory. Also, to enable better extraction of error information for the eventual formatter, I externalized formatting to separate classes and had the code collect only the logical error information. While in there, fixed use of quotes to match coding standard. BUG=138280 Review URL: https://chromiumcodereview.appspot.com/10832062 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@149455 0039d316-1c4b-4281-b951-d872f2087c98
-
binji@chromium.org authored
BUG=none Review URL: https://chromiumcodereview.appspot.com/10836044 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@149454 0039d316-1c4b-4281-b951-d872f2087c98
-
sergeyu@chromium.org authored
Previously host was opening two separate configs, but that doesn't seem to be very useful. With that change the host still supports --auth-config option, but it no longer requires auth config and doesn't try to open auth.json by default. Review URL: https://chromiumcodereview.appspot.com/10825121 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@149453 0039d316-1c4b-4281-b951-d872f2087c98
-
dalecurtis@chromium.org authored
NullAudioSink doesn't need to worry about playback rate since AudioRendererAlgorithm already compresses/expands the data as necessary. The issue is easily seen by playing any of the sync clips and timing how fast it takes for each rotation of the "clock" in the video. Setting playback rate to 2 will actually end up being 4x as fast currently. As a bonus, dropping this allows us to remove the now unused AudioRendererSink::SetPlaybackRate() method. BUG=none (?) TEST=Play sync clip, time "clock" rotation at various speeds. Review URL: https://chromiumcodereview.appspot.com/10837032 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@149452 0039d316-1c4b-4281-b951-d872f2087c98
-
dmichael@chromium.org authored
Currently, we don't tell the HostDispatcher to only unblock pepper messages when an incoming "unblock" pepper message comes in. This results in us unblocking non-pepper messages improperly (such as GPU messages). BUG=116317 TEST= Review URL: https://chromiumcodereview.appspot.com/10843017 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@149451 0039d316-1c4b-4281-b951-d872f2087c98
-