GitLab

Summary of changes available at:
https://chromium.googlesource.com/chromium/blink/+log/3be346d..0ba31e2

TBR=alph@chromium.org,jchaffraix@chromium.org

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

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

BUG=463125

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

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

BUG=470229

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

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

Only takes effect in --site-per-process mode.  Just the first phase of
adding frame entries; won't handle nested iframes or history navigations
yet.

BUG=236848
TEST=New unit and browser tests pass in --site-per-process mode.

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

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

Remove references to android_webview_build now that we no longer support
that build configuration. Fold all conditions into the general
OS==android sections.

BUG=440793

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

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

Enums used to simply be strings, but now we expose them (as string constants)
on the api object as part of the extension bindings. As such, it's appropriate
to have the externs generate an enum definition for them.

BUG=469920

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

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

BUG=462584

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

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

In this case Segoe UI and its linked fonts are already in the fallback list.
This is an attempt to fix the perf regression caused on the page_cycler.intl_hi_ru bot caused by the change to
remove duplicate fonts from the fallback list.

BUG=469700

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

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

R=brettw@chromium.org, erg@chromium.org, jamesr@chromium.org
TBR=jamesr@chromium.org

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

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

This makes the BoundedLabel override View::OnPaint, allowing us to make
View::Paint non-virtual.

The functional difference is that Paint() calls PaintCommon() which:
 - Early outs if not visible
 - Paints any children

The BoundedLabel is used by NotificationView, but no children are ever
added to it and it's not exposed outside the class. So it's safe to
convert it to use OnPaint() and remove the visible early out.

R=dewittj@chromium.org, mukai@chromium.org, stevenjb@chromium.org
BUG=466426

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

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

also, only cover the controls and not the instructions
also various tweaks to colors and font sizes and layout as per latest mocks.

BUG=none

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

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

This upstreams all the injected JS for the web layer (but not any of
the packaging bits, which will come later)

BUG=464810

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

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

This CL also have some renames and code changes related to state.

BUG=463671

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

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

This will enable building the ANGLE perf tests on Windows perf bot
builders.

BUG=angleproject:744
BUG=468852
R=sievers@chromium.org, thakis@chromium.org

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

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

Summary of changes available at:
https://chromium.googlesource.com/chromium/blink/+log/6b6978f..3be346d

TBR=alph@chromium.org,jchaffraix@chromium.org

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

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

In order to avoid future regressions.

BUG=469235
TEST=interactive_ui_tests --gtest_filter=AcceleratorInteractiveUITest.*

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

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

This logic was simplified and made more robust with
http://crrev.com/848003006, but there is still a report that this is still a
problem.

