- 18 Jun, 2014 40 commits
-
-
macourteau@chromium.org authored
Does not compile on Linux GN: http://build.chromium.org/p/chromium.linux/builders/Linux%20GN%20%28dbg%29/builds/7304/steps/compile/logs/stdio > gpu: Add base class for gpu service tests > > This allows for properly setting up and tearing down GL bindings. > Before this patch bindings were intialized in unittest_main.cc but also > reinitialized with a different GL version and extension in some > GLES2CmdDecoderTests which leaves room for problems depending on which order > tests happen to run in. > > More importantly, this will allow me to initialize GL bindings with a certain > GL version and extension bindings for some tests. > > Review URL: https://codereview.chromium.org/332303002 TBR=sievers@chromium.org Review URL: https://codereview.chromium.org/346573002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@278064 0039d316-1c4b-4281-b951-d872f2087c98
-
kelvinp@chromium.org authored
This change implements the OAuth2 implicit flow to fetch a third party token from the user. 1. Introduces a class ThirdPartyTokenFetcher to pop up a third party login page located at |tokenurl| in the browser. 2. Introduces a class OAuthRedirectActivity, which has an intent filter declared in the manifest to intercept the redirect URI upon a successful login. 3. It then starts the chromoting activity, which uses the ThirdPartyTokenFetcher to extract the code and token from the URI and pass it into the native component using JNI. BUG=329109 Review URL: https://codereview.chromium.org/337013002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@278062 0039d316-1c4b-4281-b951-d872f2087c98
-
wangxianzhu@chromium.org authored
Remove the operator to avoid misuse like the following: TRACE_EVENT1("a", "b", "c", condition ? TRACE_STR_COPY(short_live_string) : "literal"); In the above case, the argument value is actually const char*, not TraceValueWithCopy because it is automatically converted back to const char* with the const char* operator. Found no errornous usages in chromium, but found one error in WebKit. R=dsinclair BUG=382789 (perhaps the reason) Review URL: https://codereview.chromium.org/338653005 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@278061 0039d316-1c4b-4281-b951-d872f2087c98
-
jamesr@chromium.org authored
Review URL: https://codereview.chromium.org/342543002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@278060 0039d316-1c4b-4281-b951-d872f2087c98
-
sievers@chromium.org authored
This allows for properly setting up and tearing down GL bindings. Before this patch bindings were intialized in unittest_main.cc but also reinitialized with a different GL version and extension in some GLES2CmdDecoderTests which leaves room for problems depending on which order tests happen to run in. More importantly, this will allow me to initialize GL bindings with a certain GL version and extension bindings for some tests. Review URL: https://codereview.chromium.org/332303002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@278059 0039d316-1c4b-4281-b951-d872f2087c98
-
pritam.nikam@samsung.com authored
BUG=132332 Review URL: https://codereview.chromium.org/261993006 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@278058 0039d316-1c4b-4281-b951-d872f2087c98
-
zhaoqin@google.com authored
to find suitable tests running on MFYI TBR=bruening@chromium.org BUG=NONE Review URL: https://codereview.chromium.org/344583004 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@278056 0039d316-1c4b-4281-b951-d872f2087c98
-
calamity@chromium.org authored
This CL fixes an issue where the ContentsSwitcherView would dereference a pointer to a freed ContentsView. BUG=384957 Review URL: https://codereview.chromium.org/334293005 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@278054 0039d316-1c4b-4281-b951-d872f2087c98
-
sebmarchand@chromium.org authored
Strictly speaking this flag isn't here to force a deterministic build, it's here to help us to make sure that no build metadata ends up in the build artifacts to help us guarantee a deterministic build. (See the discussion in https://codereview.chromium.org/324403006/ for more details) FTR, Nico said he'll take care of getting rid of __DATE__ and __TIME__ so this new define will not be necessary for that. On the other hand we may have issues with absolute path in __FILE__ and whatever unforeseen C++ issue. This would require #ifdef'ing them out. This is not something devs would want to do normally so this flag is off by default. Eventually we'll want to fix the build system to not have to hack __FILE__ out by ensuring they are *always* relative paths but until then, this variable can be used to make progress R=maruel@chromium.org, thakis@chromium.org TBR=brettw@chromium.org BUG=314403 Review URL: https://codereview.chromium.org/330813003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@278053 0039d316-1c4b-4281-b951-d872f2087c98
-
chrome-admin@google.com authored
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@278052 0039d316-1c4b-4281-b951-d872f2087c98
-
thakis@chromium.org authored
1928: use all processors instead of hard-coded 8. 1929: Fix msvs-ninja clean build command / target. 1930: ninja: Add option for link concurrency. 1931: Add unittest for 'AR' in 'make_global_settings' 1932: Override the user's language settings during tests 1933: ninja win: Remove old comment, check cc/cxx are always overridden 1934: Use context manager to manage OS environment in tests 1935: Teach Ninja generator about 'AR' in 'make_global_settings' 1936: First crack at adding a GYP_GENERATOR for determining various things 1937: Add unittest for 'LD' in 'make_global_settings' 1938: Remove unused parameters from test code. 1939: Revise unittest for 'LD' in 'make_global_settings' 1940: Revise unittest for 'AR' in 'make_global_settings' 1941: ninja: Read 'LD'/'LD.host' from 'make_global_settings_ld'. 1942: ninja: Read 'AR.host' from 'make_global_settings'. 1943: [gyp][Android] Implement TestGypAndroid.run_built_executable. 1944: ninja: Pass .so files outside of --start-group/--end-group. BUG=362075,skia:2439,gyp:434 TBR=scottmg@chromium.org Review URL: https://codereview.chromium.org/332683003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@278051 0039d316-1c4b-4281-b951-d872f2087c98
-
isherman@chromium.org authored
Inform the adapter when a new, potentially unpaired device connects. In addition to notifying any observers of this event, this prevents a NULL pointer dereference in the existing logic. BUG=385493 TEST=Establish a Bluetooth connection from an unpaired device. chrome.bluetooth.onDeviceAdded() should be called. R=keybuk@chromium.org Review URL: https://codereview.chromium.org/341433004 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@278050 0039d316-1c4b-4281-b951-d872f2087c98
-
xiyuan@chromium.org authored
- Remove local test timeouts; - Add page flip history to PageFlipWaiter and use that for test expectations to solve the problem that a single Wait returns with multiple page flipping happened on valgrind; BUG=385167 Review URL: https://codereview.chromium.org/339933005 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@278049 0039d316-1c4b-4281-b951-d872f2087c98
-
chrome-admin@google.com authored
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@278047 0039d316-1c4b-4281-b951-d872f2087c98
-
dalecurtis@chromium.org authored
The seek timestamp should always be adjusted by the start time, though this manipulation should not be visible to external (web) clients. Since this is an FFmpeg only problem, I've removed the concept of start time from the Demuxer interface in favor of local method only for FFmpegDemuxer. FFmpegDemuxerStream's will now use this value to adjust timestamps such that external clients always see a zero based timeline. Doing so required moving some of our ogg vorbis discard code into the FFmpegDemuxerStream, which actually makes it more accurate and more narrowly scoped to ogg w/ vorbis instead of all vorbis. These changes subtly change how we handle seeking. Previously we would let FFmpeg choose the stream to perform seeking within. Now we will use the video stream only if it contains the seek timestamp, if none exists or does not contain the seek timestamp, we'll use the stream with the lowest start time. I've extended the tests around non-zero start times to verify the new behavior. An FFmpeg DEPS roll is required for the new tests to pass: 5c3de80 Pass remaining command-line arguments to ffmpeg's configure de80875 Change the sigs file to use c-style comments. cb19b2d Update ffmpeg's GN build to use new yasm assemble format. 9c12290 Revert "avformat/mp3dec: fix start time in light of initial skip samples" 1e661a6 avcodec/vorbisdec: Reset first_frame 7d88be4 avformat/oggparsevorbis: Dont attempt to calculate timestamps from gp=0 BUG=377295 TEST=New unittests. Demo page from bug works. Layout tests pass. Review URL: https://codereview.chromium.org/335273002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@278046 0039d316-1c4b-4281-b951-d872f2087c98
-
gbillock@chromium.org authored
BUG=383947 Review URL: https://codereview.chromium.org/326963002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@278045 0039d316-1c4b-4281-b951-d872f2087c98
-
estade@chromium.org authored
BUG=375545 TEST=try to edit an address to add address line 3 in chrome://settings Review URL: https://codereview.chromium.org/332213004 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@278044 0039d316-1c4b-4281-b951-d872f2087c98
-
raymes@chromium.org authored
This test was disabled because it was failing. A few changes have been made: -The page zoom cannot be assumed to be 100% on load as it will depend on the screen size being tested on. So testPageSize now doesn't make that assumption. -testAccessibilityWithPage was flaking because accessibility information was not always returned for the two text boxes. I spent a bit of time investigating, but couldn't work it out so I've left it partially disabled for now and filed crbug.com/378091. -The files have all been moved to a more appropriate location (in particular the test was moved out of the data directory). BUG=303491 Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=273819 Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=274186 Review URL: https://codereview.chromium.org/290803007 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@278043 0039d316-1c4b-4281-b951-d872f2087c98
-
thestig@chromium.org authored
Review URL: https://codereview.chromium.org/323693002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@278042 0039d316-1c4b-4281-b951-d872f2087c98
-
skuhne@chromium.org authored
BUG=- TEST=none Review URL: https://codereview.chromium.org/337423003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@278041 0039d316-1c4b-4281-b951-d872f2087c98
-
alancutter@chromium.org authored
This patch adds an optional argument to "tools/perf/run_benchmark help" to show help for a specific test_runner.py command. Review URL: https://codereview.chromium.org/309553008 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@278040 0039d316-1c4b-4281-b951-d872f2087c98
-
vollick@chromium.org authored
They're not used anymore. R=abarth@chromium.org BUG=None Review URL: https://codereview.chromium.org/333293003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@278039 0039d316-1c4b-4281-b951-d872f2087c98
-
hashimoto@chromium.org authored
TemplateURLService::fallback_search_engines_disabled only affects the behavior of DefaultSearchManager. Move it to DefaultSearchManager. BUG=371535 TEST=git cl try Review URL: https://codereview.chromium.org/340553005 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@278038 0039d316-1c4b-4281-b951-d872f2087c98
-
chrome-admin@google.com authored
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@278037 0039d316-1c4b-4281-b951-d872f2087c98
-
rouslan@chromium.org authored
This patch stops updating chrome/app/address_input_strings.grdp when running 'gclient runhooks'. This is necessary to enable checking in a static version of chrome/app/address_input_strings.grdp. https://codereview.chromium.org/337163002/ needs this patch to land. BUG=327046 Review URL: https://codereview.chromium.org/341533004 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@278036 0039d316-1c4b-4281-b951-d872f2087c98
-
holte@chromium.org authored
BUG=381325 Review URL: https://codereview.chromium.org/339003002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@278035 0039d316-1c4b-4281-b951-d872f2087c98
-
sashab@chromium.org authored
Currently, the 'about_page' API key is only enabled on dev channel. Once this is OK to launch, enable it on the stable channel. BUG=381280 Review URL: https://codereview.chromium.org/336423005 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@278034 0039d316-1c4b-4281-b951-d872f2087c98
-
chrome-admin@google.com authored
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@278032 0039d316-1c4b-4281-b951-d872f2087c98
-
dpranke@chromium.org authored
http://build.chromium.org/f/chromium/perf/dashboard/ui/changelog_blink.html?url=/trunk&range=176347:176359&mode=html TBR=schenney@chromium.org,jamesr@chromium.org BUG= Review URL: https://codereview.chromium.org/336313005 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@278031 0039d316-1c4b-4281-b951-d872f2087c98
-
bashi@chromium.org authored
We want to keep this histogram for a while. BUG=286681 Review URL: https://codereview.chromium.org/335123004 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@278030 0039d316-1c4b-4281-b951-d872f2087c98
-
jamiewalch@chromium.org authored
BUG=384696 Review URL: https://codereview.chromium.org/341733003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@278029 0039d316-1c4b-4281-b951-d872f2087c98
-
pkasting@chromium.org authored
The tabstrip now tries to ensure that if tabs would encroach on the New Tab button area or be clipped by the edge of the strip, they're made invisible instead. This prevents glitchy-looking overflow, modulo some existing bugs (on file) where the strip doesn't recalculate widths correctly. This also hides the tab next to the New Tab button if it can be shown when not selected, but might be hidden when it (or a prior tab) is selected. This prevents having this tab toggle in and out as the active tab changes. BUG=62510 TEST=Spawn lots of tabs and shrink the window to a narrow width. Once the tabs hit their min size they should start hiding at the right edge instead of drawing atop the New Tab button. Review URL: https://codereview.chromium.org/339923005 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@278028 0039d316-1c4b-4281-b951-d872f2087c98
-
michaelpg@chromium.org authored
With --enable-settings-window, About should behave the same way Settings does. BUG=385496 R=stevenjb@chromium.org,sky@chromium.org Review URL: https://codereview.chromium.org/339153005 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@278027 0039d316-1c4b-4281-b951-d872f2087c98
-
felt@chromium.org authored
Makes the background color meet minimum accessibility (contrast) requirements by deepening the shade of red. BUG=381260 Review URL: https://codereview.chromium.org/339183006 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@278026 0039d316-1c4b-4281-b951-d872f2087c98
-
perkj@chromium.org authored
Disable WebLayerImplFixedBoundsTest.CompareToWebLayerImplComplex and CompareToWebLayerImplComplex on lsan. These two tests are not new but are run under content_unittests as of r277988. https://codereview.chromium.org/317163002 BUG=386080 TBR=amogh.bihani@samsung.com, Review URL: https://codereview.chromium.org/340723004 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@278025 0039d316-1c4b-4281-b951-d872f2087c98
-
calamity@chromium.org authored
This CL is a small refactor of ContentsView::ShowSearchResults() which avoids resetting the start page before showing the search results in the experimental app list. BUG=349727 Review URL: https://codereview.chromium.org/331413003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@278024 0039d316-1c4b-4281-b951-d872f2087c98
-
caseq@chromium.org authored
This adds disabled-by-default-devtools.timeline* to the category filter to follow the new category name pattern used by timeline on the blink side. See also: https://codereview.chromium.org/337143002 BUG=381548 Review URL: https://codereview.chromium.org/334293002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@278023 0039d316-1c4b-4281-b951-d872f2087c98
-
wjywbs@gmail.com authored
R=kalman@chromium.org,meacer@chromium.org BUG=123266, 377607 Review URL: https://codereview.chromium.org/338363005 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@278022 0039d316-1c4b-4281-b951-d872f2087c98
-
felt@chromium.org authored
Changes Version3Enabled for the malware interstitial. Default is now v3 unless a flag or field trial say otherwise. BUG=381260 Review URL: https://codereview.chromium.org/331133002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@278021 0039d316-1c4b-4281-b951-d872f2087c98
-
nednguyen@google.com authored
Move telemetry/core/timeline and timeline_data to telemetry/timeline. This package handles trace parsing and it should be a standalone package. BUG= Review URL: https://codereview.chromium.org/332213003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@278020 0039d316-1c4b-4281-b951-d872f2087c98
-