| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
|
|
|
|
|
|
| |
asynchronous. So far PipelineImpl handles asynchronous MediaFilter::Stop(). The actual change to make MediaFilter::Stop() asynchronous will be in another check in.
BUG=16059
TEST=Unit tests still runs
Review URL: http://codereview.chromium.org/2101015
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@48769 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
This CL cannot be checked in until the patch for:
https://bugs.webkit.org/show_bug.cgi?id=39917
has landed.
BUG=45373
Review URL: http://codereview.chromium.org/2330004
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@48754 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
This removes a dependency cycle and should also make life easier for DRT.
Revert http://codereview.chromium.org/1658012
Review URL: http://codereview.chromium.org/2358001
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@48731 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This makes two types of changes
- More int* -> int*_t transitions, since npapi.h switched over
- Replace TRUE/FALSE, which have been removed from the upstream npapi.h. Since all our usage is C++, true/false seemed cleaner than 1/0.
BUG=42645
TEST=Compiling still works
Review URL: http://codereview.chromium.org/2484002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@48725 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
This mirrors the same flag present in DumpRenderTree.
TEST=none
BUG=none
Review URL: http://codereview.chromium.org/2477002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@48661 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
WebKit roll 60462:60647
Original review: http://codereview.chromium.org/2422001
Review URL: http://codereview.chromium.org/2459002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@48628 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
and non-Windows.
BUG=none
TEST=none
Review URL: http://codereview.chromium.org/2421002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@48624 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
BUG= 44595, 30880
TEST= http://www.ich.org/LOB/media/MEDIA3917.pdf doesn't hang the browser.
TBR= tony
Review URL: http://codereview.chromium.org/2320006
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@48551 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Chrome plugin installer. The regression was caused due to the
mouse hook proc getting installed assuming that the current plugin was flash and setting and releasing capture thus messing up the button
controls capture state causing it to not fire the WM_COMMAND messages for plugin installation and cancellation.
Fix is to not install the mouse hook for internal plugins. The other fix is to have the mouse hook be a noop if the underlying window is
not our plugin window.
Fixes bug http://code.google.com/p/chromium/issues/detail?id=28025
Bug=28025
Review URL: http://codereview.chromium.org/2332004
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@48548 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
| |
Querying the plugin path and actual mime type is moved from creation of the
WebPluginDelegate to creation of the WebPlugin. This cleaned up some code.
R=jam
BUG=45289
TEST=none
Review URL: http://codereview.chromium.org/2262002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@48484 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
| |
platform specific shortcut. Also, this makes copying through the page menu work on Mac and Linux.
Review URL: http://codereview.chromium.org/2329001
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@48443 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
| |
BUG=10952
Review URL: http://codereview.chromium.org/2282004
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@48356 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Replaces the recently-removed pdf MIME-type stripping with a version
specifically targeted to QuickTime. Other plugins to handle PDF can just be
disabled by the user without side effects since they are specific to PDF.
BUG=None
TEST=QuickTime should continue to *not* handle PDFs; any other plugin should be able to register for PDF.
Review URL: http://codereview.chromium.org/2241006
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@48334 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
disable plugins
PDF Browser and O3D were blacklisted because having them handle content in Chrome was worse than not, but there was no way for users to prevent that without uninstalling the plugin entirely. Now that there's UI for disabling plugins, there's no need to keep them on the blacklist.
BUG=25068
TEST=PDF Browser and O3D should show up in about:plugins if installed (although current versions won't actually work).
Review URL: http://codereview.chromium.org/2270004
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@48331 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Basically the linux version of http://codereview.chromium.org/2075006
BUG=10952
TEST=Go to http://www.adobe.com/shockwave/welcome/. The task manager should show that the default plugin is running, and the "Missing Plugin" text should look slightly different than it did before. Also, the plugin area is now grey instead of transparent. (this test depends on a small webkit shown below)
Requires commenting out
if (objectContentType(url, mimeType) != ObjectContentNetscapePlugin)
return 0;
in WebKit/WebKit/chromium/src/FrameLoaderClientImpl.cpp's createPlugin() as well, else the default plugin won't load.
Review URL: http://codereview.chromium.org/2079016
TBR=thakis@chromium.org
Review URL: http://codereview.chromium.org/2291004
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@48325 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Basically the linux version of http://codereview.chromium.org/2075006
BUG=10952
TEST=Go to http://www.adobe.com/shockwave/welcome/. The task manager should show that the default plugin is running, and the "Missing Plug-in" text should look slightly different than it did before. Also, the plugin area is now grey instead of transparent. (this test depends on a small webkit shown below)
Requires commenting out
if (objectContentType(url, mimeType) != ObjectContentNetscapePlugin)
return 0;
in WebKit/WebKit/chromium/src/FrameLoaderClientImpl.cpp's createPlugin() as well, else the default plugin won't load.
Review URL: http://codereview.chromium.org/2079016
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@48315 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
| |
Review URL: http://codereview.chromium.org/2271002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@48296 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
| |
TEST=none
BUG=none
Review URL: http://codereview.chromium.org/2233001
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@48293 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
| |
TEST=tests still pass
Review URL: http://codereview.chromium.org/2150005
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@48240 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Instead of showing that the entire file is buffered when the video first loads, this uses current time, current bytes, and buffered bytes to estimate buffered time.
Reviewed: http://codereview.chromium.org/2085012/
Submitted for: vrk@chromium.org
BUG=42285
TEST=NONE
Review URL: http://codereview.chromium.org/2192001
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@48199 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
minor tweaks.
TEST=this is
BUG=none
Review URL: http://codereview.chromium.org/2080021
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@48125 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
the time according to the browser, and dispatching work to the main thread.
This CL also enables catching exceptions thrown by JS during a PPB_Var method.
R=brettw
BUG=none
TEST=none
Review URL: http://codereview.chromium.org/2148001
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@48075 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
| |
switched over to the new API.
Review URL: http://codereview.chromium.org/2122018
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@48053 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Fixes a bug with tracking CORS headers and starts tracking all requests
and all cross-site requests, for calculating the fraction we would block.
Also explicitly counts the requests we would block in new UMA histograms.
Currently has a problem with "304 Not Modified" responses, since no MIME
type is provided...
BUG=none
TEST=none
Review URL: http://codereview.chromium.org/2132002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@48003 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
browser and then converted to MSAA roles - this is both because the
logic needs to be more complicated, and so we can support Mac accessibility,
too. 2. The serializable object used to pass accessibility info from the
renderer to the browser now uses a map for uncommon attributes, rather than
having a bunch of fields that are almost always empty. 3. Handles the
accSelect method with TAKE_FOCUS, allowing assistive technology to set focus
to a particular control. 4. Implements several other IAccessible2 interfaces.
BUG=25564
BUG=13291
TEST=None
Review URL: http://codereview.chromium.org/2121004
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@47922 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
test won't be enabled.
However, when that flag is missing, A/B test is added to browser_main.cc. Trial A: use npn and spdy. B: use npn but no spdy. C: do nothing. A and B are set to zero for now
The histograms we collect are:
1. Page begin to finish time when spdy is enabled/disabled across all sites;
2. Page begin to finish time when spdy is used on sites that support spdy and when spdy is intentionally ignored.
BUG=43997
TEST=Don't use --use-spdy=npn, manually change _npn_nospdy and _npn_withspdy percentile and go to spdy supported sites.
Review URL: http://codereview.chromium.org/2036012
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@47896 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
R=brettw
BUG=none
TEST=none
Review URL: http://codereview.chromium.org/2086018
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@47892 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
This is the Windows patch. There will be separate patches for Mac and Linux.
BUG=42685
TEST=none
Review URL: http://codereview.chromium.org/2126010
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@47870 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
BUG=17392
TEST=Go to http://www.adobe.com/shockwave/welcome/. The task manager should show that the default plugin is running, and the "Missing Plug-in" text should look slightly different than it did before. (this test depends on a small webkit patch)
Requires commenting out
if (objectContentType(url, mimeType) != ObjectContentNetscapePlugin)
return 0;
in WebKit/WebKit/chromium/src/FrameLoaderClientImpl.cpp's createPlugin() as well, else the default plugin won't load.
<style type="text/css">
#glass {
position: absolute; width: 128px; height: 128px;
border: 1px solid rgba(0, 0, 0, 0.8);
-webkit-border-radius: 64px;
-webkit-transition-property: background-color, -webkit-box-shadow;
-webkit-transition-duration: 0.25s, 0.25s;
background-image: -webkit-gradient(radial,
50% 8%, 0, 50% -40%, 192, from(rgba(255, 255, 255, 0.4)),
color-stop(50%, rgba(255, 255, 255, 0.07)),
color-stop(51%, rgba(255, 255, 255, 0)),
color-stop(52%, rgba(0, 0, 0, 0.2)), color-stop(70%, rgba(0, 0, 0, 0.1)),
color-stop(99%, rgba(0, 0, 0, 0)), to(rgba(255, 255, 255, 0)));
background-color: #335;
-webkit-box-shadow: 0px 1px 4px rgba(0, 0, 0, 0.5);
color: white;
font-family: Helvetica, Arial, sans-serif;
font-size: 30px; font-weight: bold; line-height: 128px; text-align: center;
text-shadow: 0px -1px 2px rgba(0, 0, 0, 0.5);
}
#glass:hover {
background-color: #448;
-webkit-box-shadow: 0px 0px 50px rgba(0, 0, 255, 0.5);
}
</style>
<div id="glass">zomfg</div>
Review URL: http://codereview.chromium.org/2075006
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@47849 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
Patches by Frank Galligan (fgalligan@google.com), Tom Finegan (tomfinegan@google.com) and James Zern (jzern@google.com).
BUG=none
TEST=none
Review URL: http://codereview.chromium.org/2093007
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@47759 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Move some code into the anonymous namespace since it doesn't need to be exposed
outside of pepper_var.cc.
R=brettw
BUG=none
TEST=To be written once the new framework is available.
Review URL: http://codereview.chromium.org/2123009
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@47738 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
address.
Fixes address type matching in |AddressField::AddressTypeFromText| method. Also, extends |AutoFillManager::FillAutoFillFormData| method to fill associated billing address when filling a credit card that has billing information set in preferences. Adds new unit test AutoFillManagerTest.FillCreditCardFormWithBilling and fixes others.
BUG=44227
TEST=AddressFieldTest.ParseOneLineAddress, \
AddressFieldTest.ParseOneLineAddressBilling, \
AddressFieldTest.ParseOneLineAddressShipping, \
AutoFillManagerTest.FillCreditCardForm, \
AutoFillManagerTest.FillCreditCardFormWithBilling, \
FormStructureTest.HeuristicsSample8, \
and manual test with everything2.html bug file.
Review URL: http://codereview.chromium.org/2078016
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@47731 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
BUG=17392
TEST=Go to http://www.adobe.com/shockwave/welcome/. The task manager should show that the default plugin is running, and the "Missing Plugin" text should look slightly different than it did before. (this test depends on a small webkit patch)
Requires commenting out
if (objectContentType(url, mimeType) != ObjectContentNetscapePlugin)
return 0;
in WebKit/WebKit/chromium/src/FrameLoaderClientImpl.cpp's createPlugin() as well, else the default plugin won't load.
Review URL: http://codereview.chromium.org/2075006
TBR=thakis@chromium.org
Review URL: http://codereview.chromium.org/2132018
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@47730 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
BUG=17392
TEST=Go to http://www.adobe.com/shockwave/welcome/. The task manager should show that the default plugin is running, and the "Missing Plug-in" text should look slightly different than it did before. (this test depends on a small webkit patch)
Requires commenting out
if (objectContentType(url, mimeType) != ObjectContentNetscapePlugin)
return 0;
in WebKit/WebKit/chromium/src/FrameLoaderClientImpl.cpp's createPlugin() as well, else the default plugin won't load.
Review URL: http://codereview.chromium.org/2075006
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@47726 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
The test should now fit on an 800x600 display, which is apparently what the 10.6 bots have.
Removes the FLAKY marker since it should now pass on 10.6 bots.
BUG=36670
TEST=PluginConvertPointTest should pass on 10.6 bots
Review URL: http://codereview.chromium.org/2100008
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@47720 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
BUG=36670
TEST=N/A
Review URL: http://codereview.chromium.org/2101008
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@47698 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
in ppapi
TEST=none
BUG=none
Review URL: http://codereview.chromium.org/2072010
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@47681 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Plugins load in two stages: first we dlopen() them to query metadata,
then later in response to a page request we actually instantiate them.
The blacklist we had in place only happened at the latter point;
gecko-mediaplayer causes us to hang in the first step. So move the
blacklist check earlier to succesfully blacklist it.
We eventually should just fix the bugs that cause this plugin to hang us,
but this will fix things for users right now.
BUG=24507,40221
TEST=run with --log-level=0 --debug-plugin-loading and verify gecko-mediaplayer is still blacklisted
Review URL: http://codereview.chromium.org/2114011
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@47656 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
- Moved animation code for indeterminate bar from WebKit
because adding change above makes webkit-side
too complicated.
Note:
- This change depends https://webkit.org/b/39269
- Appearance of indeterminate bar remains incorrect.
I filed it to http://crbug.com/44433.
Patch by Hajime Morita <morrita@g>
Original code review: http://codereview.chromium.org/2131008/show
BUG=44430
TEST=none
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@47618 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
a response header immediately after sending data.
The work around is to be more aggressive about sending
multipart data before we have a response header. We used
to buffer a response until we had the full frame.
This avoids a crash in webcore by behaving more like
CFNetwork. It also makes us behave more like Gecko,
which is probably good for compat.
BUG=30880
Review URL: http://codereview.chromium.org/2070007
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@47599 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Before we were ignoring any volume values set prior to load(), leading to a mismatch between what the element was reporting and what people were hearing on their speakers.
Patch by simonjam@google.com:
http://codereview.chromium.org/2051008/show
BUG=33023
TEST=set volume on media element, then call load(), volume should be properly set
Review URL: http://codereview.chromium.org/2123006
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@47591 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
| |
pass in Chrome. Mostly, this involves better error checking.
The most substantial change is that I made TransportDIB able to return NULL on failure to make a PlatformCanvas, which we run into when makeing too large of a canvas from a plugin. I checked all the callers of this function to make sure they all handled the problem (many already did).
TEST=pulls Pepper test
BUG=-none
Review URL: http://codereview.chromium.org/2101004
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@47526 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
| |
for Linux64.
TBR=brettw
BUG=none
TEST=none
Review URL: http://codereview.chromium.org/2067014
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@47474 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
TBR=brettw
BUG=none
TEST=none
Review URL: http://codereview.chromium.org/2074010
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@47448 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This provides for basic access to the window and element containing the
instance. Enumeration of properties is not implemented.
R=brettw
BUG=none
TEST=none
Review URL: http://codereview.chromium.org/2075005
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@47444 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
| |
TEST=None
BUG=32407
Review URL: http://codereview.chromium.org/1742004
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@47419 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
| |
of freebsd, openbsd and possibly netbsd in the future
via sprewell
Review URL: http://codereview.chromium.org/2099003
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@47313 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
xxx-xxx-xxxx
Modifies the AutoFill label inferencing logic to skip over <img> tags when deriving labels. It is common to use
small images as spacers between elements. Also, extends the phone number heuristics to recognize numbers that are
split into three fields of the form [xxx]-[yyy]-[zzzz] where xxx is the area code, yyy is the phone number prefix and zzz
is the phone number suffix.
Note: This change has a dependency on WebKit change: https://bugs.webkit.org/show_bug.cgi?id=38825.
Will need to wait for that to land before committing this change.
BUG=38218
TEST=FormManagerTest.LabelsInferredWithImageTags:FormStructureTest.ThreePartPhoneNumber:PhoneFieldTest.ThreePartPhoneNumber
Review URL: http://codereview.chromium.org/2107004
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@47298 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Modifies the AutoFill label inferencing logic to skip over <img> tags when deriving labels. It is common to use
small images as spacers between elements. Also, extends the phone number heuristics to recognize numbers that are
split into three fields of the form [xxx]-[yyy]-[zzzz] where xxx is the area code, yyy is the phone number prefix and zzz
is the phone number suffix.
Note: This change has a dependency on WebKit change: https://bugs.webkit.org/show_bug.cgi?id=38825.
Will need to wait for that to land before committing this change.
BUG=38218
TEST=FormManagerTest.LabelsInferredWithImageTags:FormStructureTest.ThreePartPhoneNumber:PhoneFieldTest.ThreePartPhoneNumber
Review URL: http://codereview.chromium.org/1979008
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@47297 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
improvements to resource management in the Chrome Pepper layer.
TEST=none
BUG=none
Review URL: http://codereview.chromium.org/2096002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@47294 0039d316-1c4b-4281-b951-d872f2087c98
|