GitLab

The feature is always on regardless. The corresponding CL for removing the
runtime flag in blink is https://codereview.chromium.org/562403002

R=jochen@chromium.org

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

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

* Move CrOSInterface from backends into platform (CrOSPlatformBackend)
* Introduce Device/CrOSDevice.
* Replace IsRunningOnCrosDevice with GetHostPlatform().GetOSName()

BUG=413637

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

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

This pulls in the following Native Client changes:

r13731: (bradnelson) Turn on goma for nacl and switch windows to ninja, fix tls_edit weirdness.
r13732: (mcgrathr) Update GCC toolchains to r13729
r13733: (dschuff) Build x86-32 IRT with PNaCl

BUG=none
TEST=browser_tests and nacl_integration
CQ_EXTRA_TRYBOTS=tryserver.chromium.linux:linux_rel_precise32,linux_arm_cross_compile,linux_nacl_sdk,linux_chromium_dbg

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

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

This namespace was used for transition between old apps v1 and v2, which is
completed. There is no need to keep this redundant layer.

TEST=Manually tested Files app, Gallery and a Video player.
BUG=412653

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

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

This makes MediaStreamTrack::getSources go through UserMediaClient instead, needed to know which frame the call came from.

Blink CL: https://codereview.chromium.org/560843002/

BUG=406094

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

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

Next step is to treat service workers similar to shared workers.

BUG=none

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

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

This CL adds support for the mojo_base target on iOS that will be required to
implement the Mojo support that is feasible on iOS (namely, C++-C++
communication via Mojo and mojom interfaces).

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

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

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

This is also changing some design with the C++ instance being owned by
its Java counterpart. ShortcutBuilder is also removed.

BUG=None

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

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

We ignore the small volume change for preventing drift but we should not do it on un-muting.

BUG=412970
TEST=manual
R=fukino@chromium.org

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

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

These functions are actually in VideoCaptureDeviceFactory.

TBR=perkj@chromium.org
BUG=323913

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

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

Currently if suggestion popup is visible and options menu is
invoked from hardware options button then in that case suggestion
popup and options menu are overlapping.
Added code to dismiss suggestion popup if options menu is shown.

BUG=412757

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

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

Changing in the intialization order of WeakPtrFactory in src/athena module
such that all member variables should appear before the WeakPtrFactory
to ensure that any WeakPtrs to Controller are invalidated before its members
variable's destructors are executed, rendering them invalid.

BUG=303818

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

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

The blink scheduler needs to know when input events are received since
it uses that as a signal to prioritize compositor tasks
over other main thread tasks which reduces queueing durations and
input event latency.

BUG=391005, 411520

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

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

Forward compositor tasks to the Blink scheduler so that they can be
prioritized over other work as necessary. We now route input and
compositor tasks through the same scheduler queue to preserve their
relative ordering.

BUG=391005

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

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

This class is used to perform a full text search on bookmarks, powered by the server side. The superclass will be used for other services in subsequent CLs.

BUG=None

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

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

BUG=413219
(for trivial refactoring)
TBR=willchan@chromium.org,rsleevi@chromium.org,nkostylev@chromium.org

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

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

Maintaing the proper WeakPtrFactory destruction order to avoid the runtime
issue like unrleased memory,etc...
WeakPtrFactory should remain the last member so it'll be destroyed and
invalidate its weak pointers before any other members are destroyed.

BUG=303818

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

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

BUG=none
R=marja@chromium.org

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

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

Changing in the intialization order of WeakPtrFactory such that all
member variables should appear before the WeakPtrFactory to ensure
that any WeakPtrs to Controller are invalidated before its members
variable's destructors are executed, rendering them invalid.

BUG=303818

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

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

Changing in the intialization order of WeakPtrFactory such that all
member variables should appear before the WeakPtrFactory to ensure
that any WeakPtrs to Controller are invalidated before its members
variable's destructors are executed, rendering them invalid.

BUG=303818

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

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

We wait to verify that the user hasn't blacklisted a form before offering
generation. This fixes a bug where we may exit early due to no credentials
being autofillable and not inform the renderer that generation is allowed.

BUG=412496

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

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

Currently links in pdf open on mouse down. This doesn't
give user any chance to cancel the action by moving the mouse
without releasing the click.

This patch adds the fix to make sure pdf opens link on mouse up
and not on mouse down.

