- 27 Sep, 2016 40 commits
-
-
servolk authored
This is necessary to fix build break for OEMs that include decoder_config.h, but don't use decoder_config_adapter.cc Review-Url: https://codereview.chromium.org/2372283002 Cr-Commit-Position: refs/heads/master@{#421316}
-
skia-deps-roller authored
https://chromium.googlesource.com/skia.git/+log/3ba96c8cecd8..1f4a874addd7 $ git log 3ba96c8ce..1f4a874ad --date=short --no-merges --format='%ad %ae %s' 2016-09-27 mtklein SkRasterPipeline: add last() and docs. CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel TBR=fmalita@google.com Review-Url: https://codereview.chromium.org/2376803002 Cr-Commit-Position: refs/heads/master@{#421315}
-
xunjieli authored
This CL adds a net::GzipSourceStream which implements net::FilterSourceStream to do gzip decoding. This is a part of the efforts to convert net::Filter into a pull-based interface. See the linked bug for more details. BUG=474859 Review-Url: https://codereview.chromium.org/2334773002 Cr-Commit-Position: refs/heads/master@{#421314}
-
fdoray authored
MessageLoop::QuitWhenIdle() is deprecated. RunLoop should be used instead. BUG=616447 Review-Url: https://codereview.chromium.org/2373033002 Cr-Commit-Position: refs/heads/master@{#421313}
-
mtrofin authored
Note that this CL depends on a V8 CL that implements binary version 0xC for WASM, which will land momentarily. R=titzer@chromium.org,jochen@chromium.org BUG=chromium:575167, chromium:650655 Review-Url: https://codereview.chromium.org/2370963003 Cr-Commit-Position: refs/heads/master@{#421312}
-
chrisha authored
This prevents the module enumeration from running repeatedly if the user revisits chrome://conflicts. BUG=617176 Review-Url: https://codereview.chromium.org/2370053002 Cr-Commit-Position: refs/heads/master@{#421311}
-
rch authored
set_response_listener from taking a pointer to a unique_ptr. Merge internal change: 134417781 Review-Url: https://codereview.chromium.org/2369863004 Cr-Commit-Position: refs/heads/master@{#421310}
-
yusukes authored
BUG=None TEST=try Review-Url: https://codereview.chromium.org/2372133002 Cr-Commit-Position: refs/heads/master@{#421309}
-
qyearsley authored
BUG=643937 Review-Url: https://codereview.chromium.org/2373063002 Cr-Commit-Position: refs/heads/master@{#421308}
-
jam authored
BUG=504347 Review-Url: https://codereview.chromium.org/2376763002 Cr-Commit-Position: refs/heads/master@{#421307}
-
mef authored
Review-Url: https://codereview.chromium.org/2369933006 Cr-Commit-Position: refs/heads/master@{#421306}
-
krasin authored
There're some issues with javascript in 3 tests cases. It's unlikely to be a ThinLTO issue, so disabling it for now. BUG=650718 Review-Url: https://codereview.chromium.org/2377713002 Cr-Commit-Position: refs/heads/master@{#421305}
-
eranm authored
Do not de-reference the cert_transparency_verifier_ field during destruction if it was not initialized, as that would lead to a crash. The original assumption was that ProfileIOData::Init() is always called so the cert_transparency_verifier_ is always initialized. However that assumption proved to be wrong - ProfileIOData::Init is only called once a ChromeURLRequestContextFactory is created with the ProfileIOData instance and a URLRequestContext is created using the Create method (see: https://cs.chromium.org/chromium/src/chrome/browser/net/chrome_url_request_context_getter.cc?sq=package:chromium&dr=C&rcl=1474946434&l=55) If that did not happen and ProfileIOData::Init was not called, de-referencing cert_transparency_verifier_ during destruction would lead to a crash. BUG=648507 Review-Url: https://codereview.chromium.org/2373753002 Cr-Commit-Position: refs/heads/master@{#421304}
-
corona10 authored
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_site_isolation Review-Url: https://codereview.chromium.org/2375583002 Cr-Commit-Position: refs/heads/master@{#421303}
-
mef authored
BUG=650782 Review-Url: https://codereview.chromium.org/2369053002 Cr-Commit-Position: refs/heads/master@{#421302}
-
caseq authored
This removes platform-specific hooks from extension API generation, as these were only relevant in WebKit. Review-Url: https://codereview.chromium.org/2374573002 Cr-Commit-Position: refs/heads/master@{#421301}
-
calvimei authored
TouchActionBrowserTest.DefaultAuto is flaky on devices which have a longer display than other devices. The test could not scroll the full height of the display, so we increase the scroll height. BUG=648755 Review-Url: https://codereview.chromium.org/2358673002 Cr-Commit-Position: refs/heads/master@{#421300}
-
bmcquade authored
BUG=647764 Review-Url: https://codereview.chromium.org/2372573005 Cr-Commit-Position: refs/heads/master@{#421299}
-
sky authored
The code was assuming window was this->window_, which is not necessarily the case. BUG=650755 TEST=none R=oshima@chromium.org Review-Url: https://codereview.chromium.org/2369413002 Cr-Commit-Position: refs/heads/master@{#421298}
-
phajdan.jr authored
extensions/browser/api/cast_channel/logger.cc includes "third_party/zlib/zlib.h" and so the target should depend on zlib. BUG=none Review-Url: https://codereview.chromium.org/2362383003 Cr-Commit-Position: refs/heads/master@{#421297}
-
sorin authored
The current implementation of base::SysInfo().OperatingSystemVersion() for Windows joins the version of the OS with some representation of the Service Pack. That created a server configuration issue in the past where a rule had been written assuming a dotted version string for the OS. Since then, the authors of configuration files have learned to write rules that consider the possibility that OS versions may not be dotted strings. Traditionally, Omaha expected the OS version and Service Pack as serparate entities in the Omaha protocol. Other platforms but Windows and Linux use dotted version strings currently. They have no Service Packs. The Linux does not provide an implementation for base::SysInfo::OperatingSystemVersionNumbers(). Linux versions could include relevant information that does not fit the typical dotted string version. In conclusion, this change is a minor improvement for the Windows OS metadata and leaves the rest, including Linux, the same as before. BUG=642476 Review-Url: https://codereview.chromium.org/2365523004 Cr-Commit-Position: refs/heads/master@{#421296}
-
v8-autoroll authored
Summary of changes available at: https://chromium.googlesource.com/v8/v8/+log/6888cbc1..f2767998 Please follow these instructions for assigning/CC'ing issues: https://github.com/v8/v8/wiki/Triaging%20issues Please close rolling in case of a roll revert: https://v8-roll.appspot.com/ This only works with a Google account. CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel;master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel TBR=hablich@chromium.org,machenbach@chromium.org,littledan@chromium.org,vogelheim@chromium.org Review-Url: https://codereview.chromium.org/2372153003 Cr-Commit-Position: refs/heads/master@{#421295}
-
catapult-deps-roller authored
https://chromium.googlesource.com/external/github.com/catapult-project/catapult.git/+log/384efea3161c..8410eebd6f4c $ git log 384efea31..8410eebd6 --date=short --no-merges --format='%ad %ae %s' 2016-09-27 benjhayden CC current sheriff instead of assigning them to perf regression bugs. CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.android:android_optional_gpu_tests_rel TBR=catapult-sheriff@chromium.org Review-Url: https://codereview.chromium.org/2375683002 Cr-Commit-Position: refs/heads/master@{#421294}
-
puthik authored
Currently ARC++ BT just supported only discoverable timeout for set_adapter_property to pass the CTS test. This CL finishes the implementation of that and refactor the current code to use common callback function. BUG=646584 TEST=Build/Unit test/CTS Review-Url: https://codereview.chromium.org/2364443002 Cr-Commit-Position: refs/heads/master@{#421293}
-
kerrnel authored
path, is not reliable. Instead, this adds a central check to the sandbox to see if filesystem access is available (by trying to stat() /proc). BUG=chromium:630420 Review-Url: https://codereview.chromium.org/2357393003 Cr-Commit-Position: refs/heads/master@{#421292}
-
wolenetz authored
Previously, an "unknown duration" signalled by a mvhd duration field was only understood to be unknown if it were 0, or a 64-bit (version 1 mvhd) special value of all bits set. This change adds the same special value handling for a 32-bit (version 0 mvhd). This change corrects detection of unknown mp4 MSE durations in some cases. Unknown duration is web-app-visible and changes MSE behavior. Also, it's used as a signal to enable our experimental low-delay video rendering path. I used a hex editor create the new test media file from bear-1280x720-av_frag.mp4: truncated the file to just the initialization segment (~1.5k, compared to full original file ~750k), and modified the mvhd duration field values to be all bits set instead of 0. The new unit test fails (due the the max-uint32 value being interpreted as a real duration) before the rest of the code changes were included into this CL. TEST=MP4StreamParserTest.UnknownDuration_V0_AllBitsSet BUG=649882 R=xhwang@chromium.org Review-Url: https://codereview.chromium.org/2368773002 Cr-Commit-Position: refs/heads/master@{#421291}
-
xianglu authored
This CL introduces the basic structure of Shape Detection API, including four classes: Detector, FaceDetector, DetectedObject, DetectedFace (dictionaries are not yet implemented) and adds simple layout tests in shapedetection-creation.html to verify corresponding constructors. R=mcasas@chromium.org BUG=646083 Review-Url: https://codereview.chromium.org/2360783003 Cr-Commit-Position: refs/heads/master@{#421290}
-
rch authored
Merge internal change: 134339524 Review-Url: https://codereview.chromium.org/2368183003 Cr-Commit-Position: refs/heads/master@{#421289}
-
sadrul authored
BUG=none CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel Review-Url: https://codereview.chromium.org/2369593002 Cr-Commit-Position: refs/heads/master@{#421288}
-
lukasza authored
Before this CL, blocking of top-level navigations in extension pop-ups was accomplished by routing them via OpenURL path and blocking (or rather - silently dropping) CURRENT_TAB navigations via ExtensionViewHost::OpenURLFromTab. This was problematic for a few reasons: 1. This unnecessarily blocked navigations that end-up being treated as downloads (i.e. because HTTP response says Content-Disposition: attachment). This was the root cause of the regression raised in https://crbug.com/646261 2. There are still some remaining issues in handling of POST requests via OpenURL path (e.g. dropping Content-Type header - https://crbug.com/648648). 3. In the long-term we want to rely less on process isolation accomplished via OpenURL - an exploited renderer process does not necessarily have to go through OpenURL path and can instead choose to use the regular, renderer-initiated path. After this CL: 1. ExtensionViewHost::ShouldTransferNavigation is used to block top-level navigations in extension pop-ups (and background pages). 2. POST navigations do not go through OpenURL path anymore (i.e. this CL effectively reverts the essence of r407586). In the long-term we want to make all extension navigations to not go through OpenURL path, but this seems too risky to merge back to M54, so for now we only do #1 above (i.e. avoid OpenURL only for POST requests). BUG=646261 CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_site_isolation Review-Url: https://codereview.chromium.org/2352083003 Cr-Commit-Position: refs/heads/master@{#421287}
-
markellis authored
libosmesa.so is referenced in blimp/engine/browser_tests/integration_test.cc:68 This is a temporary workaround for crbug.com/650285 Also, add tests-manifest-blacklist.txt as an input to generate_blimp_engine_env_tests_manifest, so that when the blacklist changes the test bundle is re-generated. R=dtrainor@chromium.org BUG=650285 Review-Url: https://codereview.chromium.org/2366973003 Cr-Commit-Position: refs/heads/master@{#421286}
-
skia-deps-roller authored
https://chromium.googlesource.com/skia.git/+log/53c7bbfaaf80..3ba96c8cecd8 $ git log 53c7bbfaa..3ba96c8ce --date=short --no-merges --format='%ad %ae %s' 2016-09-27 mtklein Turn on -Wgnu-anonymous-struct and -Wnested-anonymous-types. 2016-09-27 egdaniel Revert of Add command buffer submits before copy calls in vulkan. (patchset #5 id:80001 of https://codereview.chromium.org/2363773006/ ) CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel TBR=fmalita@google.com Review-Url: https://codereview.chromium.org/2369753005 Cr-Commit-Position: refs/heads/master@{#421285}
-
recipe-roller authored
This is an automated CL created by the recipe roller. This CL rolls recipe changes from upstream projects (e.g. depot_tools) into downstream projects (e.g. tools/build). More info is at https://goo.gl/zkKdpD. Use https://goo.gl/noib3a to file a bug (or complain) build: https://crrev.com/14c8f0a44c13e0d91bae2ac4ca74acfd0aa084a8 Remove the download_android_tools step (jsimmons@google.com) TBR=martiniss@chromium.org,phajdan.jr@chromium.org BUG= Recipe-Tryjob-Bypass-Reason: Autoroller Bugdroid-Send-Email: False Review-Url: https://codereview.chromium.org/2368413004 Cr-Commit-Position: refs/heads/master@{#421284}
-
mdjones authored
When loading a resource, it is possible that it will not have a bitmap (OOM cases for example). This change checks that the bitmap exists in the resource before calling native OnResourceReady. BUG=650361 Review-Url: https://codereview.chromium.org/2368373003 Cr-Commit-Position: refs/heads/master@{#421283}
-
pilgrim authored
Direct access to LayoutPart is deprecated and being removed soon. The replacement is LayoutPartItem, which routes methods through the new block layout API. (See linked crbug for details.) This means the ownerLayoutObject() method is also deprecated, to be replaced with ownerLayoutItem(). There are no functional changes. BUG=499321 TBR=eae@chromium.org Review-Url: https://codereview.chromium.org/2374613003 Cr-Commit-Position: refs/heads/master@{#421282}
-
derat authored
Add a metric to measure clicks on the "Annotate" button in screenshot notifications. Also list additional owners for DownloadNotification.Button_Annotate. BUG=650497 TEST=manual: chrome://user-actions shows Screenshot_Annotate action after clicking button Review-Url: https://codereview.chromium.org/2376553002 Cr-Commit-Position: refs/heads/master@{#421281}
-
hans authored
BUG=649539 Review-Url: https://codereview.chromium.org/2371953002 Cr-Commit-Position: refs/heads/master@{#421280}
-
donnd authored
Refactoring done in that CL looks like it could introduce potential problems because now there's a path through the affected method that does not reset the context. This CL refactors the affected method to always reset the context while preserving the refactoring done in 2157793002. BUG=626770 Review-Url: https://codereview.chromium.org/2300753003 Cr-Commit-Position: refs/heads/master@{#421279}
-
tedchoc authored
BUG=649874 Review-Url: https://codereview.chromium.org/2359363004 Cr-Commit-Position: refs/heads/master@{#421278}
-
japhet authored
https://html.spec.whatwg.org/multipage/semantics.html#hyperlink-auditing part 4 states that a Ping-From header should only be sent for same origin requests or if the document containing the hyperlink is unencrypted. I think this was regressed accidentally in https://trac.webkit.org/changeset/91306 BUG=637459 TEST=PingLoaderTest.HTTPSToHTTPS Review-Url: https://codereview.chromium.org/2360753002 Cr-Commit-Position: refs/heads/master@{#421277}
-