GitLab

BUG=448859

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

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

Also check that non-heartbeat messages are 'license-request'.
Only done for unprefixed API.

BUG=442558
TEST=EME browser_tests pass

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

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

The Google style guide states that only one of {virtual,override,final}
should be used for each declaration, since override implies virtual and
final implies both virtual and override.

This patch was automatically generated with a ChromeOS build using a
variation of https://codereview.chromium.org/598073004.

BUG=417463
R=achuith@chromium.org

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

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

Summary of changes available at:
https://chromium.googlesource.com/skia/+log/04ea634..a2ef642

CQ_EXTRA_TRYBOTS=tryserver.blink:linux_blink_rel,linux_blink_dbg
TBR=djsollen@google.com

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

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

BUG=449276
R=tony@chromium.org

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

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

The Google style guide states that only one of {virtual,override,final}
should be used for each declaration, since override implies virtual and
final implies both virtual and override.

This patch was automatically generated with a ChromeOS build using a
variation of https://codereview.chromium.org/598073004.

BUG=417463
R=sky@chromium.org

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

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

The Google style guide states that only one of {virtual,override,final}
should be used for each declaration, since override implies virtual and
final implies both virtual and override.

This patch was automatically generated with a ChromeOS build using a
variation of https://codereview.chromium.org/598073004.

BUG=417463
R=jamescook@chromium.org

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

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

The Google style guide states that only one of {virtual,override,final}
should be used for each declaration, since override implies virtual and
final implies both virtual and override.

This patch was automatically generated with a ChromeOS build using a
variation of https://codereview.chromium.org/598073004.

BUG=417463
R=piman@chromium.org

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

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

Change QUIC's RTO implementation. Protected by FLAGS_quic_use_new_rto.

Decides whether to invoke SendAlgorithmInterface's
OnRetransmissionTimeout after an ack has been received instead of
pro-actively invoking it and then having to revert it.

Fix a minor QUIC bug when both quic_use_new_rto and
quic_rto_uses_last_sent were enabled, a spurious retransmission didn't
expire the SRTT.

Merge internal change: 83625311, 83966746

https://codereview.chromium.org/841603005/

Remove unused original transmission stats from QuicAckNotifier.
No behavior change.

Merge internal change: 83598791
https://codereview.chromium.org/798873005/

Removing quic version 21

Merge internal change: 83551025
https://codereview.chromium.org/786953009/

Fixes typo, changing OnSpuriousPacketRetransmition to
OnSpuriousPacketRetransmission.

Merge internal change: 83546153
https://codereview.chromium.org/837273004/

Adds an alarm for sending a QUIC FEC packet.

This CL adds an alarm for sending an FEC packet. The alarm is based on
the connection's SRTT, and is set when the first FEC protected packet in
an FEC group is sent. This alarm puts some time between the sending of
an FEC protected packet and the corresponding FEC packet, to reduce the
probability of both these packets getting dropped in the network.

Merge internal change: 83543120
https://codereview.chromium.org/839163003/

Tell the QUIC AckNotifierManager about packets on serialization, rather
than when successfully sent.
Protected by ENABLEd FLAGS_quic_ack_notifier_informed_on_serialized.

Merge internal change: 83525991
https://codereview.chromium.org/854463003/

R=rch@chromium.org
BUG=448688

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

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

The Google style guide states that only one of {virtual,override,final}
should be used for each declaration, since override implies virtual and
final implies both virtual and override.

This patch was automatically generated with a ChromeOS build using a
variation of https://codereview.chromium.org/598073004.

BUG=417463
R=zork@chromium.org

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

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

BUG=

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

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

Note: My google.com account had commit access earlier; but now I have
commit access for chromium.org account.

BUG=

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

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

BUG=none

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

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

BUG=None
TBR=jamesr@chromium.org
TBR=darin@chromium.org

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

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

Use this option in AutofillWalletPreferences. We can also use this to simplify BandwidthReductionPreferences (which is where this code was mostly copied from), but that's in the internal repo, so is a different CL.

BUG=446700

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

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

These tests are currently disabled on Linux and Windows. They were only run
on Mac with toolkit_views:

MenuViewDragAndDropTestTestInMenuDrag.TestInMenuDrag
MenuViewDragAndDropTestNestedDrag.MenuViewDragAndDropNestedDrag

This CL disables the tests on Mac. On Windows, it makes the tests disabled
instead of if-def'd out.

BUG=449058

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

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

This removes the base version that takes const char*, and replace it by the one that takes const std::string&. Consolidating this code in one place and having a single function, instead of two with the same name in two different places.

This is a redo of a patch that started in
https://codereview.chromium.org/269943004/

