- 08 Sep, 2016 40 commits
-
-
catapult-deps-roller authored
https://chromium.googlesource.com/external/github.com/catapult-project/catapult.git/+log/2c67e5988500..fe40502b6dd4 $ git log 2c67e5988..fe40502b6 --date=short --no-merges --format='%ad %ae %s' 2016-09-08 ccraik Support ANDROID_SERIAL env var as default serial 2016-09-08 alexandermont Add metric for power consumption between navigation start and TTI. TBR=catapult-sheriff@chromium.org Review-Url: https://codereview.chromium.org/2321203002 Cr-Commit-Position: refs/heads/master@{#417421}
-
msw authored
Migrate ash/wallpaper files to ash/common/. (move WallpaperController to WmShell) (move [Animating]WallpaperWidgetController to WmRootWindowController) Add WmRootWindowController wallpaper animation functions. (TODO: port system wallpaper and boot screen to mash) Move Shell::blocking_pool_ to WmShell; init for mash and tests. Make WallpaperResizer use a scoped_refptr<TaskRunner>. Move debug wallpaper shortcut to common. BUG=629605 TEST=Automated; no ChromeOS wallpaper changes. R=jamescook@chromium.org,sky@chromium.org TBR=achuith@chromium.org,bshe@chromium.org Review-Url: https://codereview.chromium.org/2318223003 Cr-Commit-Position: refs/heads/master@{#417420}
-
rsesek authored
BUG=644759 Review-Url: https://codereview.chromium.org/2313393003 Cr-Commit-Position: refs/heads/master@{#417419}
-
rjkroege authored
BUG=581462 Review-Url: https://codereview.chromium.org/2325473002 Cr-Commit-Position: refs/heads/master@{#417418}
-
dfalcantara authored
Because of timing delays, and because cancelling a Runnable doesn't guarantee that it won't run, the LoadingView can get into a state where it posts a task to animate itself showing up on screen but it doesn't re-check if it still needs to be visible. BUG=641262 Review-Url: https://codereview.chromium.org/2319213004 Cr-Commit-Position: refs/heads/master@{#417417}
-
stip authored
BUG=589168 Review-Url: https://codereview.chromium.org/2318813002 Cr-Commit-Position: refs/heads/master@{#417416}
-
https://codereview.chromium.org/2317473002/nednguyen authored
Reason for revert: This is probably not the root cause of crbug.com/643320 Original issue's description: > Revert of Roll typ to v0.9.7. (patchset #1 id:1 of https://codereview.chromium.org/2289303002/ ) > > Reason for revert: > Let's see if reverting this helps w/ crbug.com/643320. > > Original issue's description: > > Roll typ to v0.9.7. > > > > This picks up a change that will help address flaky telemetry failures. > > > > This includes the following changes (v0.9.5..v0.9.7): > > > > 79fe79db Change typ's interpretation of a test that first fails > > and is then skipped. > > 101acd31 Bump version to 0.9.6, clean up Python 3 failures > > 2f8787b8 rework run script to remove python3 log > > d5023636 rework sharding tests > > > > TBR=rnephew@chromium.org, nednguyen@google.com > > BUG=618330 > > > > Committed: https://crrev.com/71c9ade4e91e16846779311f1ae800ed550c8b69 > > Cr-Commit-Position: refs/heads/master@{#415392} > > TBR=nednguyen@google.com,rnephew@chromium.org > # Not skipping CQ checks because original CL landed more than 1 days ago. > BUG=618330, 643320 > > Committed: https://crrev.com/1023620c13385de9fcea55ec35f66dc22b454ff8 > Cr-Commit-Position: refs/heads/master@{#416573} TBR=rnephew@chromium.org,nednguyen@chromium.org,dpranke@chromium.org # Not skipping CQ checks because original CL landed more than 1 days ago. BUG=618330, 643320 Review-Url: https://codereview.chromium.org/2328753002 Cr-Commit-Position: refs/heads/master@{#417415}
-
dfalcantara authored
* Add observers to ensure that the dialog goes away when it shouldn't be kept on top of Clank. * Add tests to make sure that navigating somewhere, closing the tab, or opening a new tab dismisses the dialog. BUG=644855,629071 Review-Url: https://codereview.chromium.org/2320063003 Cr-Commit-Position: refs/heads/master@{#417414}
-
dmurph authored
Revert of Move collectGarbage* methods to ThreadState (patchset #3 id:40001 of https://codereview.chromium.org/2307003002/ ) Reason for revert: Causing weird MSAN and other issues: BUG=645227,645253 Reverting. Original issue's description: > Move collectGarbage* methods to ThreadState > > Move collectGarbage* methods to ThreadState to indicate that these run GC for the current thread heap. > > BUG= > > Committed: https://crrev.com/89d83a49a6c126d91e25e2d3520cb398e8e1f7d5 > Cr-Commit-Position: refs/heads/master@{#417269} TBR=oilpan-reviews@chromium.org,haraken@chromium.org,keishi@chromium.org # Skipping CQ checks because original CL landed less than 1 days ago. NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG= Review-Url: https://codereview.chromium.org/2321223003 Cr-Commit-Position: refs/heads/master@{#417413}
-
alokp authored
BUG=571155 Review-Url: https://codereview.chromium.org/2324483002 Cr-Commit-Position: refs/heads/master@{#417412}
-
mattreynolds authored
When Chrome is started, enable foreground scanning for Physical Web URLs. The scanner will remain enabled until Chrome is closed or the Physical Web preference is disabled, but scanning will only occur when the Chrome app is in the foreground. BUG=630769 Review-Url: https://codereview.chromium.org/2317183002 Cr-Commit-Position: refs/heads/master@{#417411}
-
cco3 authored
In the transition to use a new class for subscriptions, the background subscription got dropped because it wasn't actually run. This change actually performs the subscription. BUG=623638,645182 Review-Url: https://codereview.chromium.org/2316413002 Cr-Commit-Position: refs/heads/master@{#417410}
-
gmanikpure authored
BUG= Review-Url: https://codereview.chromium.org/2298363003 Cr-Commit-Position: refs/heads/master@{#417409}
-
boliu authored
BUG=619055 Review-Url: https://codereview.chromium.org/2326743002 Cr-Commit-Position: refs/heads/master@{#417408}
-
xlai authored
This CL enables sending compositor frame from OffscreenCanvas on worker to browser. It also makes commit() throws InvalidStateError when user call the function without calling transferControlToOffscreen. Two layout tests are added. TBR=tsepez@chromium.org BUG=563858 Review-Url: https://codereview.chromium.org/2318383002 Cr-Commit-Position: refs/heads/master@{#417407}
-
dtapuska authored
BUG=626101 Review-Url: https://codereview.chromium.org/2322073002 Cr-Commit-Position: refs/heads/master@{#417406}
-
derat authored
Make dbus::ObjectManager automatically fetch managed objects after the remote service becomes available. Also remove Bus::GetManagedObjects(), which produces a large amount of log spam due to some services not yet being ready. (It was called by chromeos::DBusThreadManager and bluez::BluezDBusManager at startup.) BUG=636554 Review-Url: https://codereview.chromium.org/2239123002 Cr-Commit-Position: refs/heads/master@{#417405}
-
iannucci authored
build: https://crrev.com/8eaa27680015cf0fdd5349a568229350f83a0c9f Roll recipe dependencies (nontrivial). (recipe-roller@chromium.org) depot_tools: https://crrev.com/1332f33d42e1a08f4acfb69f4707001ffeaf8ea9 Roll recipe dependencies (trivial). (recipe-roller@chromium.org) https://crrev.com/990ab1b42c1f347686a509aade4303289f779902 Remove force parameter from bot_update.ensure_checkout. (iannucci@chromium.org) https://crrev.com/16e7f097a7ab958c843fcc989856a47b842eb3d9 Remove bogus master/builder/slave parameters from bot_update. (iannucci@chromium.org) https://crrev.com/92ded3f29204ce9c900f9227be4f713dc18e251c Roll recipe dependencies (trivial). (recipe-roller@chromium.org) https://crrev.com/0abcd2640ae3c36165fbbc7531571a0a22294243 Fix typobug in bot_update.py. (iannucci@chromium.org) recipe_engine: https://crrev.com/4b3ff6014bcb018aff9ae2755faf47cc304f47af Track step nesting in StreamEngine. (dnj@chromium.org) https://crrev.com/339eace18cba35868b8bbb039100448fbfc58d15 Revert of Track step nesting in StreamEngine. (patchset #5 id:80001 of https://codereview.chromium.org/2245113002/ ) (dnj@chromium.org) R=agable@chromium.org, estaab@chromium.org, hinoka@chromium.org, martiniss@chromium.org BUG=643885 Review-Url: https://codereview.chromium.org/2327603002 Cr-Commit-Position: refs/heads/master@{#417404}
-
sdefresne authored
Pass required flags "-rpath @executable/Frameworks" when linking executable to point to the location where the custom frameworks are located in iOS builds. Pass flags "-objc_abi_version 2" to linker flag for execuable and dynamic libraries as is done by Xcode and remove one intermediate config object (unnecessary). BUG=None Review-Url: https://codereview.chromium.org/2326543003 Cr-Commit-Position: refs/heads/master@{#417403}
-
estade authored
BUG=626786 Review-Url: https://codereview.chromium.org/2316183003 Cr-Commit-Position: refs/heads/master@{#417402}
-
pdfium-deps-roller authored
https://pdfium.googlesource.com/pdfium.git/+log/a31da74cffa8..c29fc707b24b $ git log a31da74cf..c29fc707b --date=short --no-merges --format='%ad %ae %s' 2016-09-08 weili Fix memory management errors for font loading and copying TBR=dsinclair@chromium.org Review-Url: https://codereview.chromium.org/2324933002 Cr-Commit-Position: refs/heads/master@{#417401}
-
pdr authored
SVGImageForContainer is confusing and needs better documentation. This patch adds a class-level comment in SVGImageForContainer.h that describes the dimension and fragment url quirks that require this class. BUG=644913 NOTRY=true Review-Url: https://codereview.chromium.org/2322803002 Cr-Commit-Position: refs/heads/master@{#417400}
-
sunyunjia authored
The previous solution at https://codereview.chromium.org/2316113003/ does not check all the frames for nullptr. This patch solves the problem in a more clean and thorough way. BUG=361036 Review-Url: https://codereview.chromium.org/2322013002 Cr-Commit-Position: refs/heads/master@{#417399}
-
erikchen authored
There are two types of EventListeners: those with web_view_instance_id and those without. EventListeners with a web_view_instance_id need to always consider embedder_process_id for equality comparisons. EventListeners without a web_view_instance_id should sometimes ignore embedder_process_id (when being removed) for equality comparisons. The previous code had two major issues: 1) EventListeners were stored in a std::set. std:set requires a stable sort operator. There is no way to create a stable sort operator that sometimes ignores embedder_process_id. Aside: std::set is almost never the right container to use from a performance perspective. In this case, the code frequently needs to iterate through the entire container away, so I've changed the container to be a std::vector (we expect the total number of elements to be small). If we find performance issues, we should switch to std::unordered_set. 2) EventListeners need to be messaged asynchronously, but it's possible that the EventListener is no longer registered by the time the callback comes around. The previous code would make a copy of the EventListener, and then try to "find" the original copy in the callback. Once again, this requires clear semantics for equality. The new code makes several changes: 1) Create the class EventListener::ID. As its name implies, this uniquely identifies an EventListener. 2) Disallow copy and assignment of EventListeners. The only use case is async identity checking, which should use EventListener::ID instead. 3) This cleanup means EventListener::blocked_requests no longer has to be mutable. (A clear sign that something was wrong with the previous scheme). The most important result is that it is no longer possible to have identity confusion between EventListeners, as all comparisons use EventListener::ID. This was the root cause of the leak from Issue 643025. BUG=643025, 641958 Review-Url: https://codereview.chromium.org/2303113002 Cr-Commit-Position: refs/heads/master@{#417398}
-
cwallez authored
The origins of this workaround go waaaaay in the past and seem to contradict themselves (only OSX, vs. not OSX) BUG=590870 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/2326713002 Cr-Commit-Position: refs/heads/master@{#417397}
-
dcheng authored
This overload is deprecated: prefer the std::unique_ptr<Value> version, which helps the compiler enforce that ownership transfer occurs. BUG=581865 R=danakj@chromium.org TBR=brettw@chromium.org Review-Url: https://codereview.chromium.org/2285933003 Cr-Commit-Position: refs/heads/master@{#417396}
-
lpy authored
This patch re-enables js/mozilla/strict/13.1.html and fixes lexical redeclaration of arguments. BUG=v8:4577 Review-Url: https://codereview.chromium.org/2318933007 Cr-Commit-Position: refs/heads/master@{#417395}
-
pdr authored
This patch sets the device scale factor on cc's root transform node, fixing several layout tests. This approach is different from how cc builds property trees (see post_local_scale_factor in property_tree_builder's AddTransformNodeIfNeeded) where a post_local transform is used, but aligns closer to blink's approach of using a single transform per transform node. This approach is temporary and will need to be refactored when we begin supporting viewport transforms such as pinch-zoom. BUG=642240 CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2 Review-Url: https://codereview.chromium.org/2318933006 Cr-Commit-Position: refs/heads/master@{#417394}
-
Daniel Murphy authored
This reverts commit 9cdba144. https://codereview.chromium.org/2308153002/ BUG=645177,645181 TBR=qyearsley Previous commit message: Import wpt@65954a35b4e0a5cee1f4839271ba0e44f187c0a6 Using update-w3c-deps in Chromium c7eb73ac. Review URL: https://codereview.chromium.org/2323113002 . Committed: https://crrev.com/9cdba1441437ab946977a224cc45971cfadd8abb Cr-Original-Commit-Position: refs/heads/master@{#417302} Cr-Commit-Position: refs/heads/master@{#417393}
-
justincohen authored
This is the same as the previous toolchain (Xcode 5.1.1 with 10.10 SDK), but also includes the 10.11 SDK. BUG=474373, 626093 Review-Url: https://codereview.chromium.org/2323523003 Cr-Commit-Position: refs/heads/master@{#417392}
-
lushnikov authored
Reland of [Fetch API] Start reading blob only when the reader wants to read contents. (patchset #1 id:1 of https://codereview.chromium.org/2319093004/ ) Reason for revert: With the revert in place, the Linux MSAN is still failing: https://uberchromegw.corp.google.com/i/chromium.webkit/builders/WebKit%20Linux%20MSAN/builds/12113 This proofs this patch to be innocent. Original issue's description: > Revert of [Fetch API] Start reading blob only when the reader wants to read contents. (patchset #3 id:40001 of https://codereview.chromium.org/2303753003/ ) > > Reason for revert: > Speculative revert: this patch is a suspect to cause Linux MSAN failures: > > https://uberchromegw.corp.google.com/i/chromium.webkit/builders/WebKit%20Linux%20MSAN/builds/12091 > > The other suspected patch, the only one related to canvas, was reverted and didn't help. > > Original issue's description: > > [Fetch API] Start reading blob only when the reader wants to read contents. > > > > BUG=643174 > > > > Committed: https://crrev.com/f1ca96c413912ff1768e4274b8b7ac78eb41de64 > > Cr-Commit-Position: refs/heads/master@{#416923} > > TBR=hiroshige@chromium.org,jkarlin@chromium.org,yhirano@chromium.org > # Not skipping CQ checks because original CL landed more than 1 days ago. > BUG=643174 > > Committed: https://crrev.com/2cbd2420c74783570442264a3e9a1c76cc42152e > Cr-Commit-Position: refs/heads/master@{#417338} TBR=hiroshige@chromium.org,jkarlin@chromium.org,yhirano@chromium.org # Skipping CQ checks because original CL landed less than 1 days ago. NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG=643174 Review-Url: https://codereview.chromium.org/2327583002 Cr-Commit-Position: refs/heads/master@{#417391}
-
geofflang authored
https://chromium.googlesource.com/angle/angle.git/+log/9df01f8..c955058 BUG= TBR=jmadill@chromium.org TEST=bots CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.win:win_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.android:android_optional_gpu_tests_rel Review-Url: https://codereview.chromium.org/2320863003 Cr-Commit-Position: refs/heads/master@{#417390}
-
mvanouwerkerk authored
BUG=641412 Review-Url: https://codereview.chromium.org/2321063003 Cr-Commit-Position: refs/heads/master@{#417389}
-
fdoray authored
SequencedWorkerPool is being migrated to base/task_scheduler and there is no equivalent to IsRunningSequenceOnCurrentThread() in base/task_scheduler. Developers should use SequenceChecker to verify that tasks run sequentially. Unlike SequencedWorkerPool::IsRunningSequenceOnCurrentThread(), SequenceChecker works everywhere in Chrome (MessageLoop, SequencedWorkerPool, base/task_scheduler...). BUG=622400 Review-Url: https://codereview.chromium.org/2318123002 Cr-Commit-Position: refs/heads/master@{#417388}
-
catapult-deps-roller authored
https://chromium.googlesource.com/external/github.com/catapult-project/catapult.git/+log/8917fb02c5d6..2c67e5988500 $ git log 8917fb02c..2c67e5988 --date=short --no-merges --format='%ad %ae %s' 2016-09-08 benjhayden Disable Show all in value-set-table when all values are source values. 2016-09-08 benjhayden Move value/time_display_mode.html to base/time_display_modes.html 2016-09-08 benjhayden Linkify url strings in generic-object-view. TBR=catapult-sheriff@chromium.org Review-Url: https://codereview.chromium.org/2320233002 Cr-Commit-Position: refs/heads/master@{#417387}
-
rego authored
The grid-column-gap and grid-row-gap allow now percentage as one of the possible values. This patch adds the required support for both parsing and layout logic. This is based on a patch by Javier Fernández http://crrev.com/2057113002/. However it uses a different approach to compute the percentage gaps based on the sizing operation. During intrinsic size phase the percentage gaps are considered 0px. Also for indefinite heights the percentage row gaps are 0px too. BUG=615248 TEST=fast/css-grid-layout/grid-gutters-as-percentage.html Review-Url: https://codereview.chromium.org/2323793002 Cr-Commit-Position: refs/heads/master@{#417386}
-
mgersh authored
The comment on connect_start and connect_end incorrectly stated that DNS lookup time is included in connect time. The comment above about order of events has the correct order. Review-Url: https://codereview.chromium.org/2326693002 Cr-Commit-Position: refs/heads/master@{#417385}
-
ethannicholas authored
Review-Url: https://codereview.chromium.org/2322673003 Cr-Commit-Position: refs/heads/master@{#417384}
-
v8-autoroll authored
Summary of changes available at: https://chromium.googlesource.com/v8/v8/+log/a12446e2..0cb5bdd1 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/2324493004 Cr-Commit-Position: refs/heads/master@{#417383}
-
wychen authored
BUG=408658 Review-Url: https://codereview.chromium.org/2212663003 Cr-Commit-Position: refs/heads/master@{#417382}
-