BUG=409753

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

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

This makes windows with a shape implicitly non-resizable and
non-maximizable.

BUG=408737

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

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

CL 93333002 added a main_render_frame() method to RenderFrameImpl,
then about a week later CL 107893003 added a virtual
GetMainRenderFrame() method with the same implementation.
To avoid confusion, we could just use the latter and remove
main_render_frame().

BUG=304341

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

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

Hooking it up where it makes sense in other mounts.

BUG=https://code.google.com/p/chromium/issues/detail?id=413397
TEST=None
R=binji@chromium.org

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

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

BUG=N/A
TEST=N/A

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

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

This CL cleans up some tile size code and centralizes the calculation of
the apps grid's size.

It also incidentally fixes a bug where the padding between pages was
smaller than it should have been.

BUG=411775

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

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

This gets rid of console warning on startup when maximized.

BUG=

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

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

I'm seeing my previous logging with empty window titles.

TBR=sky

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

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

Removed some unused code.
Updated interfaces from plain pointers to scoped_ptr<>
SaveTo(base::FilePath) replaced with SaveTo(base::File).

BUG=408184
TBR=sgurun@chromium.org

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

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

TBR=machenbach@chromium.org

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

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

Blink should make use of blink_resources.grd for the inline resources of user agent stylesheets.
This removes the dependency upon using make-file-arrays.py which embeds these resources as strings.

This patch adds the necessary plumbing required for the webkit_unit_tests target.
As the resources are being moved into blink_resources.grd, this generates blink_resources.pak.
This pak doesn't exists as a separate pak file but its repacked into "content_shell.pak".
The same must be loaded into the data_pack_ of the TestContentClient so that the webkit (blink) unit
tests run successfully.

Part 1: https://codereview.chromium.org/436843004/
Part 1 with conditional: https://codereview.chromium.org/440683004/
Part 2: https://codereview.chromium.org/422023008
Part 3: https://codereview.chromium.org/438843004/

BUG=312586

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

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