and of another try that was made in
https://codereview.chromium.org/834183005/.

BUG=None
TEST=base_unittests, ui_base_unittests and chrome.
R=sky@chromium.org,thakis@chromium.org

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

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

Summary of changes available at:
https://chromium.googlesource.com/chromium/blink/+log/0b09228..0db94e2

TBR=yutak@chromium.org,morrita@chromium.org

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

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

Consumers of this API would crash since Init wasn't getting called.
Also the incorrect dbus object was getting called.

BUG=441765

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

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

Android generates alternate resources when dependent on a higher sdk version then the minsdkversion.
This is conflicts with Playstore rules that only permit one resource for App Restrictions.
I have now qualified the resource to apply only for sdk > 21 where the App Restriction API was exposed.

BUG=448829
BUG=446795

TBR=bartfab@chromium.org

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

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

Two fixes for accessibility of out-of-process iframes and guest frames
(e.g. <webview>):

1. BrowserAccessibilityManager::GetFocus should return the focused node
   from a child frame.
2. BrowserAccessibilityManager::GetLocalBoundsRect and
   BrowserAccessibilityCocoa::pointInScreen need to correctly take
   child frames into account.

This is sufficient for correct spoken feedback and UI element bounding
rects in VoiceOver when tabbing into and out of an out-of-process iframe
or <webview> guest frame. There may be other bugs, but simple examples
work in VoiceOver now.

BUG=330307,368298

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

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

BUG=440818
TEST=None

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

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

The new left-nav wasn't showing up for the top-level parents. This is because there's a dependency between the selected item, and showing the selected item's child elements in the navigation.

Adding redirects is a quick solution. But we should tweak the logic for rendering the navigation to do this better. Created a bug to make this happen: https://code.google.com/p/chromium/issues/detail?id=449272

I also added the nativeMessaging article to chrome_sidenav.json so it's not an orphan page (and so it gets a left-side navigation).

BUG=447853

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

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

These are briefly broken because location is disabled on test devices,
and the mechanism for mocking out system location settings was broken.
To be fixed shortly.

BUG=449341
NOTRY=true

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

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

This is a follow-up clean up to
https://chromiumcodereview.appspot.com/842493004/
so we can:
- Use the appropriate constant to detect autocomplete="off".
- Add test case for autocomplete="off" (returns key code rather than 229).

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

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

I see it often not completing in time.

BUG=448877

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

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

Previously, this used a thread, but this did not work within a PID
namespace because /proc/<tid> can refer to a completely different
process.

BUG=312380

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

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

Note that class templates are currently ignored by all Chromium checkers; see crbug.com/441916.

BUG=303818

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

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

