- 12 Jan, 2017 40 commits
-
-
hychao authored
Chrome OS audio server is going to report more types of microphones, implement the UI side corresponding code to parse these types from node info. BUG=672537 Review-Url: https://codereview.chromium.org/2609863007 Cr-Commit-Position: refs/heads/master@{#443271}
-
derat authored
Change the note-taking app setting's "Waiting for Android apps..." string to a simpler "Initializing...". BUG=633596,633243 Review-Url: https://codereview.chromium.org/2628673004 Cr-Commit-Position: refs/heads/master@{#443270}
-
catapult-deps-roller authored
https://chromium.googlesource.com/external/github.com/catapult-project/catapult.git/+log/0e201cdd06ed..7f52163a4f80 $ git log 0e201cdd0..7f52163a4 --date=short --no-merges --format='%ad %ae %s' 2017-01-12 simonhatch Bisect - Average list_of_scalars in compare_samples. BUG=669608 Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+/master/autoroll/README.md If the roll is causing failures, see: http://www.chromium.org/developers/tree-sheriffs/sheriff-details-chromium#TOC-Failures-due-to-DEPS-rolls CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.android:android_optional_gpu_tests_rel TBR=catapult-sheriff@chromium.org Review-Url: https://codereview.chromium.org/2628203002 Cr-Commit-Position: refs/heads/master@{#443269}
-
geofflang authored
TBR=zmo@chromium.org BUG=668223 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;master.tryserver.chromium.android:android_optional_gpu_tests_rel Review-Url: https://codereview.chromium.org/2620133005 Cr-Commit-Position: refs/heads/master@{#443268}
-
davidben authored
Rather than have an extra copy of the data hanging off the SSLClientSocketImpl, we can extract it out of BoringSSL on demand. BUG=none Review-Url: https://codereview.chromium.org/2601493002 Cr-Commit-Position: refs/heads/master@{#443267}
-
gambard authored
BUG=679369 Review-Url: https://codereview.chromium.org/2618333005 Cr-Commit-Position: refs/heads/master@{#443266}
-
titzer authored
R=ahaas@chromium.org,jochen@chromium.org Review-Url: https://codereview.chromium.org/2622273005 Cr-Commit-Position: refs/heads/master@{#443265}
-
chrome://pluginstommycli authored
BUG=615738 CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation Review-Url: https://codereview.chromium.org/2630443002 Cr-Commit-Position: refs/heads/master@{#443264}
-
mmenke authored
TBR=clamy@chromium.org BUG=659317 Review-Url: https://codereview.chromium.org/2581393002 Cr-Commit-Position: refs/heads/master@{#443263}
-
xlai authored
Revert of Auto-rebaseline for r443157 (patchset #1 id:1 of https://codereview.chromium.org/2628173002/ ) Reason for revert: Suspected CL that cause webkit test virtual/stable/paint/invalidation/composited-overflow-with-borderbox-background.html to fail on Mac10.9 https://build.chromium.org/p/builders/WebKit%20Mac10.9/builds/41596 Original issue's description: > Auto-rebaseline for r443157 > > https://chromium.googlesource.com/chromium/src/+/13a86c2faeebc > > BUG=453414 > TBR=ktyliu@chromium.org > > Review-Url: https://codereview.chromium.org/2628173002 . > Cr-Commit-Position: refs/heads/master@{#443164} > Committed: https://chromium.googlesource.com/chromium/src/+/14f223988bfd4444a1238450919cef246d0ed51a TBR=ktyliu@chromium.org,blink-rebaseline-bot@chromium.org # Skipping CQ checks because original CL landed less than 1 days ago. NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG=453414 Review-Url: https://codereview.chromium.org/2627783007 Cr-Commit-Position: refs/heads/master@{#443262}
-
fdoray authored
The following traits are used: Priority: Inherited (default) The priority is inherited from the calling context (i.e. TaskTraits are initialized with the priority of the current task). Shutdown behavior: SKIP_ON_SHUTDOWN (default) Tasks posted with this mode that have not started executing at shutdown will never run. However, any task that has already begun executing when shutdown is invoked will be allowed to continue and will block shutdown until completion. Note: Previously, the task was posted to the blocking pool with BLOCK_SHUTDOWN (default in SequencedWorkerPool). May Block: Tasks posted with MayBlock() may block. This includes but is not limited to tasks that wait on synchronous file I/O operations: read or write a file from disk, interact with a pipe or a socket, rename or delete a file, enumerate files in a directory, etc. This trait isn't required for the mere use of locks. BUG=667892 Review-Url: https://codereview.chromium.org/2627083005 Cr-Commit-Position: refs/heads/master@{#443261}
-
tschumann authored
This is primarily to verify behavior fixed in https://codereview.chromium.org/2616633002/ but also adds tests for more fundamental behavior of the BookmarkSuggestionsProvider. This CL also moves ContentSuggestionsProvider test doubles out of specific unit tests and into separate targets so that we can reuse them. BUG=674178 Review-Url: https://codereview.chromium.org/2618243004 Cr-Commit-Position: refs/heads/master@{#443260}
-
skia-deps-roller authored
https://skia.googlesource.com/skia.git/+log/0c2997b6d85e..d32303e7276b $ git log 0c2997b6d..d32303e72 --date=short --no-merges --format='%ad %ae %s' 2017-01-10 mtklein disable runtime detected AVX2 raster pipelines 2017-01-12 djsollen Remove defunct include/images directory from GN. BUG=679147,654213,664864,666707,etc. Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+/master/autoroll/README.md If the roll is causing failures, see: http://www.chromium.org/developers/tree-sheriffs/sheriff-details-chromium#TOC-Failures-due-to-DEPS-rolls CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel TBR=kjlubick@google.com Review-Url: https://codereview.chromium.org/2633483002 Cr-Commit-Position: refs/heads/master@{#443259}
-
pdfium-deps-roller authored
https://pdfium.googlesource.com/pdfium.git/+log/ffdfbb9ec274..e7fb8eb3e9f2 $ git log ffdfbb9ec..e7fb8eb3e --date=short --no-merges --format='%ad %ae %s' 2017-01-11 npm Prevent undefined shift in putagreytile BUG=655008 Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+/master/autoroll/README.md If the roll is causing failures, see: http://www.chromium.org/developers/tree-sheriffs/sheriff-details-chromium#TOC-Failures-due-to-DEPS-rolls TBR=dsinclair@chromium.org Review-Url: https://codereview.chromium.org/2624133005 Cr-Commit-Position: refs/heads/master@{#443258}
-
sigbjornf authored
Undo r440475's temporary use of SameThreadCheckedMember<>. R= BUG=655926 Review-Url: https://codereview.chromium.org/2629613002 Cr-Commit-Position: refs/heads/master@{#443257}
-
zqzhang authored
When switching src or modifying metadata, the whole metadata will be sent to MediaSessionTabHelper causing it to download the same image src twice even if the artwork stays the same. This CL make MediaImageManager to remember the src of the last downloaded image so that it can avoid fetch the same src. BUG=672023 Review-Url: https://codereview.chromium.org/2627103002 Cr-Commit-Position: refs/heads/master@{#443256}
-
estade authored
See bug for details. tl;dr: we should not use direct links like support.google.com/chrome/answer/123456 For now this is an ignoreable upload-only prompt. BUG=679462 Review-Url: https://codereview.chromium.org/2627023003 Cr-Commit-Position: refs/heads/master@{#443255}
-
warx authored
Changes: Add one second grace period For LockScreenIfRequired. Basically, an one second OneShotTimer is used to set/reset the behavior. BUG=633304 BUG=680509 TEST=device test, and modify test coverage Review-Url: https://codereview.chromium.org/2620383003 Cr-Commit-Position: refs/heads/master@{#443254}
-
v8-autoroll authored
Summary of changes available at: https://chromium.googlesource.com/v8/v8/+log/7ca33baf..679672b1 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_trusty_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/2628893004 Cr-Commit-Position: refs/heads/master@{#443253}
-
mkwst authored
Nonces are valuable, as they allow script execution. It would be lovely if we could raise the bar on exfiltration to reduce the effectiveness of some of the attacks noted at http://sebastian-lekies.de/csp/bypasses.php. One mechanism that might be effective against some kinds of exfiltration is to stop treating the 'nonce' content attribute as the source of truth, instead pulling the nonce value into an internal slot on the HTMLScriptElement at parse-time. That prevents exfiltration via attribute leakage, mitigating the effect of vectors like `[nonce^=ab]` and `content: attr(nonce)` (http://cspnonce-test.appspot.com/exploit?reset=1 and http://sebastian-lekies.de/csp/social_engineering.php, respectively). We also clear the nonce after use ("number used _once_", right?) which mitigates the style of attack hinted at in https://sirdarckcat.github.io/csp/fakexss.html (though that specific issue is also resolved by fixing the browser bug in https://codereview.chromium.org/2618323002). Here, we're replacing the nonce content attribute with '[Replaced]', as that gives developers a hint at what's going on (e.g. in devtools), but we could pretty easily drop that in the future and just make it a devtools feature entirely. Not sure what the right thing to do is.. This prototype just effects `<script>`; once we decide on reasonable behavior, we can extend it to `<link>` and `<style>`. BUG=680419 Review-Url: https://codereview.chromium.org/2628733005 Cr-Commit-Position: refs/heads/master@{#443252}
-
xlai authored
Revert of MediaRecorder: use VideoTrackRecorder::GetPreferredCodecId() when available (patchset #5 id:250001 of https://codereview.chromium.org/2624053002/ ) Reason for revert: Suspected CL-to-blame for webkit tests failing on Mac Retina: fast/mediacapturefromelement/HTMLMediaElementCapture-capture.html fast/mediacapturefromelement/CanvasCaptureMediaStream-framerate-0.html https://build.chromium.org/p/chromium.webkit/builders/WebKit%20Mac10.11%20%28retina%29/builds/10965 Original issue's description: > MediaRecorder: use VideoTrackRecorder::GetPreferredCodecId() when available > > This CL adds a static method to VTR to query which > one, if any, is the preferred video codec. > > A new singleton Lazy Leaky class CodecEnumerator is > added to encapsulate poking the VEA to see which > codecs are supported, and to further query > a) the preferred codec id > b) if a given codec is supported > and wraps the previous CodecIdToVEAProfile() > functionality. > > BUG=679946 > > Review-Url: https://codereview.chromium.org/2624053002 > Cr-Commit-Position: refs/heads/master@{#443165} > Committed: https://chromium.googlesource.com/chromium/src/+/904b7a6f6dffa9cee5ac930c37dd591c330d14c6 TBR=emircan@chromium.org,chfremer@chromium.org,mcasas@chromium.org # Skipping CQ checks because original CL landed less than 1 days ago. NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG=679946 Review-Url: https://codereview.chromium.org/2622273006 Cr-Commit-Position: refs/heads/master@{#443251}
-
asvitkine authored
Based on post-review comments on: https://codereview.chromium.org/2558913003/ BUG=none TBR=mattm@chromium.org,donnd@chromium.org,afakhry@chromium.org,noyau@chromium.org Review-Url: https://codereview.chromium.org/2627033003 Cr-Commit-Position: refs/heads/master@{#443250}
-
siggi authored
This implements the crash handler companion to the launcher in https://codereview.chromium.org/2596463002/. The handler parses the command line as created by the launcher. After this, it can write a minidump with MinidumpWriteDump, then augment it with Crashpad metadata and massage such that Crashpad can parse it. After this, the dump is dropped into the Crashpad database for eventual upload. BUG=678959 Review-Url: https://codereview.chromium.org/2611393002 Cr-Commit-Position: refs/heads/master@{#443249}
-
sugoi authored
BUG=630728 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;master.tryserver.chromium.android:android_optional_gpu_tests_rel Review-Url: https://codereview.chromium.org/2618933002 Cr-Commit-Position: refs/heads/master@{#443248}
-
eugenebut authored
Use webState:didStartProvisionalNavigationForURL: instead. BUG=674991 Review-Url: https://codereview.chromium.org/2626733003 Cr-Commit-Position: refs/heads/master@{#443247}
-
nicholss authored
This work is to support more generic rendering for mobile platforms. The thinking behind using a stack of 'drawable' objects is we could add future features like fps stats or more animations, and also add debug screens to apps in development for gesture feedback and connection health. R=yuweih@chromium.org BUG=671692 Review-Url: https://codereview.chromium.org/2591363002 Cr-Commit-Position: refs/heads/master@{#443246}
-
msramek authored
Change the exclusive end time bound in WebHistoryService to be inclusive, as required by the history.google.com API. BUG=674165 Review-Url: https://codereview.chromium.org/2621143004 Cr-Commit-Position: refs/heads/master@{#443245}
-
scottmg authored
Part of removing dependency of Windows browser DLL on v8. R=jochen@chromium.org BUG=581766 Review-Url: https://codereview.chromium.org/2627713007 Cr-Commit-Position: refs/heads/master@{#443244}
-
bzanotti authored
The model was already being reloaded but not the the collection view, this could cause crashes as the collection view and its model were not in sync anymore. The collection view is now reloaded as well. BUG=680098 Review-Url: https://codereview.chromium.org/2622103005 Cr-Commit-Position: refs/heads/master@{#443243}
-
skia-deps-roller authored
https://skia.googlesource.com/skia.git/+log/621ea115e0b1..0c2997b6d85e $ git log 621ea115e..0c2997b6d --date=short --no-merges --format='%ad %ae %s' 2017-01-11 brianosman Remove second version of SkImageInfo2GrPixelConfig 2017-01-11 brianosman Remove kIndex_8_GrPixelConfig 2017-01-12 borenet Move recipe expectations into example.py recipes inside of modules 2017-01-12 borenet Move most recipes into modules Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+/master/autoroll/README.md If the roll is causing failures, see: http://www.chromium.org/developers/tree-sheriffs/sheriff-details-chromium#TOC-Failures-due-to-DEPS-rolls CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel TBR=kjlubick@google.com Review-Url: https://codereview.chromium.org/2630543002 Cr-Commit-Position: refs/heads/master@{#443242}
-
jlebel authored
BUG=677012 Review-Url: https://codereview.chromium.org/2617583006 Cr-Commit-Position: refs/heads/master@{#443241}
-
stkhapugin authored
Semi-autogenerated ARCMigrate commit. Notable issues: * Since one of the weak_nsobjects is exposed in a header and I change it to a __weak instead, converted two unit tests to ARC alongside this. * Fixed some assignments of newly created views to weak properties by introducing an intermediate local variable to retain the view until it's added as subview. BUG=624363 TEST=None Review-Url: https://codereview.chromium.org/2624963003 Cr-Commit-Position: refs/heads/master@{#443240}
-
sky authored
WmWindowMus and WmWindowAura only differ in a couple of ways that are better isolated in WmWindowAura rather than two separate classes. Next I'm going to rename WmWindowAura to WmWindow. BUG=671246 TEST=none R=msw@chromium.org Review-Url: https://codereview.chromium.org/2625873007 Cr-Commit-Position: refs/heads/master@{#443239}
-
sebmarchand authored
Changelog: [78fa5bc24d] SyzyAsan - Improve the performance of the heap corruption detector. BUG=675697 Review-Url: https://codereview.chromium.org/2626853004 Cr-Commit-Position: refs/heads/master@{#443238}
-
vitaliii authored
Ensure that dismissing a category cannot increase its clicks and use the previous category clicks, not the next one. Due to PassingMargin, it is possible that lower categories have more clicks. Before this CL, this could lead to dismissed category clicks increase, which was not intended. Now we take mininum of new and old clicks. In addition to this before this CL, click count was reduced based on the next category in the list, after this CL, on the previous one. BUG=678586 Review-Url: https://codereview.chromium.org/2616813003 Cr-Commit-Position: refs/heads/master@{#443237}
-
dgn authored
The RecyclerView library has a mechanism to perform partial refresh of views from the adapter via partial binds[1]. Added support for it in the NewTabPageAdapter and related classes, and rewrote the refresh of the offline badge using that. This patch also fixes an issue where the badge was not removed when an offline page was deleted Preview: https://goo.gl/photos/MKD2WXTiNqbQ17xD8 BUG=616090 Review-Url: https://codereview.chromium.org/2622793003 Cr-Commit-Position: refs/heads/master@{#443236}
-
pdfium-deps-roller authored
https://pdfium.googlesource.com/pdfium.git/+log/e8b0281d1342..ffdfbb9ec274 $ git log e8b0281d1..ffdfbb9ec --date=short --no-merges --format='%ad %ae %s' 2017-01-12 tsepez Remove unused CJS_Object::ExitInstance() method Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+/master/autoroll/README.md If the roll is causing failures, see: http://www.chromium.org/developers/tree-sheriffs/sheriff-details-chromium#TOC-Failures-due-to-DEPS-rolls TBR=dsinclair@chromium.org Review-Url: https://codereview.chromium.org/2627783005 Cr-Commit-Position: refs/heads/master@{#443235}
-
krb authored
To pick up our upstreamed optimization for suggestions, and fuzzing bug fixes. The license requirements were approved, by having README.chromium specify which license it was using. BUG=673424 Review-Url: https://codereview.chromium.org/2587363003 Cr-Commit-Position: refs/heads/master@{#443234}
-
msramek authored
In https://codereview.chromium.org/2554413002/, BrowsingDataRemover was split into content-related and embedder-specific datatypes (BrowsingDataRemover[Impl] and [Chrome]BrowsingDataRemoverDelegate, respectively). The split was done according to where the relevant data storage backends resided. This CL intends to make the split more precise by looking at it more semantically. 1. While SSLHostStateDelegate lives in content/, it's state is deleted as a part of the HISTORY datatype, which is not a content/ concept. Therefore, this was moved to ChromeBrowsingDataRemoverDelegate. 2. The DCHECK verifying that all UI entrypoints to history deletion were disabled if !|may_delete_history| was also moved to ChromeBrowsingDataRemoverDelegate, as UI is also not a content/ concept. 3. The deletion of auth cache is done in BrowsingDataRemoverDelegate for both the COOKIES and PASSWORDS datatypes. However, PASSWORDS are not a content/ concept. Therefore, BrowsingDataRemoverImpl now deletes the auth cache for COOKIES, and ChromeBrowsingDataRemoverDelegate for PASSWORDS. This is currently done by the duplication of the code, as some duplication is necessary anyway, and only about ~10 lines of code are duplicated unnecessarily; this seems to be too few to justify creating a new file in content/public to be shared between content/ and chrome/. TODO: This is the second duplication between the two classes. If this keeps happenning, consider putting the shared functionality to content/public anyway. BUG=668114 Review-Url: https://codereview.chromium.org/2617403005 Cr-Commit-Position: refs/heads/master@{#443233}
-
fdoray authored
The following traits are used: Priority: Inherited (default) The priority is inherited from the calling context (i.e. TaskTraits are initialized with the priority of the current task). Shutdown behavior: SKIP_ON_SHUTDOWN (default) Tasks posted with this mode that have not started executing at shutdown will never run. However, any task that has already begun executing when shutdown is invoked will be allowed to continue and will block shutdown until completion. Note: Previously, the task was posted to the blocking pool with BLOCK_SHUTDOWN (default in SequencedWorkerPool). May Block: Tasks posted with MayBlock() may block. This includes but is not limited to tasks that wait on synchronous file I/O operations: read or write a file from disk, interact with a pipe or a socket, rename or delete a file, enumerate files in a directory, etc. This trait isn't required for the mere use of locks. BUG=667892 Review-Url: https://codereview.chromium.org/2625193002 Cr-Commit-Position: refs/heads/master@{#443232}
-