I am suspicious of the popover detection logic. The popover could belong
to any window, and could be in the middle of fading in or out. Change the
logic to a hopefully more robust on-demand detection using
[NSApplication targetForAction:@selector(performClose:].

BUG=47134

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

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

1.6.0c provides the ability for clients to manually provide an app version when instantiating the analytics service.

BUG=469258

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

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

Summary of changes available at:
https://chromium.googlesource.com/skia/+log/46bffd6..48dc6dc

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

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

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

There are assumptions that the IPC message limit is 128MB
so ChannelMojo has to allow messages in upto the message size.
Such large messages happen when IDB reads/writes huge records.

BUG=464151
R=agl@chromium.org

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

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

The answers in suggest doesn't put a space between, for example
+ the degrees Fahrenheit and the day of the week
+ the stock price and the stock delta (+/- value)

This CL puts a space between those values, which is more readable for the user.

BUG=

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

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

Gate out ICU initialization for OS_NACL,
as it calls other code that is not built for nacl.
Undetected in gyp build since we always build the nacl irt as release
with gc-sections.

Stop building the ppb_var_deprecated_proxy as it's only used in the
trusted build for flash.
Undetected in gyp build since we always build the nacl irt as release
with gc-sections.

BUG=https://code.google.com/p/chromium/issues/detail?id=462791
TEST=trybots
R=dmichael@chromium.org,dpranke@chromium.org

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

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

Design doc: https://docs.google.com/a/chromium.org/document/d/1PCQltNDdyZyuPHUdqYIjeObRrVOhf8WTr6BLfuTQ4ew/edit?pli=1#

BUG=468093
NOPRESUBMIT=true

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

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

The HTML encoding spec unified Big5 and Big5-HKSCS and we're not
distinguishing them any more.

The spec also has IBM866 that is not in the menu, yet.

BUG=None
TEST=manual:
1. load a page in ibm866 and check the encoding in More Tools|Encoding.
2. load pages labelled as in Big5 and Big5-HKSCS and check the encoding.
TEST=browser_tests --gtest_filter=*ncodin*
TEST=unit_tests --gtest_filter EncodingMenu*

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

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

BUG=439651

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

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

I extended the current CaptureAPIType exposure on chrome://media-internals page to work on ANDROID with the enum types defined below.

    enum CaptureApiType {
      API1,
      API2_LEGACY,
      API2_FULL,
      API2_LIMITED,
      TANGO,
      API_TYPE_UNKNOWN
    };

Enums are sourced from media/video/capture/video_capture_device.h file and auto-generated on the Android Java side by using java_cpp_enum scripts. They correspond to the same int values, and this is how it is communicated to cpp side on media/video/capture/android/video_capture_device_factory_android.cc.

                        +------------+
                        |VideoCapture|
                        +--+-------+-+
                           |       |
          +----------------+-+   +-+-----------------+
          |VideoCaptureCamera|   |VideoCaptureCamera2|
          +---+------------+-+   +-------------------+
              |            |
+-------------+-----+  ----+-------------+
|VideoCaptureAndroid|  |VideoCaptureTango|
+-------------------+  +-----------------+

On Java side, camera class structure is as above. CaptureApiType is defined by static getCaptureApiType() methods in each child.

Note that media/video/capture/android/imageformat_list.h is deleted as it is currently not used. Instead, enums defined on media/video/capture/android/video_capture_device_factory_android.h were autogenerated on Java side using java_cpp_enum scripts.

BUG=458743

TEST=Built with gyp and deployed:
- on Nexus 7, shows API2_LEGACY
- on Nexus 5, shows API2_LIMITED for front camera and API2_FULL for back camera
I added Android tests to MediaInternalsVideoCaptureDeviceTest.* and tested on Nexus 7.
Looked for the adding unittests on video_capture_device_unittest but ANDROID tests are currently disabled with a TODO specified[0]. I will investigate further, but let me know if you have suggestions.

[0] https://code.google.com/p/chromium/codesearch#chromium/src/media/video/capture/video_capture_device_unittest.cc&l=42

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

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

BUG=465884

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

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

This provides a list of new browser to run perf try jobs on tryserver.chromium.perf:
try-all: For tryjobs on all bisect bots (android, mac, win, winx64, linux).
trybot-all-android: tryjobs on all android bisect bots.
trybot-all-win: tryjobs on all windows bisect bots (win, win x64).
trybot-all-mac: tryjobs on all mac bisect bots.
trybot-all-linux: tryjobs on all linux bisect bots.

BUG=467124

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

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

Revert of Decrease the lag when switching between different categories in the Cros wallpaper selector. (patchset #5 id:140001 of https://codereview.chromium.org/1028513003/)

Reason for revert:
I suspect this is causing failures in the cros_trunk official builder:
http://master.chrome.corp.google.com:8011/builders/cros%20trunk/builds/31816

See https://code.google.com/p/chromium/issues/detail?id=470237

Original issue's description:
> Decrease the lag when switching between different categories in the Cros wallpaper selector.
>
> In the CL, we maintain a thumbnail list so that we don't need to fetch the thumbnail images from file system
> each time we switch between different categories. It might take some time at the first time, but will be
> instantly ready after that.
>
> And we also just preload grid items' thumbnail images in the current viewport, so that we don't experience noticeable
> lag when switching to a category at the first time.
>
> BUG=454932
> TEST=tested on devices (pixel & peppy)
>
> Committed: https://crrev.com/bb033ee318fc12de0b9e136c99acd25950b08e8f
> Cr-Commit-Position: refs/heads/master@{#322015}

TBR=bshe@chromium.org,xdai@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=454932

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

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

This makes the remaining NSS ports match the BoringSSL ports.

BUG=469387

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

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

BUG=141198
Test=The change can be previewed at:
https://chrome-apps-doc.appspot.com/_patch/1013383004/apps/tags/webview#type-FullscreenPermissionRequest
(I'm getting internal server error atm though)

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

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

BUG=462636
TBR=glider@chromium.org

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

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

Summary of changes available at:
https://chromium.googlesource.com/skia/+log/f5b17fb..46bffd6

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

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

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

Latest version of PepperFlash now includes the correct x-ppapi-arch on x64, so commit 0bc0349c can be safely reverted.

BUG=458894

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

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

In the case on the new avatar button, the UpdateAvatar...() methods just updates
the visibility of the button, not its contents, and the visibility doesn't depend
on the name.

I've also tried to name the methods a little more consistently. There's
a giant refactor that needs to be done to rename avatar_button to avatar_icon and
new_avatar_button to avatar_button, but this isn't that.

BUG=466692

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

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

Summary of changes available at:
https://chromium.googlesource.com/chromium/blink/+log/d0c7791..6b6978f

TBR=alph@chromium.org,jchaffraix@chromium.org

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

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

When we allow non-Service Workers to have access to the Cache Storage
API we will want to match other storage systems and use a database
identifier generated from a Blink SecurityOrigin object, rather than
just the origin URL, as the key for per-origin storage.

This change changes the computation of the directory name on disk (a
hash) from using the origin to a database identifier, in anticipation
of an identifier being passed in after some future CLs.

Existing data is preserved by preceding the uses of the directory name
(open, delete, or enumeration) with a directory rename from the
"legacy" name computed from the origin to the "new" name computed from
the directory identifier. The rename is done by posting a task to the
cache task runner just before the actual task. After a few releases we
remove the migration code.

R=michaeln,jkarlin
BUG=439389

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

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

When Chrome's leveldb Env (ChromiumEnv) switched to using base::File for I/O, in
crrev.com/710373002, it also (unintentionally) switched some logged histogram
values from errno to base::File::Error. Prior to r305020 some values were logged
with errno values, others with PlatformFileError values, and still others with
Windows Error values (i.e. GetLastError()). The Windows values were introduced
in r245135 - also uintentionally. Teasing these apart is too difficult now so
switching to a new set of histograms reporting base::File::Error's.

BUG=431914

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

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

The test sometimes triggers a shutdown race condition that results in
a UAF in blink::FocusController::focusedOrMainFrame() in the renderer,
due to WebFrameWidgetImpl outliving Page.  Disabling the test until
this is fixed.

BUG=470055,468319

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

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

Currently, when an activity is stopped, we explicitly hide the
foreground Tab. This is problematic, as current hiding semantics
might clear the visual front buffer before the window is hidden.
This in turn causes an unpleasant flickering during activity
transitions, e.g., when backgrounding Chrome or locking the screen.

Wire Activity onPause/onResume notifications to WindowAndroidObservers,
allowing the foreground tab to preserve its front buffer while hiding
its web content. If the tab is explicitly hidden, or the root window
is lost, the front buffer will be cleared as usual.

BUG=462752,434401

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

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