Revert of Revert of cc: Remove tile rasterize on demand for GPU rasterization (patchset #1 id:1 of https://codereview.chromium.org/832353004/)

Reason for revert:
this will conflict with vmpstr's change :(

Original issue's description:
> Revert of cc: Remove tile rasterize on demand for GPU rasterization (patchset #2 id:20001 of https://codereview.chromium.org/842483003/)
>
> Reason for revert:
> Speculative revert for https://code.google.com/p/chromium/issues/detail?id=449288
>
> Original issue's description:
> > cc: Remove tile rasterize on demand for GPU rasterization
> >
> > This code was recently added and isn't needed because it signals to the tile
> > that it should raster on demand (checkerboard), but this will happen anyway with
> > the delgated renderer.
> >
> > I've also removed a test that was breaking due to the change.
> >
> > The test wasn't really testing what we thought, because we wouldn't end up
> > rasterizing anything with GPU, even though force was on.
> >
> > Committed: https://crrev.com/01a4b8027c2e9e678cd6afd0a97bf83807d55f5a
> > Cr-Commit-Position: refs/heads/master@{#311109}
>
> TBR=vmpstr@chromium.org,danakj@chromium.org
> NOPRESUBMIT=true
> NOTREECHECKS=true
> NOTRY=true
>
> Committed: https://crrev.com/6bf7f22dd46c31ffcf72e48397edf3ec08533ad5
> Cr-Commit-Position: refs/heads/master@{#311764}

TBR=vmpstr@chromium.org,danakj@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true

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

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

Revert of mac: Implement custom AppKit Enter Fullscreen transition. (patchset #6 id:240001 of https://codereview.chromium.org/846443004/)

Reason for revert:
Trying to track down a sizes failure.  Speculatively reverting.  Will re-apply if not the cause.

Original issue's description:
> mac: Implement custom AppKit Enter Fullscreen transition.
>
> AppKit's default Fullscreen transition cross-fades between two static images: a
> snapshot of the window before the transition began, and a snapshot of the
> window resized to the full size of the screen. This has the problem that
> frequently, when AppKit takes the snapshot of the final state of the window,
> the web contents view has not yet drawn any content in the new size. The
> animation itself looks weird, and then there's a flash when the animation
> finishes where the snapshot of the old web content is replaced by the new web
> content.
>
> This custom fullscreen transition is very similar, except instead of taking a
> snapshot of the window in its final state, the transition uses a live version
> of the window in its final state. See the associated bug for more details about
> the implementation.
>
> BUG=414527
>
> Committed: https://crrev.com/a48cbbba0075b7a7c135605ab576bad2cfa5a1b3
> Cr-Commit-Position: refs/heads/master@{#311717}

TBR=andresantoso@chromium.org,rsesek@chromium.org,mark@chromium.org,erikchen@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=414527

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

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

Previously, the generator could only ignore changes in the:
// From FILENAME.idl modified DAY MON DATE TIME YEAR.
line if it was not wrapped or wrapped in a specific place. (Note this line
would otherwise change every time you generate).

This CL makes it a little better, in that it will ignore timestamp changes
regardless of where the "modified" output is wrapped. Otherwise, the generator
wants to update a few thunk files because it's now 2015. See:
https://codereview.chromium.org/849473003/
for context.

BUG=
R=bbudge@chromium.org

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

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

The second call to WebView::SetWebContents(..) that replaces an existing
WebContents firsts calls NativeViewHost::Detach() to remove the old
WebContents. This calls NativeViewHost::ClearFocus(), which ensures the
FocusManager knows that the WebContents (and any children) are being
removed.

For Mac the problem was that, unlike Aura, the WebContents can not own
child Widgets. GetAllChildWidgets(foo) should return |foo| and all its
children, but on Mac it was including |foo|'s parent. Since |foo|'s
parent Contain()s the WebView which has focus, FocusManager was clearing
focus. Instead, the WebView should keep focus, because its only a
(native) child of the WebView that's getting removed.

This CL adds a check to Mac's NativeWidgetPrivate::GetAllChildWidgets()
to ensure only the Widget that directly corresponds to |native_view|
(and its children) are included.

On Mac, the test also needs to ensure active status is updated so that
the TextInputClient is set.

BUG=402679

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

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

Fixes compile of views_examples_with_content_exe and app_list_demo on
Mac with toolkit_views.

BUG=417463

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

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

Revert of cc: Remove tile rasterize on demand for GPU rasterization (patchset #2 id:20001 of https://codereview.chromium.org/842483003/)

Reason for revert:
Speculative revert for https://code.google.com/p/chromium/issues/detail?id=449288

Original issue's description:
> cc: Remove tile rasterize on demand for GPU rasterization
>
> This code was recently added and isn't needed because it signals to the tile
> that it should raster on demand (checkerboard), but this will happen anyway with
> the delgated renderer.
>
> I've also removed a test that was breaking due to the change.
>
> The test wasn't really testing what we thought, because we wouldn't end up
> rasterizing anything with GPU, even though force was on.
>
> Committed: https://crrev.com/01a4b8027c2e9e678cd6afd0a97bf83807d55f5a
> Cr-Commit-Position: refs/heads/master@{#311109}

TBR=vmpstr@chromium.org,danakj@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true

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

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

The Google style guide states that only one of {virtual,override,final}
should be used for each declaration, since override implies virtual and
final implies both virtual and override.

This patch was automatically generated with a ChromeOS build using a
variation of https://codereview.chromium.org/598073004.

BUG=417463
R=thakis@chromium.org

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

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

Magic Mouse gestures don't cause -touches*WithEvent: events to be sent down the
responder chain. As such, all state related to Magic Mouse gestures needs to be
reset during -beginGestureWithEvent:.

BUG=317161

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

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

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

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

The Google style guide states that only one of {virtual,override,final}
should be used for each declaration, since override implies virtual and
final implies both virtual and override.

This patch was automatically generated with a ChromeOS build using a
variation of https://codereview.chromium.org/598073004.

BUG=417463
R=xiyuan@chromium.org

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

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

The Google style guide states that only one of {virtual,override,final}
should be used for each declaration, since override implies virtual and
final implies both virtual and override.

This patch was automatically generated with a ChromeOS build using a
variation of https://codereview.chromium.org/598073004.

BUG=417463
R=avi@chromium.org

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

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

For Non-SFI plugins, NaCl executables retrieved via the "fast path" are no
longer registered, and file tokens are no longer generated. This prevents
unnecessary work from happening in the browser, and prevents tokens from being
exposed to Non-SFI plugins.

An earlier change was attempted that prevented using the "fast path" in the
case of non-sfi plugins, but a performance regression was observed on ARC.

BUG=394130

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

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