summaryrefslogtreecommitdiffstats
path: root/android_webview/renderer
Commit message (Collapse)AuthorAgeFilesLines
* Move cache clearing from AwRenderViewExt to AwRenderFrameExt.avi2016-03-104-13/+19
| | | | | | | | | BUG=593473 TEST=no change Review URL: https://codereview.chromium.org/1785493002 Cr-Commit-Position: refs/heads/master@{#380494}
* blink: Rename modules/ method to prefix with get when they collide.Dana Jansens2016-03-091-1/+1
| | | | | | | | | | | | | | | | | | | 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}
* Sets a transparent background for out-of-process subframeslfg2016-02-262-3/+13
| | | | | | | | | | | | | | | | | 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}
* Remove the last long from Android IPCs.jam2016-02-092-3/+3
| | | | | | | | | | 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}
* Delete WebKit/public/web/WebSecurityOrigin.h.mek2016-02-041-1/+1
| | | | | | | | | | | | | 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}
* Remove obsolete histogram values from SSL.InsecureContentestark2016-01-152-2/+0
| | | | | | | | | | | | | | 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}
* Make sure only frames in the current frame tree send IPC to AwRenderViewHostExthush2016-01-144-301/+294
| | | | | | | | | | | | | | | | | | | | | | | | | | | | 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}
* Revert 3 CLs for shouldOverrideUrlLoading implementation.gsennton2016-01-122-0/+66
| | | | | | | | | | | | | | | | | | | | | | | | | | 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}
* Switch to standard integer types in android_webview/.avi2015-12-206-1/+9
| | | | | | | | | BUG=488550 TBR=mnaganov@chromium.org Review URL: https://codereview.chromium.org/1538223002 Cr-Commit-Position: refs/heads/master@{#366299}
* Enforce marking "override" for functions overriding Blink.Avi Drissman2015-12-071-4/+4
| | | | | | | | | | | | | | | 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}
* Revert of Enforce marking "override" for functions overriding Blink. ↵avi2015-12-071-4/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | (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}
* Enforce marking "override" for functions overriding Blink.avi2015-12-071-4/+4
| | | | | | | | | | | 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}
* Reland "Use resource throttle to implement shouldOverrideUrlLoading, core ↵gsennton2015-11-102-66/+0
| | | | | | | | | | | | | | | | | | 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}
* Propagate pageScaleFactor to GuestViews.mcnee2015-11-092-16/+8
| | | | | | | | | | | | | | | | | | | | | | | 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}
* Implement extended shouldOverrideUrlLoading callback in Android WebViewmnaganov2015-11-021-2/+4
| | | | | | | | | | | | | | | | | | | | | 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}
* Make NetErrorHelper more OOPIF-friendly.mmenke2015-10-272-4/+2
| | | | | | | | | | | | 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}
* Delete policyDocumentLoaderjaphet2015-10-192-4/+3
| | | | | | | | | | | | | | | 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}
* Revert "Use resource throttle to implement shouldOverrideUrlLoading, core ↵gsennton2015-10-052-0/+65
| | | | | | | | | | | | | | | | | | | | | | | 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}
* reland: android_webview: Initalize blink before using WebStringbashi2015-10-021-8/+10
| | | | | | | | | | | | | | | | 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}
* Revert of android_webview: Initialize blink before using WebString (patchset ↵phoglund2015-09-301-4/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | #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}
* android_webview: Initialize blink before using WebStringbashi2015-09-291-2/+4
| | | | | | | | | | | 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}
* Enable page visibility on WebView by defaulthush2015-09-292-8/+9
| | | | | | | | | | Also add a "--disable-page-visibility" commandline flag. BUG=520089 Review URL: https://codereview.chromium.org/1361903003 Cr-Commit-Position: refs/heads/master@{#351355}
* Update page scale factor when layout updateshush2015-09-282-14/+17
| | | | | | | | | | | | | | | 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}
* base: Template methods on Timer classes instead of the classes themselves.danakj2015-09-241-1/+1
| | | | | | | | | | | | | | | | | | | 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}
* Replace RenderViewImpl::IsEditableNode with WebElement::isEditable.esprehn2015-09-171-3/+2
| | | | | | | | | | | | | | | 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}
* Plumb smooth scrolling in Chromium compositorhush2015-09-082-0/+12
| | | | | | | | | | | | | | | | | | 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}
* aw: Page visiblity behind switchboliu2015-08-142-2/+9
| | | | | | | | | | | | | | | | | 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}
* [Android WebView] Prepare loading error messages for TCmnaganov2015-08-074-57/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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}
* [Android WebView] Put error page files into WebView resourcesmnaganov2015-08-055-9/+83
| | | | | | | | | | | | | | | | 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}
* The scheme granting code seems to become redundant, even formnaganov2015-08-051-2/+2
| | | | | | | | | | | | 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}
* Make UTF16ToASCII and UTF16TOUTF8 take a StringPiecebrettw2015-07-242-4/+4
| | | | | | | | | | | | | | | 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}
* Use resource throttle to implement shouldOverrideUrlLoading, core changegsennton2015-07-162-66/+0
| | | | | | | | | | | | | | | | | | 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}
* Check user gesture before firing an intent in WebViewmnaganov2015-07-091-1/+2
| | | | | | | | | | | | | 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}
* Don't put every <img> in the page in a WebVector in ↵esprehn2015-07-081-6/+5
| | | | | | | | | | | | | | | | | | | | | | | 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}
* Add more string_util functions to base namespace.brettw2015-06-241-2/+2
| | | | | | | | | | 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}
* Remove a few more uses of WebElement::innerText in Chromium.dglazkov2015-05-151-3/+3
| | | | | | | | | BUG=488088 R=boliu Review URL: https://codereview.chromium.org/1140253006 Cr-Commit-Position: refs/heads/master@{#330223}
* Remove the OOP PDF flagraymes2015-05-072-5/+0
| | | | | | | | | | | | | 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}
* Support Android secure codecs in requestMediaKeySystemAccess().sandersd2015-05-061-5/+1
| | | | | | | | | | 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}
* android_webview: Use v8::Local instead of v8::Handle.tfarina2015-04-301-3/+3
| | | | | | | | | | | | 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}
* Replace kMediaDrmEnableNonCompositing swicth with ↵ddorwin2015-04-021-1/+5
| | | | | | | | | | | | | | | 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}
* Refactor NavigationState to use NavigationParamsclamy2015-03-181-2/+2
| | | | | | | | | | | | | | | | 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}
* Fix crash when JS postMessage undefined to WebView.hush2015-03-101-3/+7
| | | | | | | | BUG=393291 Review URL: https://codereview.chromium.org/990533003 Cr-Commit-Position: refs/heads/master@{#319982}
* Rename blink::WebPermissionClient to blink::WebContentSettings.mlamouri2015-02-274-45/+46
| | | | | | | | | | | | | 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}
* Add is_same_page_navigation to RFO::DidCommitProvisionalLoad.mlamouri2015-02-272-2/+4
| | | | | | | | | | | 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}
* Implement the close() API for Message portssgurun2015-02-262-0/+7
| | | | | | | | | | | | | | | 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}
* Move V8ValueConverter from content/renderer to content/child.mek2015-02-132-1/+2
| | | | | | | | | | | 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}
* Update {virtual,override,final} to follow C++11 style.dcheng2015-02-047-44/+41
| | | | | | | | | | | | | | | | 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}
* Refactor webview to use //components/printingdgn2015-02-039-2706/+82
| | | | | | | | | | | | | | 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}
* Post a message from Java to JS using message channels.sgurun2015-01-292-9/+41
| | | | | | | | BUG=393291 Review URL: https://codereview.chromium.org/869133005 Cr-Commit-Position: refs/heads/master@{#313817}
* Replicate sandbox flags for OOPIF (Chromium part 2).alexmos2015-01-271-10/+0
| | | | | | | | | | | | | | | | * 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}
generated by cgit v1.1 at 2024-09-08 11:04:37 +0000