| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
|
|
|
|
| |
BUG=593473
TEST=no change
Review URL: https://codereview.chromium.org/1785493002
Cr-Commit-Position: refs/heads/master@{#380494}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This focuses on modules/ to rename methods that are named foo() and
that return a Foo*, which will have collisions when foo() is renamed
to Foo() in chromium style.
This patch uses the following to do many of the replacements, so it's
a bit on the larger side..
git gs '\b$foo()'|grep '\.\(cpp\|h\|cc\):'|cut -d: -f1|sort|uniq|xargs sed -ie 's/\b$foo()/$getFoo()/g'
R=haraken@chromium.org
TBR=brettw, chrishtr
BUG=582312
Review URL: https://codereview.chromium.org/1773813007 .
Cr-Commit-Position: refs/heads/master@{#380209}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
by default.
This CL moves background color from WebView to WebFrameWidget,
which allows for setting the background color independently per
local root. WebViewImpl still implements background color for
the main frame, but it now goes through a WebViewFrameWidget
interface.
BUG=515625
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
Review URL: https://codereview.chromium.org/1565893004
Cr-Commit-Position: refs/heads/master@{#377937}
|
|
|
|
|
|
|
|
|
|
| |
This is split off from https://codereview.chromium.org/1619363002/. After this change, we can remove the long traits on Android which is not compatible with 32 and 64 bit processes communicating.
BUG=581409
Review URL: https://codereview.chromium.org/1678593002
Cr-Commit-Position: refs/heads/master@{#374314}
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Somehow a code-reorg-gone-wrong left us with two copies of the
WebSecurityOrigin.h file, one in public/web and one in public/platform.
Since then the two files have diverged, but fortunately all the changes
were made to the correct copy in public/platform. This finished said
reorg by deleting the old no-longer-a-copy and updating all references
to refer to the new location.
Review URL: https://codereview.chromium.org/1671493003
Cr-Commit-Position: refs/heads/master@{#373662}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Mixed content histogram values that depend on the URL of the page have
since been rendered obsolete by Rappor, and they complicate mixed
content checking with OOPIFs. This CL deprecates the histogram values
that break down mixed content by Google property but preserves the basic
counters for displaying or running mixed content.
BUG=486936
Review URL: https://codereview.chromium.org/1583813003
Cr-Commit-Position: refs/heads/master@{#369851}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
There are 3 IPC messages from Renderer to Browser in question:
1) AwViewHostMsg_DocumentHasImagesResponse
2) AwViewHostMsg_UpdateHitTestData
3) AwViewHostMsg_OnContentsSizeChanged
1) This is a response to an IPC from browser
(AwViewMsg_DocumentHasImages) to do a query on the document of the main
frame as the current implementation. This CL matches that
implementation.
2) This IPC can come from blink changing focused node, or as a response
to Browser doing hit test at a specific location
(AwContents::RequestNewHitTestResultAt). It could be from a main frame
or a subframe of the current frame tree, because the focused node could
be anywhere.
3) This is sent from main frame whenever its content size changed.
This CL moves logic around these IPC messages into AwRenderFrameExt
(a RenderFrameObserver). Other logic in AwRenderViewExt remains, until
RenderView is removed.
BUG=570906
Review URL: https://codereview.chromium.org/1564613002
Cr-Commit-Position: refs/heads/master@{#369480}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Since https://codereview.chromium.org/1424263003 does not handle
overriding about:blank loads (or rather, about:blank loads are
handled synchronously and can thus be finished before we have a chance
to override them) we cannot land that patch without potentially breaking
lots of apps using WebView.
The second CL being reverted is a fix for the first one to post
didStopLoad and didFailLoad in the correct order.
The third CL being reverted here is a small change removing code that
is not used by https://codereview.chromium.org/1424263003 but is used
now that we revert that CL.
This reverts commit bae3ff13e6959b713b872df3c4a734d6a1ea01ad.
This reverts commit e1da704f67f34022b7aaf3afe277c86217844114.
This reverts commit c002c7ea9c5a3dbeb665e4070d852e6724d1ff86.
TBR=palmer@chromium.org,jam@chromium.org
BUG=325351,462306
Review URL: https://codereview.chromium.org/1530183004
Cr-Commit-Position: refs/heads/master@{#368849}
|
|
|
|
|
|
|
|
|
| |
BUG=488550
TBR=mnaganov@chromium.org
Review URL: https://codereview.chromium.org/1538223002
Cr-Commit-Position: refs/heads/master@{#366299}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
BUG=535367
TEST=it all stays working
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
R=thakis@chromium.org
TBR=ben@chromium.org
Committed: https://crrev.com/32ff552858e014d454ce8b181f5aca1dfcf8f8fd
Cr-Commit-Position: refs/heads/master@{#363482}
Review URL: https://codereview.chromium.org/1370923002 .
Cr-Commit-Position: refs/heads/master@{#363564}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
(patchset #19 id:360001 of https://codereview.chromium.org/1370923002/ )
Reason for revert:
Missed one.
Original issue's description:
> Enforce marking "override" for functions overriding Blink.
>
> BUG=535367
> TEST=it all stays working
> CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
> TBR=ben@chromium.org
>
> Committed: https://crrev.com/32ff552858e014d454ce8b181f5aca1dfcf8f8fd
> Cr-Commit-Position: refs/heads/master@{#363482}
TBR=thakis@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=535367
Review URL: https://codereview.chromium.org/1504913002
Cr-Commit-Position: refs/heads/master@{#363492}
|
|
|
|
|
|
|
|
|
|
|
| |
BUG=535367
TEST=it all stays working
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
TBR=ben@chromium.org
Review URL: https://codereview.chromium.org/1370923002
Cr-Commit-Position: refs/heads/master@{#363482}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
change"
Now that https://codereview.chromium.org/1381003004/ has landed we can
reland this patch since we now receive a didFinishLoad callback when
the current navigation finishes even in the case where a new navigation
has reached a provisional state.
This reverts commit e5c3091df04749b232359b6ef22938006af840d5.
TBR=palmer@chromium.org,jam@chromium.org
BUG=325351
Review URL: https://codereview.chromium.org/1424263003
Cr-Commit-Position: refs/heads/master@{#358719}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
We make all changes to pageScaleFactor available to the
browser process, so that WebContentsObservers may observe
them.
Previously, we were only sending notifications to the browser
process when the PageScaleFactorIsOne changed. Now that we're
notifying whenever the pageScaleFactor changed, we get rid of
this extra message and have the browser process handle the
PageScaleFactorIsOne logic as part of its handling of all
pageScaleFactor changes.
We also replace WebContentsImpl::ResetPageScale with
WebContentsImpl::SetPageScale, so that the browser process can
set any pageScaleFactor.
BUG=445319
Review URL: https://codereview.chromium.org/1408393003
Cr-Commit-Position: refs/heads/master@{#358631}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This includes:
-- passing 'isMainFrame' bit to WebViewClient.scheduleContentIntent
(in blink);
-- also passing this bit in AwViewHostMsg_ShouldOverrideUrlLoading;
-- refactoring of the "call client or launch intent" logic
in AwContentsClient;
-- extending and refactoring of
AwContentsClientShouldOverrideUrlLoadingTest suite.
BUG=549369
Review URL: https://codereview.chromium.org/1411173009
Cr-Commit-Position: refs/heads/master@{#357468}
|
|
|
|
|
|
|
|
|
|
|
|
| |
Now create one per frame instead of creating them only for main frames,
and get the WebFrame directly from a WebFrame rather than by going
through the RenderView.
BUG=543226,529976
Review URL: https://codereview.chromium.org/1406303002
Cr-Commit-Position: refs/heads/master@{#356395}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
The only reason this is needed is for a few callbacks in
content/ (decidePolicyForNavigation, loadURLExternally). Plumb
the needed bits, or read them off of RenderFrameImpl's state.
pending_navigation_params_ gets cleared when the DocumentLoader
is created, so doing policy checks before the DocumentLoader
means that any relevant information is still there.
BUG=545117
Review URL: https://codereview.chromium.org/1385123004
Cr-Commit-Position: refs/heads/master@{#354855}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
change"
Reverting because of crbug.com/534758, to fix that bug we will have to
make a blink change adding a callback to let WebView know when a
document is not longer loading. The current callback DidFinishLoad does
not fire if a new load is in its provisional state, which is sometimes
the case if we load a webpage containing a redirect in the form of a
href assignment: window.location.href = '...'.
This revert was not clean because of the transition to using
NavigationThrottles over in https://codereview.chromium.org/1363483007
This reverts commit 777bb78a111aee919e07f5206267915a87639f88.
TBR=palmer@chromium.org,jam@chromium.org
BUG=534758
Review URL: https://codereview.chromium.org/1375993004
Cr-Commit-Position: refs/heads/master@{#352318}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
The original CL (https://codereview.chromium.org/1373923005/) broke
an AndroidWebViewTest. Initialize blink after
RenderThread::AddObserver() calls.
Original description:
WebString is a part of blink so blink should be initialized before
using WebString.
BUG=501171
Review URL: https://codereview.chromium.org/1382983002
Cr-Commit-Position: refs/heads/master@{#351952}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
#1 id:1 of https://codereview.chromium.org/1373923005/ )
Reason for revert:
Appears to break AndroidWebViewTest: https://build.chromium.org/p/chromium.linux/builders/Android%20Tests/builds/22508.
Another possible culprit is https://codereview.chromium.org/1372873003, but I'll try this one first as it's more specifically connected to the failure.
Original issue's description:
> android_webview: Initialize blink before using WebString
>
> WebString is a part of blink so blink should be initialized before
> using WebString.
>
> BUG=501171
>
> Committed: https://crrev.com/421c442c6dc2fde27d22383b22086116dd9f6791
> Cr-Commit-Position: refs/heads/master@{#351416}
TBR=michaelbai@chromium.org,bashi@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=501171
Review URL: https://codereview.chromium.org/1383443002
Cr-Commit-Position: refs/heads/master@{#351512}
|
|
|
|
|
|
|
|
|
|
|
| |
WebString is a part of blink so blink should be initialized before
using WebString.
BUG=501171
Review URL: https://codereview.chromium.org/1373923005
Cr-Commit-Position: refs/heads/master@{#351416}
|
|
|
|
|
|
|
|
|
|
| |
Also add a "--disable-page-visibility" commandline flag.
BUG=520089
Review URL: https://codereview.chromium.org/1361903003
Cr-Commit-Position: refs/heads/master@{#351355}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
The base class for OneShotTimer and DelayedTimer is templated but
only the Start method needs to be (which has a TODO to make it go away
entirely too).
The DelayedTimer class is also templated but only its constructor
needs to be, and the type can be inferred at the callsite, so less
typing all around.
R=thakis@chromium.org
TBR=sky,brettw
BUG=148832
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
Review URL: https://codereview.chromium.org/1355063004
Cr-Commit-Position: refs/heads/master@{#350496}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
isEditable() handles more cases correctly and has tests, lets use that
instead. This also lets us remove WebNode::isContentEditable and
WebElement::isTextFormControlElement.
I also removed a null check on GetWebView() from
SpellCheckProvider::FocusedNodeChanged which didn't make sense. The lines
right above it call GetWebView()->focusedFrame(), so if it was null we would
have crashed and never gotten here.
Review URL: https://codereview.chromium.org/1349803002
Cr-Commit-Position: refs/heads/master@{#349367}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Android WebView uses android.widget.OverScroller#startScroll
implement pageUp/pageDown. Since blink internally knows how
to do a smooth scroll via page scale animation, WebView just
need to send an IPC to blink with the target position and
duration.
The blink change to support smooth scroll:
https://codereview.chromium.org/1251473003
BUG=378984
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
Review URL: https://codereview.chromium.org/1251323002
Cr-Commit-Position: refs/heads/master@{#347770}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Enable page visibility behind a command line switch. There should be
no change in behavior if enable-page-visibility switch is missing.
When it's added:
* Visibility is changed to
(view visible) && (!attached or window visible) && (!onPause-ed)
The existing implementation is simply !onPaused.
* Pass page visibility signal into blink, and in turn, the compositor.
BUG=520089
Review URL: https://codereview.chromium.org/1291653004
Cr-Commit-Position: refs/heads/master@{#343396}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
While trying to hook up new strings from crrev.com/1270793002 into TC,
I have realized that some changes need to be made to their definitions.
1. In Chrome, html fragments (grit 'structures') are not used. Instead, they
just put into .grd files as regular messages, with markup being isolated
inside fake placeholders to avoid the translators messing with it.
(and grit also requires non-empty 'example' tags to present for them).
Thus, removed 'structure.html' and put its contents into a scary-looking
message.
2. .grd files containing messages must not include anything, as only .grd
files themselves are copied into the internal tree for processing.
Thus, split the messages into a separate .grd file with no includes.
3. From the strings .grd, per-locale .pak files must be produced, and then
all those xxx_strings_xx.pak files repacked into a single xx.pak file.
As WebView already uses content_strings, changed the script to pack
content_strings together with the new strings, and renamed the script
accordingly.
4. As I would not like to repeat this process again would we need some
native strings on the browser part, and as we share native resources
between the browser and the renderer, moved the .grd files out from
'renderer' into a new 'ui' subdir.
BUG=515691
Review URL: https://codereview.chromium.org/1273263002
Cr-Commit-Position: refs/heads/master@{#342367}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Historically, the load error pages displayed by WebView were a part of
Android framework resources. This change adds them into WebView
resources. This way, the pages will be accessible to sandboxed child
services, using Chromium's resource sharing mechanisms.
Unlike Chrome's error pages, which use WebUI, WebView pages are
completely static, as JavaScript can be disabled.
BUG=515691
Review URL: https://codereview.chromium.org/1270793002
Cr-Commit-Position: refs/heads/master@{#341951}
|
|
|
|
|
|
|
|
|
|
|
|
| |
multi-process mode.
Also, removed duplicate constant for the "content:" scheme.
BUG=516546
Review URL: https://codereview.chromium.org/1269783007
Cr-Commit-Position: refs/heads/master@{#341909}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This is for consistency with the other UTF APIs. This is annoying in some cases because a WebString can be converted implicitly to a string16, but not a StringPiece, necessitating explicit conversions when this is done. And we can't have both versions because they're ambiguous. But this change will allow greater use of StringPieces without copying.
The parameters to the UTF converters that take StringPieces were converted to by-value given the current guidance in StringPiece.
The main non-search-and-replace change is in aw_render_view_ext.cc which saves two string copies.
NOPRESUBMIT=true
(due to legacy wstrings)
Review URL: https://codereview.chromium.org/1259673002
Cr-Commit-Position: refs/heads/master@{#340306}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
We have been using both a resource throttle and a sync IPC to implement
shouldOverrideUrlLoading, with this patch we use only a resource
throttle instead.
This patch depends on https://codereview.chromium.org/1194383003 which
adds a flag indicating that the current request was overridden. The flag
is passed down to AwWebContentsObserver.didFailLoad. That CL in turn
depends on https://codereview.chromium.org/1178273007/ which adds the
flag to blink errors.
BUG=325351
Review URL: https://codereview.chromium.org/1155713005
Cr-Commit-Position: refs/heads/master@{#339109}
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Only fire intents when navigation is initiated by the user
(via a gesture), or by the server (a redirect). This curbs
down attempts to launch intents purely by JS code, which
lead to a poor UX -- tasks appearing from nowhere.
BUG=501633
Review URL: https://codereview.chromium.org/1180223004
Cr-Commit-Position: refs/heads/master@{#338133}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
AwRenderViewExt::OnDocumentHasImagesRequest.
This method was doing:
blink::WebVector<blink::WebElement> images;
webview->mainFrame()->document().images(images);
hasImages = !images.isEmpty();
which meant Blink would walk the entire document putting every <img>
into a Vector, then it would copy that Vector into this WebVector all
so this code could check if the Vector wasn't empty.
Instead just use the NodeList version of getElementsByHTMLTagName,
which we can already just call in this file with GetImgChild. This
only walks until it finds the first <img> and doesn't need additional
storage.
Review URL: https://codereview.chromium.org/1220333011
Cr-Commit-Position: refs/heads/master@{#337920}
|
|
|
|
|
|
|
|
|
|
| |
Moves ReplaceFirstSubstringAfterOffset. It also changes the find/replace arguments to be string pieces (normally these are constants).
TBR=gauravsh@chromium.org for chromeos/network
Review URL: https://codereview.chromium.org/1200393002
Cr-Commit-Position: refs/heads/master@{#335999}
|
|
|
|
|
|
|
|
|
| |
BUG=488088
R=boliu
Review URL: https://codereview.chromium.org/1140253006
Cr-Commit-Position: refs/heads/master@{#330223}
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This removes the flag for enabling OOP PDF since it is enabled by default and
the in process viewer no longer works anyway. The old in-process browser tests
have been removed. They are mostly replaced by pdf_extension_test.cc and they
are disabled anyway.
BUG=483870,436444
Review URL: https://codereview.chromium.org/1128453003
Cr-Commit-Position: refs/heads/master@{#328687}
|
|
|
|
|
|
|
|
|
|
| |
This adds the bits necessary to determine if a configuration requires hardware-secure codecs. It does not add the logic to actually enable secure surfaces based on the requirement, that decision is still based on the renderer preference.
BUG=467779
Review URL: https://codereview.chromium.org/1106263004
Cr-Commit-Position: refs/heads/master@{#328464}
|
|
|
|
|
|
|
|
|
|
|
|
| |
Turns out, Handle is just an alias for Local:
https://chromium.googlesource.com/v8/v8.git/+/master/include/v8.h#334
BUG=424445
R=boliu@chromium.org,sgurun@chromium.org
Review URL: https://codereview.chromium.org/1113193002
Cr-Commit-Position: refs/heads/master@{#327813}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
use_video_overlay_for_embedded_encrypted_video pref
Disables kMediaDrmEnableNonCompositing by default in WebView.
It is now controlled by the pref.
Also updates comments related to enabling unprefixed EME.
BUG=470275,470279
Review URL: https://codereview.chromium.org/1045173002
Cr-Commit-Position: refs/heads/master@{#323420}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This CL separates NavigationState into a public interface with a limited number
of functions exposed outside of content/, and an implementation. The
implementation now owns a set of NavigationParams. For browser-initiated
navigations they are passed by the RenderView, and for renderer-intiated
navigations they have default values. This also allows to reset the
pending_navigation_params of RenderView with their values when the provisional
load fails for browser-initiated navigations.
BUG=459033
Review URL: https://codereview.chromium.org/1001043002
Cr-Commit-Position: refs/heads/master@{#321120}
|
|
|
|
|
|
|
|
| |
BUG=393291
Review URL: https://codereview.chromium.org/990533003
Cr-Commit-Position: refs/heads/master@{#319982}
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This is part of a series of three CLs:
1. https://codereview.chromium.org/962713002
2. <this>
3. https://codereview.chromium.org/962733002
BUG=462255
Review URL: https://codereview.chromium.org/960983002
Cr-Commit-Position: refs/heads/master@{#318462}
|
|
|
|
|
|
|
|
|
|
|
| |
This seems to be something that most consumers want to know. Better to
pass a boolean instead of expecting all consumers to do the right thing.
BUG=459242
Review URL: https://codereview.chromium.org/950653002
Cr-Commit-Position: refs/heads/master@{#318441}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
The close API has to consider these cases:
1. A message port can be in a pending state
2. A message could be queued in renderer waiting for conversion
Therefore close() immediately closes the port, but it does not
cleanup the resources until all the messages are transferred.
BUG=393291
Review URL: https://codereview.chromium.org/956763002
Cr-Commit-Position: refs/heads/master@{#318271}
|
|
|
|
|
|
|
|
|
|
|
| |
This will be used by changes in content/child/webmessageportchannel_impl.cc
in a future CL to enable conversions between v8::Value and base::Value.
BUG=426458
Review URL: https://codereview.chromium.org/921543002
Cr-Commit-Position: refs/heads/master@{#316117}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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 an OS=android build using a
variation of https://codereview.chromium.org/598073004.
BUG=417463
R=sgurun@chromium.org
Review URL: https://codereview.chromium.org/893273003
Cr-Commit-Position: refs/heads/master@{#314467}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Some printing related //android_webview were copied over from
//chrome. Now that the ones in //chrome can move to a component,
webview should also use these files.
Depends on https://codereview.chromium.org/811563008/
BUG=444883, 311308
Review URL: https://codereview.chromium.org/822133003
Cr-Commit-Position: refs/heads/master@{#314307}
|
|
|
|
|
|
|
|
| |
BUG=393291
Review URL: https://codereview.chromium.org/869133005
Cr-Commit-Position: refs/heads/master@{#313817}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
* Remove old versions of createChildFrame.
* Add browser tests.
Related CLs:
Chromium part 1: https://codereview.chromium.org/837283003/
Blink part 1: https://codereview.chromium.org/793493003/
Blink part 2: https://codereview.chromium.org/838903002/
BUG=426512
Review URL: https://codereview.chromium.org/797813006
Cr-Commit-Position: refs/heads/master@{#313384}
|