Revert of Force calling PostDisplayConfigurationChange() even if earyling out in UpdateDisplays (patchset #4 id:60001 of https://codereview.chromium.org/559213002/)

Reason for revert:
I am reverting this as it resulted in test VirtualKeyboardUsabilityExperimentTest.VirtualKeyboardWindowTest consistently crashing on Linux ChromiumOS Tests, see:

http://build.chromium.org/p/chromium.chromiumos/builders/Linux%20ChromiumOS%20Tests%20%28dbg%29%281%29/builds/31731
http://build.chromium.org/p/chromium.memory/builders/Linux%20Chromium%20OS%20ASan%20LSan%20Tests%20(1)/builds/2914

Stack trace:

Objects involved in the operation:
sequence "this" @ 0x0x3e568fbdcc90 {
}
Received signal 6
#0 0x7fcb9c8936fe base::debug::StackTrace::StackTrace()
#1 0x7fcb9c893230 base::debug::(anonymous namespace)::StackDumpSignalHandler()
#2 0x7fcb8c474cb0 \u003Cunknown>
#3 0x7fcb8bbc4425 gsignal
#4 0x7fcb8bbc7b8b abort
#5 0x7fcb8c1c75ad __gnu_debug::_Error_formatter::_M_error()
#6 0x7fcb9a306513 std::__debug::vector\u003C>::operator[]()
#7 0x7fcb9a2fbba9 ash::DisplayManager::GetCurrentDisplayIdPair()
#8 0x7fcb9a2d96cf ash::DisplayController::PostDisplayConfigurationChange()
#9 0x7fcb9a2d98bc ash::DisplayController::PostDisplayConfigurationChange()
#10 0x7fcb9a2fd22a ash::DisplayManager::UpdateDisplays()
#11 0x7fcb9a2fdbe8 ash::DisplayManager::SetDisplayRotation()
#12 0x7fcb9a33425b ash::VirtualKeyboardWindowController::FlipDisplay()
#13 0x7fcb9a3340d7 ash::VirtualKeyboardWindowController::UpdateWindow()
#14 0x7fcb9a2d92df ash::DisplayController::CreateOrUpdateNonDesktopDisplay()
#15 0x7fcb9a2d937f ash::DisplayController::CreateOrUpdateNonDesktopDisplay()
#16 0x7fcb9a300ea9 ash::(anonymous namespace)::NonDesktopDisplayUpdater::~NonDesktopDisplayUpdater()
#17 0x7fcb9a300df1 ash::DisplayManager::CreateMirrorWindowIfAny()

Original issue's description:
> Force calling PostDisplayConfigurationChange() even if earyling out in UpdateDisplays
>
> When changing from software mirroring mode to sinlge display mode, it
> is possible there is no need to update |displays_| and we early out
> UpdateDisplays(). But we still want to run the PostDisplayConfigurationChange()
> cause there are some clients need to act on this, e.g.
> TouchTransformerController needs to adjust the TouchTransformer when
> switching from dual displays to single display.
>
> BUG=chrome-os-partner:31868
> TEST=tested on Big, after existing software mirroring mode, the touch location
>      transformation is still correct.
>
> Committed: https://crrev.com/4802a8552a40e1f80606ca7171dc2f79930e7fb3
> Cr-Commit-Position: refs/heads/master@{#294481}

TBR=oshima@chromium.org,miletus@chromium.org
NOTREECHECKS=true
NOTRY=true
BUG=chrome-os-partner:31868

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

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

Revert of Do not create proxy hosts in the subtree of navigating frame. (patchset #11 id:200001 of https://codereview.chromium.org/536143002/)

Reason for revert:
SitePerProcessBrowserTest.ProxyCreationSkipsSubtree repeatedly fails on Mac with ASAN reporting use-after-free, see:

http://build.chromium.org/p/chromium.memory/builders/Mac%20ASan%2064%20Tests%20%281%29/builds/2174
http://build.chromium.org/p/chromium.fyi/builders/Chromium%20Mac%2010.8%20Buildrunner/builds/19009

@Nasko, as there are some simultaneous breakages right now, so I could not investigate this further and opted to revert the whole thing to get the tree green ASAP.

Snippets from test log:

ASSERTION FAILED: !localFrame || (localFrame->isLocalFrame())
../../third_party/WebKit/Source/core/frame/LocalFrame.h(253) : blink::LocalFrame *blink::toLocalFrame(blink::Frame *)
../../content/browser/site_per_process_browsertest.cc:690: Failure
Value of: child->child_count()
  Actual: 2
Expected: 0U
Which is: 0

Original issue's description:
> Do not create proxy hosts in the subtree of navigating frame.
>
> When a frame is navigating cross-process, its existing document will be completely destroyed and all child frames will be gone. This means that we don't need to create proxy objects for the new SiteInstance in the subtree of the navigating frame.
>
> BUG=357747
>
> Committed: https://crrev.com/faab4288197b20b73610aad3e5579751b0c3a97b
> Cr-Commit-Position: refs/heads/master@{#294520}

TBR=ajwong@chromium.org,kenrb@chromium.org,jam@chromium.org,creis@chromium.org,dmazzoni@chromium.org,nasko@chromium.org
NOTREECHECKS=true
NOTRY=true
BUG=357747

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

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

Changing in the intialization order of WeakPtrFactory such that all
member variables should appear before the WeakPtrFactory to ensure
that any WeakPtrs to Controller are invalidated before its members
variable's destructors are executed, rendering them invalid.

BUG=303818

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

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

When WM supports _NET_ACTIVE_WINDOW notification, we get OnActiveWindowChanged() from two places:
 - X11DesktopHandler::ProcessXEvent (FocusIn)
 - X11DesktopHandler::DispatchEvent (PropertyNotify)

One is enough, two can lead to confusion. FocusIn/Out should not be handled if WM supports active window.

BUG=None
TEST=None

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

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

TBR=sky

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

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

With the CL, https://codereview.chromium.org/456413002, we are migrating from embedding inline resources
inside blink towards using chromium's grd resource system. As part of this effort, most of the tests
would require to use the 'pak' files packaged as android application assets.

These assets need to be extracted to the application data directory during the startup of the test
application. The native libraries would be able to work with these 'pak' files only after the extraction
from the asset manager.

To achieve this goal, we would require that ResourceExtractor.java to be moved inside the org.chromium.base
pacakge so that it can be utilized in ChromeNativeTestActivity.java

BUG=312586

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

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

https://skia.googlesource.com/skia/+log/fbe4136845fd68f583f9f9bfb628706c4b87480b..1f777e8580a67f2f975151bc414a1695696aa635

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

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

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