GitLab

Summary of changes available at:
https://chromium.googlesource.com/v8/v8/+log/9ac808a6..d0987fac

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/2258423002
Cr-Commit-Position: refs/heads/master@{#413369}

BUG=633930

Review-Url: https://codereview.chromium.org/2254753002
Cr-Commit-Position: refs/heads/master@{#413368}

This patch makes the code more robust in that it adds a class which
automatically calls image draw finished on exit. This ensures that
any changes to the code (more early outs) won't forget to call that
function.

R=ericrk
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel

Review-Url: https://codereview.chromium.org/2256833006
Cr-Commit-Position: refs/heads/master@{#413367}

CreateAndConsumeTextureCHROMIUM is an odd function that doesn't really fit in
any of the models (it's a bit of a mix between "Create" and "PUT"). Rather than
doing a 1-off model, introduce in the generator a notion of an "internal"
function, that only generates a command, that is never directly exposed to the
client interfaces.
Introduce a CreateAndConsumeTextureINTERNAL that fits the existing "PUT" model,
and trivially implement CreateAndConsumeTextureCHROMIUM in terms of the former.

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/2257533007
Cr-Commit-Position: refs/heads/master@{#413366}

refer to detail spec about FlipX()[1] and FlipY()[2]

[1] = https://drafts.fxtf.org/geometry/#dom-dommatrixreadonly-flipx
[2] = https://drafts.fxtf.org/geometry/#dom-dommatrixreadonly-flipy

BUG=388780

Review-Url: https://codereview.chromium.org/2260393002
Cr-Commit-Position: refs/heads/master@{#413365}

BUG=634139
TBR=kcc@chromium.org

Review-Url: https://codereview.chromium.org/2265773002
Cr-Commit-Position: refs/heads/master@{#413364}

Cr-Commit-Position: refs/heads/master@{#413363}

This build target should never be reached on MIPS builds, but apparently it's
doing an early check to ensure the file exists anyway.

BUG=639472
TBR=bshe@chromium.org

Review-Url: https://codereview.chromium.org/2264773002
Cr-Commit-Position: refs/heads/master@{#413362}

This means we'll stay in sync with other light-on-dark surfaces
(e.g. incognito) and we won't need to manually set label colors.

BUG=638694

Review-Url: https://codereview.chromium.org/2254933002
Cr-Commit-Position: refs/heads/master@{#413361}

Range instantiation is expensive, and is not necessary for all of the uses
of the WebRange. So let's use EphemeralRanges instead.

BUG=636216
R=esprehn

Review-Url: https://codereview.chromium.org/2260283002
Cr-Commit-Position: refs/heads/master@{#413360}

BUG=634139
TBR=kcc@chromium.org

Review-Url: https://codereview.chromium.org/2265763002
Cr-Commit-Position: refs/heads/master@{#413359}

If choose not to crash, the callsite can choose to ignore the result
or early return from the function.

BUG=590856
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2

Review-Url: https://codereview.chromium.org/2259983003
Cr-Commit-Position: refs/heads/master@{#413358}

Buttons should normally be 28dp tall. Other scenarios with bigger than
usual text require adjustment to that value.

BUG=635151

Review-Url: https://codereview.chromium.org/2256403002
Cr-Commit-Position: refs/heads/master@{#413357}

Data pipe producer handles may change status while in transit
and the receiving end may never be notified of this change unless
they call MojoWait on the handle.

This fixes that, and adds test coverage to verify that the problem
does not exist and does not regress for data pipe consumers (fixed by
http://crrev.com/411707) or message pipe handles (don't currently
have this problem) either.

BUG=639363
R=yzshen@chromium.org

Review-Url: https://codereview.chromium.org/2259363002
Cr-Commit-Position: refs/heads/master@{#413356}

https://chromium.googlesource.com/native_client/src/native_client.git/+log/ed65d7fff19b..090f907ce9b3

$ git log ed65d7fff..090f907ce --date=short --no-merges --format='%ad %ae %s'
2016-08-19 ddchen Fix bugs with SPEC2k for WebAssembly

CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_chromium_dbg_32_ng,linux_nacl_sdk_build
TBR=mseaborn@chromium.org

Review-Url: https://codereview.chromium.org/2261663004
Cr-Commit-Position: refs/heads/master@{#413355}

BUG=639410,474273

Review-Url: https://codereview.chromium.org/2261833003
Cr-Commit-Position: refs/heads/master@{#413354}

This CL implements a TODO by creis@ to ensure that the renderer doesn't
classify history navigations as in-page if the document sequence number
of the current document and the one being navigated to are mismatched.

BUG=628677, 630103

Review-Url: https://codereview.chromium.org/2253233002
Cr-Commit-Position: refs/heads/master@{#413353}

https://chromium.googlesource.com/external/github.com/catapult-project/catapult.git/+log/30ce231266c6..cda4e72f8776

$ git log 30ce23126..cda4e72f8 --date=short --no-merges --format='%ad %ae %s'

BUG=639632

TBR=catapult-sheriff@chromium.org

Review-Url: https://codereview.chromium.org/2263883002
Cr-Commit-Position: refs/heads/master@{#413352}

Summary of changes available at:
https://chromium.googlesource.com/v8/v8/+log/8e5f0c15..9ac808a6

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/2261963002
Cr-Commit-Position: refs/heads/master@{#413351}

Summary of changes available at:
https://chromium.googlesource.com/v8/v8/+log/46e45e95..8e5f0c15

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/2262933002
Cr-Commit-Position: refs/heads/master@{#413350}

This reverts commit 936aa9ea.

Highly speculative revert for https://crbug.com/639618. If this doesn't
make things more green I'll reland, in which case I apologize for the
noise.

NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=639618
TBR=chili@chromium.org

Review-Url: https://codereview.chromium.org/2264873002
Cr-Commit-Position: refs/heads/master@{#413349}

https://chromium.googlesource.com/skia.git/+log/589a39eb81d2..deb8136c9ce4

$ git log 589a39eb8..deb8136c9 --date=short --no-merges --format='%ad %ae %s'
2016-08-20 reed remove unused field from SkDrawIter

CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel
TBR=robertphillips@google.com

Review-Url: https://codereview.chromium.org/2262953002
Cr-Commit-Position: refs/heads/master@{#413348}

This patch switches from a test-only GeometryPrinters and debug-only
toString() to a common pretty printer format for logging and testing.

For example, to print the value of a LayoutRect:
    LayoutRect overflowRect(1, 2, 3, 4);
    LOG(INFO) << "overflow: " << overflowRect.toString();
Which prints:
    overflow: "1,2 3x4"

In tests, gtests will automatically call the PrintTo functions. In
debuggers, toString() should work as expected.

Minor changes:
1) Switched floating point precision to use "%lg" which prints up to
6 significant figures, and fewer if the value is an integer.
2) Added special-cases to catch LayoutUnit's min/max/nearly{Min,Max}.

BUG=632096

Review-Url: https://codereview.chromium.org/2191233002
Cr-Commit-Position: refs/heads/master@{#413347}

The experiment is controlled by a static call for the entire process
which must be made before any SequencedWorkerPool instance is posted
a task (i.e. before it creates a Worker).

The experiment being at run-time, sequenced_worker_pool.cc is now divided
between the two worlds. DCHECKs were added to associate as many methods as
possible to the world they belong to.

FYI, the experiment (and TEST= below) will not actually work until
https://codereview.chromium.org/2237643002/ or a variant of it lands
but this is a stepping stone to getting there and the CLs
themselves can land in any order.

BUG=622400
TEST=
A) "out\Debug\chrome.exe" runs without a problem, BlockingPool tasks execute as normal in chrome://tracing
B) "out\Debug\chrome --force-fieldtrials=BrowserScheduler/Enabled/ --force-fieldtrial-params=BrowserScheduler.Enabled:Background/3;3;1;0;10000/BackgroundFileIO/3;3;1;0;10000/Foreground/3;3;1;0;10000/ForegroundFileIO/3;3;1;0;10000/RedirectSequencedWorkerPools/true"
runs without a problem and chrome://tracing shows BlockingPool/CachePool/etc. tasks in the TaskScheduler workers :-)

Review-Url: https://codereview.chromium.org/2241703002
Cr-Commit-Position: refs/heads/master@{#413346}

We don't really make use of of the special properties of SMILTime for
this case, and using double means slightly less impedance mismatching.

BUG=631879

Review-Url: https://codereview.chromium.org/2261443002
Cr-Commit-Position: refs/heads/master@{#413345}

This CL reimplements AllocateNearTo on 64bit so that it searches for a
free memory range rather than the naive brute force approach we used
before.

BUG=604149
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.win:win10_chromium_x64_rel_ng

Review-Url: https://codereview.chromium.org/2258583002
Cr-Commit-Position: refs/heads/master@{#413344}

With RLS enabled, paint property trees will not use any properties
set on the FrameView; instead, all properties will come from the
LayoutView.

BUG=490942
R=pdr@chromium.org,chrishtr@chromium.org,skobes@chromium.org
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2

Review-Url: https://codereview.chromium.org/2244873002
Cr-Commit-Position: refs/heads/master@{#413343}

https://chromium.googlesource.com/chromium/src/+/ff27367de

BUG=638611
TBR=wkorman@chromium.org

Review URL: https://codereview.chromium.org/2254733014 .

Cr-Commit-Position: refs/heads/master@{#413342}

https://chromium.googlesource.com/skia.git/+log/e2348ccb477b..589a39eb81d2

$ git log e2348ccb4..589a39eb8 --date=short --no-merges --format='%ad %ae %s'
2016-08-20 reed store info in basedevice, change getter to non-virtual const&

CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel
TBR=robertphillips@google.com

Review-Url: https://codereview.chromium.org/2265743002
Cr-Commit-Position: refs/heads/master@{#413341}

Use the scrolling contents GraphicsLayer as the DisplayItemClient for the
background drawing when we're painting the full background of a composited
scroller.

BUG=638611
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2

Review-Url: https://codereview.chromium.org/2254893005
Cr-Commit-Position: refs/heads/master@{#413340}

Revert of Fix MediaRouterContextualMenuUnitTest (patchset #1 id:1 of https://codereview.chromium.org/2253613003/ )

Reason for revert:
Reverting this CL as well as the other CL[1] this was meant to fix. They are causing crashes and need a redesign.

[1] https://codereview.chromium.org/2155293002

Original issue's description:
> Fix MediaRouterContextualMenuUnitTest
>
> This CL resolves the compliation error in the unit test introduced
> by https://codereview.chromium.org/2155293002.
>
> TBR=msw@chromium.org
> BUG=638300
>
> Committed: https://crrev.com/0d3d4235ebcd37fa56b05517c9d97c018e8c0f5f
> Cr-Commit-Position: refs/heads/master@{#412332}

TBR=msw@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=638300

Review-Url: https://codereview.chromium.org/2260873003
Cr-Commit-Position: refs/heads/master@{#413339}

Revert of Show the Cast toolbar icon ephemerally when Cast is in use (patchset #15 id:440001 of https://codereview.chromium.org/2155293002/ )

Reason for revert:
Reverting this CL since it is causing crashes and needs a redesign. Also reverting a related CL: https://codereview.chromium.org/2260873003/

Original issue's description:
> Show the Cast toolbar icon ephemerally when Cast is in use
>
> This CL changes the behavior of the Cast/Media Router toolbar component
> action icon. Currently, the only way to add the icon is by installing
> the Cast extension from the Web Store. After this change, the icon will
> be displayed temporarily whenever the Media Router dialog is open, or there
> is an active local media route.
>
> This CL also replaces the "Remove icon" option in the action context menu
> with "Always show icon" which can be checked. When this option is unchecked,
> the icon is neither on the toolbar nor in the overflow menu, and is only shown
> ephemerally in the situations mentioned above.
>
> We replace the word "popup" with "dialog" wherever possible in
> MediaRouterAction, to stay consistent with the rest of Media Router files.
>
> We also change the default position of the component actions on the toolbar
> from the leftmost to the rightmost (same behavior as extension actions).
>
> TBR=grt@chromium.org
> BUG=594577
>
> Committed: https://crrev.com/de716ca8b6cffb4c7ae30990b3288b3499d28a4c
> Cr-Commit-Position: refs/heads/master@{#412280}

TBR=anthonyvd@chromium.org,apacible@chromium.org,msw@chromium.org,rdevlin.cronin@chromium.org,mfoltz@chromium.org,grt@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=594577

Review-Url: https://codereview.chromium.org/2260343002
Cr-Commit-Position: refs/heads/master@{#413338}

BUG=636558,637866
R=dcheng@chromium.org, dpranke@chromium.org, inferno@chromium.org, rnk@chromium.org

Review URL: https://codereview.chromium.org/2241413003 .

Cr-Commit-Position: refs/heads/master@{#413337}

Before the timeline has started we shouldn't update animations. This
makes resume() symmetric with pause().

BUG=631879

Review-Url: https://codereview.chromium.org/2254303005
Cr-Commit-Position: refs/heads/master@{#413336}

BUG=catapult:#2584
TBR=perejzu@chromium.org

Review-Url: https://codereview.chromium.org/2265713002
Cr-Commit-Position: refs/heads/master@{#413335}

https://chromium.googlesource.com/external/github.com/catapult-project/catapult.git/+log/04dd47e541e2..30ce231266c6

$ git log 04dd47e54..30ce23126 --date=short --no-merges --format='%ad %ae %s'

TBR=catapult-sheriff@chromium.org

Review-Url: https://codereview.chromium.org/2259193005
Cr-Commit-Position: refs/heads/master@{#413334}

https://pdfium.googlesource.com/pdfium.git/+log/5b13e1dc5770..9b777deb00fb

$ git log 5b13e1dc5..9b777deb0 --date=short --no-merges --format='%ad %ae %s'
2016-08-19 weili Fix an embedder test with leaked page object
2016-08-19 npm Move CFX_FaceCache to its own file
2016-08-19 tsepez Avoid signed overflow in  CPDF_ToUnicodeMap::StringToCode()
2016-08-19 tsepez Introduce pdfium::FakeUniquePtr for keys to sets of unique ptrs.
2016-08-19 weili Fix leaked array buffer allocators of isolates

BUG=638489

TBR=dsinclair@chromium.org

Review-Url: https://codereview.chromium.org/2265733002
Cr-Commit-Position: refs/heads/master@{#413333}

The CL makes CPUProfile encode sample timestamps as deltas, which makes the
representation more compact.
startTime and endTime are represented in microseconds rather than seconds.

BUG=635947

Review-Url: https://codereview.chromium.org/2262543002
Cr-Commit-Position: refs/heads/master@{#413332}

Summary of changes available at:
https://chromium.googlesource.com/v8/v8/+log/fab9cb11..46e45e95

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/2264843002
Cr-Commit-Position: refs/heads/master@{#413331}

HitRegion is need to many information to expand(ie. cursor, parent_id, label, etc) in canvas2d.
"std::pair<element*, string>" can't have a lot of information.
so HitTestCanvasResult was added to expend.

BUG=592947

Review-Url: https://codereview.chromium.org/2229743003
Cr-Commit-Position: refs/heads/master@{#413330}