GitLab

BUG=514340

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

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

BUG=528940
R=skobes@chromium.org,pdr@chromium.org

Before this change:

https://codereview.chromium.org/1295933003

... blocks with overflow:auto would delay updating their scrollbars until
after all flex layout was finished.  After the change, scrollbar info is
updated immediately during the course of layout.  Flex items may run
layout multiple times during flex layout; if a flex item has auto scrollbars,
it may create and destroy its scrollbars multiple times.

Aside from being a performance problem, this can cause WebScrollbarThemePainter
to point to a stale Scrollbar object.  If a flex item has scrollbars prior to
layout; then the flex item destroys and creates scrollbars during layout; and
at the end of layout, it still has a scrollbar; then
CompositedDeprecatedPaintLayerMapping::updateOverflowControlsLayers will not
update the WebScrollbarThemePainter with the final Scrollbar object.

We could fix this in updateOverflowControlsLayers, but that wouldn't address
the performance issue of needlessly creating and destroying scrollbars during
flex layout.  This patch avoids destroying scrollbars that are no longer
deemed necessary, until after all flexing is finished.

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

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

Revert of exclude PhishingDOMFeatureExtractorTest.* browser_tests on Dr. Memory bots (patchset #1 id:1 of https://codereview.chromium.org/1371193003/ )

Reason for revert:
I'm reverting the relevant CL, https://codereview.chromium.org/1373993002/

so this exclusion is no longer necessary.

Original issue's description:
> exclude PhishingDOMFeatureExtractorTest.* browser_tests on Dr. Memory bots
>
> TBR=erikchen@chromium.org
> BUG=536891
> NOTRY=true
>
> Committed: https://crrev.com/2b4bfec5eab7ab8986557934ef9bd27e87c8e22b
> Cr-Commit-Position: refs/heads/master@{#351107}

TBR=zhaoqin@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=536891

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

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

Revert of ipc: Change the timing of the construction of the attachment broker. (patchset #3 id:40001 of https://codereview.chromium.org/1367013003/ )

Reason for revert:
Still causes problems in at least one Dr. Memory test. https://code.google.com/p/chromium/issues/detail?id=536891
Requires further investigation - Reverting for now

Original issue's description:
> ipc: Change the timing of the construction of the attachment broker.
>
> This CL ensures that the privileged attachment broker is constructed earlier in
> the process of creating a child process. This fixes a race condition for single
> process mode where the child thread's attachment broker would become the global
> attachment broker.
>
> BUG=534539
>
> Committed: https://crrev.com/8758b4bd5bd210552606fa39d084aef7e786abd1
> Cr-Commit-Position: refs/heads/master@{#350899}

TBR=avi@chromium.org,tsepez@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=534539

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

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

Switch libvpx from static_library to none
https://codereview.chromium.org/1365973002

R=blundell@chromium.org,tomfinegan@chromium.org
BUG=534943

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

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

These flags were getting large and out of control, so it's nice to separate them from the generic compiler stuff. Also, this allows us to duplicate the GYP state where the sanitizer flags are not set on the sanitizer options helper code itself.

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

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

TBR=jsbell@chromium.org
BUG=536881
NOTRY=true

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

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

This is fixed in ToT (was fixed in M46), so remove the workaround.

BUG=chrome-os-partner:36554

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

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

BUG=535125

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

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

TBR=creis@chromium.org
BUG=536907
NOTRY=true

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

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

BUG=536293

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

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

For media_stream_audio example, add 'audioCapture' permission
For media_stream_video example, add 'videoCapture' permission

Missing these permissions will cause NavigatorUserMediaError

BUG=NONE
TEST=Build the app and load it into Chrome, you should see it's working.

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

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

Bookmarked origins still have to request the permission but will get it
without any user interaction.

BUG=533197

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

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

Added maxWidth:screen.width and maxHeight:screen.height constrains for video in webkitGetUserMedia().Therefore, the stream may have higher resolution than 640*480 which is default in webkitGetUserMedia().

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

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

Update the ARC 'getting started' doc to include information about
32-bit x86 targets and ARC.

BUG=None
R=mkearney@chromium.org

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

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

Page scale factor is updated when a main frame is committed. But that is
not enough because when WebView is invisible, there won't be main frame
commits.

In order to get page scale factor updates even when WebView is
invisible, check page scale factor change when the layout is updated.

BUG=520089

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

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

BUG=528228
TEST=none

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

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

BUG=512643

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

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

Remove FaviconHandler::PageChangedSinceFaviconWasRequested() since both
- FaviconHandler::url_ changes (FaviconHandler::FetchFavicon())
- The NavigationController's last committed URL changes
  (NavigationControllerImpl::RendererDidNavigate())
only as a result of NavigatorImpl::DidNavigate()

BUG=517089
TEST=None

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

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

This should prevent all commands run via timeout_retry.Run from triggering the
20-minute buildbot timeout. In particular, this should prevent long-running
gtest suites, e.g. the webrtc perf tests, from timing out.

BUG=

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

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

Previously we would directly pass developer-provided icons on to the
Android NotificationManager, but this has led to OOM errors on devices
with less memory when developers provide, supposedly, large icons.

Starting from this CL, we ensure that icons are at most the size at
which they will be presented to the user.

TBR=mvanouwerkerk (reviewed in CL 1362943002)
BUG=529980

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

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

This refactoring prepares the code to introduction of
NonBlockingDataTypeController for USS datatypes. The goal
was to split the very large NonBlockingDataTypeController
change into a couple of smaller to make it easier to review
and verify.

The following changes are included here:
1) Introduced ActivationContext which is a structure that
   contains all arguments needed to activate a USS datatype.
   For now ActivationContext is passed via SyncContext /
   SyncContextProxy, but the goal is to pass it directly
   via BackendDataTypeConfigurer as an argument for
   ActivateNonBlockingDataType.
   ActivationContext is needed as a separate class because
   NonBlockingDataTypeController will have to receive it
   from the type processor's callback and temporarily hold
   on to it.
2) BackendDataTypeConfigurer - two activation methods are
   renamed to be directory specific and two more activation
   methods for non-blocking data types are added.
3) DataTypeController cleanup - OnModelLoaded() virtual
   method shouldn't be on the base class because it applies
   only to some of the subclasses and is never invoked via the
   base class.

BUG=515962

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

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

This patch moves the more fundamental scheduling primitives (e.g.,
TaskQueueManager) to a separate base/ directory. This makes the layered
architecture of the scheduler more obvious and reduces the likelihood of
accidentally introducing layering violations.

With this refactoring, the overall dependencies are:

       +------+
       | base |
       +------+
           ^
           |
       +-------+     +------+
       | child |<----| test |
       +-------+     +------+
          ^ ^          ^ ^
          | |          | |
          | '----------|-|---.
          | .----------' '-. |
          | |              | |
      +----------+      +-------+
      | renderer |      | ppapi |
      +----------+      +-------+

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

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

SyncPolicyHandler depends on policy::TypeCheckingPolicyHandler which is
included in policy_component target.
GYP missed the dependency while BUILD.gn has it correctly.

BUG=

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

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

TBR=erikchen@chromium.org
BUG=536891
NOTRY=true

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

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

Summary of changes available at:
https://chromium.googlesource.com/skia/+log/209c162..c9e190d

CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
TBR=robertphillips@google.com

Commits in this roll:
c9e190d reed@google.com Revert of change pixel-serializer to support reencoding existing data (patchset #5 id:80001 of https://codereview.chromium.org/1373683003/ )
25a5b0d robertphillips@google.com Revert of SkImage-backed SkPictureShader (patchset #4 id:60001 of https://codereview.chromium.org/1343153003/ )
0869267 herb@google.com Add cast and assignment operators to SkAtomic.
13f48dc reed@google.com change pixel-serializer to support reencoding existing data

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

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

This CL sends the message to actually clear the browsing
data when the button is pressed.  It also adds enable/disable
based on messages from c++ about permissions and state.

BUG=532739

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

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

window.scroll properties being relative to visual viewport breaks some pages
under pinch-zoom. The original plan was to have all APIs reflect the layout
viewport (and introduce new APIs for visual). This CL is an initial step
towards that and the window.scroll properties will be relative to the layout
viewport if the 'inert-visual-viewport' flag is set.

BUG=489206

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

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

BUG=
TBR=kalman@chromium.org

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

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

Create new class UIStringOverrider that implements the logic to use the
data generated by generate_resources_map.py to override UI strings.

BUG=534257

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

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

BUG=536773,536601
TEST=new regression test.
TBR=chcunningham

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

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

This CL introduces IOSLiveTab, an iOS-specific implementation of the LiveTab
interface that is backed by WebState (the iOS analogue of WebContents).

BUG=371476

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

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

BUG=535731

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

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

BUG=527558
TEST=unit test

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

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

Previously the close button was only available for Custom Tabs in
the maximized state. Now it is present in the expanded state and in
regular Chrome.

BUG=534680

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

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

Disable per-frame timer scheduling for now since it is causing crashes
in situations where the underlying timer task runner becomes null.

BUG=536656,510398

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

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

Remove rpaquay@chromium.org, who is no longer working in this area.

Include armansito in test/OWNERS so that patches seeking reviewers
do not mistakenly see scheib as perferable to armansito.

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

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

+ Reorganising the context menus into better groupings
+ Renaming menu item labels for consistency
+ Removing items that are rarely used and have other access points (e.g. spellchecker submenu, page info)
+ Fix writing redirection UMA metrics collection

BUG=506874

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

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

BUG=511129

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

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

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